[PATCH] D125265: [OpenMP] Don't include the device wrappers if -nostdinc is used

Artem Belevich via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon May 9 13:07:22 PDT 2022


tra accepted this revision.
tra added a comment.
This revision is now accepted and ready to land.

LGTM in general.



================
Comment at: clang/lib/Driver/ToolChains/Clang.cpp:1311
   if (JA.isDeviceOffloading(Action::OFK_OpenMP) &&
+      !Args.hasArg(options::OPT_nostdlib) &&
       (getToolChain().getTriple().isNVPTX() ||
----------------
jhuber6 wrote:
> tra wrote:
> > Did you mean `nostdinc` ?
> Yes, thank you for catching that. That was an autocomplete mishap that I accidentally reintroduced from undoing too many times. I'll fix it.
A test would be useful.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D125265



More information about the cfe-commits mailing list