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

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to