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

[Gheorghe-Teodor Bercea via Phabricator via cfe-commits]

gtbercea marked an inline comment as done.
gtbercea 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);
----------------
ABataev wrote:
> 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.
Even better!! I'll make the change!


Repository:
  rC Clang

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

https://reviews.llvm.org/D60568