================
@@ -1063,10 +1063,18 @@ void CudaToolChain::AddClangSystemIncludeArgs(const 
ArgList &DriverArgs,
 
   if (DriverArgs.hasFlag(options::OPT_offload_inc, options::OPT_no_offload_inc,
                          true) &&
-      CudaInstallation.isValid())
+      CudaInstallation.isValid()) {
+    StringRef IncludePath = CudaInstallation.getIncludePath();
     CC1Args.append(
-        {"-internal-isystem",
-         DriverArgs.MakeArgString(CudaInstallation.getIncludePath())});
+        {"-internal-isystem", DriverArgs.MakeArgString(IncludePath)});
+
+    // From CUDA 13.0 on, libcu++, CUB and Thrust are in include/cccl, and some
----------------
carodewig wrote:

I'm new to this ecosystem so not sure if there's a reason for the divergence -- 
would you be comfortable keeping `-internal-isystem` here for now and 
considering alignment with HIP separately?

https://github.com/llvm/llvm-project/pull/235910
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to