@@ -188,6 +190,28 @@ void
SPIRVTargetCodeGenInfo::setCUDAKernelCallingConvention(
}
}
+LangAS
+SPIRVTargetCodeGenInfo::getGlobalVarAddressSpace(CodeGenModule &CGM,
+ const VarDecl *D) const {
+ assert(!CGM.getLangOpts().OpenC
asudarsa wrote:
> I'm okay with the patch in general, but I'd like either @michalpaszkowski or
> @VyacheslavLevytskyy to take a look. Just one question about doubling the
> test scope for LLVM tests with spir64 target triple.
+1 on this. I will take a look before end of week on this. Thanks
h
https://github.com/asudarsa approved this pull request.
Thanks for this change. LGTM
https://github.com/llvm/llvm-project/pull/88455
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
asudarsa wrote:
Thanks @AlexVlx for this change. This should work fine for
SPIRV-LLVM-Translator (and SPIR-V backend). Adding @michalpaszkowski for input
from SPIR-V backend side. Recently, this restriction on LLVM IR input to our
translator was docuemnted:
https://github.com/KhronosGroup/SPIR