gregrodgers added a comment.

We want amdgcn to remain on old deviceRTL till we have verified it .   I made 
inline comments on how this could be done.



================
Comment at: clang/lib/Driver/ToolChains/Clang.cpp:5905
       // runtime.
       if (Args.hasFlag(options::OPT_fopenmp_target_new_runtime,
                        options::OPT_fno_openmp_target_new_runtime,
----------------
Add a check for amdgcn something like this. 


================
Comment at: clang/lib/Driver/ToolChains/Clang.cpp:5907
                        options::OPT_fno_openmp_target_new_runtime,
-                       /*Default=*/false))
+                       /*Default=*/true))
         CmdArgs.push_back("-fopenmp-target-new-runtime");
----------------
This will keep AMDGCN on old runtime till we are ready to switch.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D114890/new/

https://reviews.llvm.org/D114890

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to