[PATCH] D108379: [OpenCL] Fix version reporting of C++ for OpenCL 2021
Justas Janickas via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Thu Aug 19 08:19:50 PDT 2021
Topotuna created this revision.
Topotuna added a reviewer: Anastasia.
Herald added subscribers: ldrumm, dexonsmith, yaxunl.
Topotuna requested review of this revision.
Herald added a project: clang.
Herald added a subscriber: cfe-commits.
C++ for OpenCL version 2021 and later are expected to consist of a
major version number only. Therefore, a different constructor for
`VersionTuple` needs to be called when reporting language version.
Repository:
rG LLVM Github Monorepo
https://reviews.llvm.org/D108379
Files:
clang/lib/Basic/LangOptions.cpp
Index: clang/lib/Basic/LangOptions.cpp
===================================================================
--- clang/lib/Basic/LangOptions.cpp
+++ clang/lib/Basic/LangOptions.cpp
@@ -47,6 +47,8 @@
VersionTuple LangOptions::getOpenCLVersionTuple() const {
const int Ver = OpenCLCPlusPlus ? OpenCLCPlusPlusVersion : OpenCLVersion;
+ if (OpenCLCPlusPlus && Ver != 100)
+ return VersionTuple(Ver / 100);
return VersionTuple(Ver / 100, (Ver % 100) / 10);
}
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D108379.367506.patch
Type: text/x-patch
Size: 468 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20210819/ef116c2b/attachment.bin>
More information about the cfe-commits
mailing list