[PATCH] D157438: [OpenMP] Ensure wrapper headers are included on both host and device
Matt Arsenault via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Tue Aug 8 14:22:24 PDT 2023
arsenm added inline comments.
================
Comment at: clang/lib/Driver/ToolChains/Clang.cpp:1190-1191
// the resource directory at clang/lib/Headers/llvm_libc_wrappers.
- if (C.getActiveOffloadKinds() == Action::OFK_None) {
+ if ((getToolChain().getTriple().isNVPTX() ||
+ getToolChain().getTriple().isAMDGCN()) &&
+ C.getActiveOffloadKinds() == Action::OFK_None) {
----------------
can we do something better than this NVPTX||AMDGCN checks
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D157438/new/
https://reviews.llvm.org/D157438
More information about the cfe-commits
mailing list