You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
It would be really convenient if it was possible to build SPIRV reflect as a static or shared library so multiple things could link against it at the same time without having to deal with symbol conflicts.
Additionally, it would be useful to be able to configure the CMakeLists to only build said library and optionally turn off the main executable and stripper util.
If this is something you're happy for me to do, it probably shouldn't be a massive hassle.
Many thanks,
Alister
The text was updated successfully, but these errors were encountered:
- SPIRV_REFLECT_EXECUTABLE toggles building of spirv-reflect
executable. Default is ON.
- SPIRV_REFLECT_EXAMPLES toggles building of examples. Default is ON.
- SPIRV_REFLECT_STRIPPER toggles building of stripper util. Default is
OFF.
- SPIRV_REFLECT_sTATIC_LIB togges building of spirv-reflect as a static
library. Default is OFF. (Added in previous commit)
I added some build options to toggle building of the various targets in 05b6ec3.
I was only able to add a build target for a static library. The shared library is going to take some amount of non-trivial work to get all the export declarations in place. I'll have to find a larger chunk of time for that.
To build only the static library, add these CMake command line arguments: -DSPIRV_REFLECT_STATIC_LIB=1 -DSPIRV_REFLECT_EXECUTABLE=0 -DSPIRV_REFLECT_EXAMPLES=0 -DSPIRV_REFLECT_STRIPPER=0
Hi there,
It would be really convenient if it was possible to build SPIRV reflect as a static or shared library so multiple things could link against it at the same time without having to deal with symbol conflicts.
Additionally, it would be useful to be able to configure the CMakeLists to only build said library and optionally turn off the main executable and stripper util.
If this is something you're happy for me to do, it probably shouldn't be a massive hassle.
Many thanks,
Alister
The text was updated successfully, but these errors were encountered: