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

[Reid Kleckner via Phabricator via cfe-commits]

rnk added a comment.

In D8467#3125386 <https://reviews.llvm.org/D8467#3125386>, @rjmccall wrote:

> Conceptually, this is (and will always be) a platform decision.  On Apple platforms, we have a formalized concept of deployment target, where specific minimum OS versions support sized deallocation and others do not.  On most other platforms, this is much vaguer — basically down to what C++ libraries you've got installed — and probably has to be controlled by a flag.  Enabling that flag by default on any particular Linux distribution release is something I'm not sure we can do unconditionally.

It is already a flag, `-fsized-deallocation`. On some level, whatever default we choose is just a guess about the C++ library support level. Clang tries to encode all kinds of Linux distro-specific knowledge, and it's often wrong anyway. After all that logic, there will be some default. I think at this point the feature should default to being enabled.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D8467