[PATCH] D60568: [OpenMP] Add support for registering requires directives with the runtime

Alexey Bataev via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue May 21 06:29:05 PDT 2019


ABataev added inline comments.


================
Comment at: lib/Frontend/CompilerInvocation.cpp:2840
       Opts.OpenMP && Args.hasArg(options::OPT_fopenmp_is_device);
+  Opts.OpenMPHasTargets =
+      Opts.OpenMP && Args.hasArg(options::OPT_fopenmp_targets_EQ);
----------------
We already have `Opts.OMPTargetTripples`. We use it currently to check if the offloading enabled `!CGM.getLangOpts().OMPTargetTriples.empty()`, so, probably, we don't need this new flag.


Repository:
  rC Clang

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

https://reviews.llvm.org/D60568





More information about the cfe-commits mailing list