[PATCH] D114890: [OpenMP] Make the new device runtime the default

[Greg Rodgers via Phabricator via cfe-commits]

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