[libcxx-commits] [PATCH] D112921: [clang] Enable sized deallocation by default in C++14 onwards

[Louis Dionne via Phabricator via libcxx-commits]

ldionne added a subscriber: ahatanak.
ldionne added inline comments.


================
Comment at: clang/lib/Frontend/CompilerInvocation.cpp:4078-4082
+  if (!Args.getLastArg(options::OPT_fsized_deallocation,
+                       options::OPT_fno_sized_deallocation))
+    Opts.SizedDeallocation = Opts.SizedDeallocation &&
+                             T.getVendor() != llvm::Triple::VendorType::Apple;
+
----------------
Why not implement it correctly from the start? @arphaman @ahatanak What would be the right incantation to enable sized deallocation starting in macOS 10.12, iOS 10.0, watchOS 3.0, and tvOS 10.0?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D112921/new/

https://reviews.llvm.org/D112921