[PATCH] D8467: C++14: Disable sized deallocation by default due to ABI breakage

Reid Kleckner via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Oct 29 11:20:22 PDT 2021


rnk added a comment.

In D8467#3095610 <https://reviews.llvm.org/D8467#3095610>, @zixuan-wu wrote:

> Hi, I am wondering could -fsized-deallocation this be enabled by default nowadays in 2021?

Yes, I think so. This should be a matter of adjusting the default in the driver. We should not bring back the code that defines the sized deallocation thunk, so a straight revert of this patch doesn't make sense.

I don't have time to work on this at the moment, but I would approve a patch.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D8467



More information about the cfe-commits mailing list