[PATCH] D48062: Fix that AlignedAllocation.h doesn't compile because of VersionTuple
Raphael Isemann via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Mon Jun 11 20:21:13 PDT 2018
teemperor created this revision.
https://reviews.llvm.org/rL334399 put VersionTuple in the llvm namespace, but this header still assumes it's in the clang namespace.
This leads to compilation failures with enabled modules when building Clang.
Repository:
rC Clang
https://reviews.llvm.org/D48062
Files:
include/clang/Basic/AlignedAllocation.h
Index: include/clang/Basic/AlignedAllocation.h
===================================================================
--- include/clang/Basic/AlignedAllocation.h
+++ include/clang/Basic/AlignedAllocation.h
@@ -22,18 +22,18 @@
namespace clang {
-inline VersionTuple alignedAllocMinVersion(llvm::Triple::OSType OS) {
+inline llvm::VersionTuple alignedAllocMinVersion(llvm::Triple::OSType OS) {
switch (OS) {
default:
break;
case llvm::Triple::Darwin:
case llvm::Triple::MacOSX: // Earliest supporting version is 10.13.
- return VersionTuple(10U, 13U);
+ return llvm::VersionTuple(10U, 13U);
case llvm::Triple::IOS:
case llvm::Triple::TvOS: // Earliest supporting version is 11.0.0.
- return VersionTuple(11U);
+ return llvm::VersionTuple(11U);
case llvm::Triple::WatchOS: // Earliest supporting version is 4.0.0.
- return VersionTuple(4U);
+ return llvm::VersionTuple(4U);
}
llvm_unreachable("Unexpected OS");
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D48062.150884.patch
Type: text/x-patch
Size: 962 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20180612/367b0ff9/attachment.bin>
More information about the cfe-commits
mailing list