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

Louis Dionne via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Apr 25 14:17:03 PDT 2022


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



More information about the cfe-commits mailing list