[PATCH] D95299: Fix truncated __OPENMP_NVPTX__ preprocessor condition

2023-08-15 Thread Mehdi AMINI via Phabricator via cfe-commits
This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rGc49142e4f5c8: Fix truncated __OPENMP_NVPTX__ preprocessor condition (authored by r-burns, committed by mehdi_amini). Repository: rG LLVM Github

[PATCH] D95299: Fix truncated __OPENMP_NVPTX__ preprocessor condition

2023-07-24 Thread Ryan Burns via Phabricator via cfe-commits
r-burns added a comment. Hi! You can credit Ryan Burns . Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D95299/new/ https://reviews.llvm.org/D95299 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[PATCH] D95299: Fix truncated __OPENMP_NVPTX__ preprocessor condition

2023-07-24 Thread Mehdi AMINI via Phabricator via cfe-commits
mehdi_amini added a comment. Herald added a subscriber: wangpc. @r-burns : what should I use for the authorship (name/pseudo and email)? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D95299/new/ https://reviews.llvm.org/D95299

[PATCH] D95299: Fix truncated __OPENMP_NVPTX__ preprocessor condition

2023-06-23 Thread Nicolas Capens via Phabricator via cfe-commits
capn added a comment. Can a committer go ahead and land this please? I'm running into this warning for any CUDA code compiled by Clang. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D95299/new/ https://reviews.llvm.org/D95299

[PATCH] D95299: Fix truncated __OPENMP_NVPTX__ preprocessor condition

2021-01-25 Thread Artem Belevich via Phabricator via cfe-commits
tra added reviewers: tra, ABataev. tra accepted this revision. tra added a comment. This revision is now accepted and ready to land. LGTM Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D95299/new/ https://reviews.llvm.org/D95299

[PATCH] D95299: Fix truncated __OPENMP_NVPTX__ preprocessor condition

2021-01-23 Thread Ryan Burns via Phabricator via cfe-commits
r-burns created this revision. Herald added subscribers: guansong, yaxunl. r-burns requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. I don't have a test case for this but noticed this warning when including system headers with `-I` rather