SPIRV-LLVM-Translator: fix handling of built-in SPIRV targer#3166
SPIRV-LLVM-Translator: fix handling of built-in SPIRV targer#3166lumag wants to merge 1 commit intoKhronosGroup:mainfrom
Conversation
If the SPIRV-LLVM-Translator is being built together with the LLVM, set the defines declaring whether SPIRV target is also included into the LLVM library or not. Otherwise llvm-spirv execution fails because spirv-ext option is registered twice. Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@oss.qualcomm.com>
|
|
|
According to my information, this contribution should be covered by the Corporate OS CLA between Qualcomm and Khronos and as such should not require any extra CLA. Could you please check it? |
@KhronosWebservices could you please comment on that? |
|
I do not see @lumag in our Corporate CLA system. Can you please email me webservices@khronosgroup.org. Thanks. |
If the SPIRV-LLVM-Translator is being built together with the LLVM, set the defines declaring whether SPIRV target is also included into the LLVM library or not. Otherwise llvm-spirv execution fails because spirv-ext option is registered twice.