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

[Alexey Bataev via Phabricator via cfe-commits]

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



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