[clang] [clang-tools-extra] [libcxx] [clang] Enable sized deallocation by default in C++14 onwards (PR #83774)

Louis Dionne via cfe-commits cfe-commits at lists.llvm.org
Fri Mar 8 12:46:08 PST 2024


================
@@ -2912,16 +2912,70 @@ static bool sdkSupportsBuiltinModules(const Darwin::DarwinPlatformKind &TargetPl
   }
 }
 
-void Darwin::addClangTargetOptions(const llvm::opt::ArgList &DriverArgs,
-                                   llvm::opt::ArgStringList &CC1Args,
-                                   Action::OffloadKind DeviceOffloadKind) const {
+static inline llvm::VersionTuple
+sizedDeallocMinVersion(llvm::Triple::OSType OS) {
+  switch (OS) {
+  default:
+    break;
+  case llvm::Triple::Darwin:
+  case llvm::Triple::MacOSX: // Earliest supporting version is 10.12.
----------------
ldionne wrote:

I checked and these versions seem correct. I'd be curious to know how you managed to find that out, though, cause I had to dig pretty deep!

https://github.com/llvm/llvm-project/pull/83774


More information about the cfe-commits mailing list