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
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
