[clang] [clang][SPIRV] Default AS generic for Intel-flavored SPIR-V (PR #153647)

[Victor Lomuller via cfe-commits]

================
@@ -219,8 +219,11 @@ class LLVM_LIBRARY_VISIBILITY BaseSPIRTargetInfo : public 
TargetInfo {
     setAddressSpaceMap(
         /*DefaultIsGeneric=*/Opts.SYCLIsDevice ||
         // The address mapping from HIP/CUDA language for device code is only
-        // defined for SPIR-V.
-        (getTriple().isSPIRV() && Opts.CUDAIsDevice));
+        // defined for SPIR-V, and all Intel SPIR-V code should have the 
default
+        // AS as generic.
+        (getTriple().isSPIRV() &&
----------------
Naghasan wrote:

Given the trend, wouldn't changing the condition to `!Opts.OpenCL && 
getTriple().isSPIRV()` ? Although I'm not sure `getTriple().isSPIRV()` is 
strictly required

https://github.com/llvm/llvm-project/pull/153647
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits