rjmccall added a comment. In D8467#3125471 <https://reviews.llvm.org/D8467#3125471>, @rnk wrote:
> 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. Okay. I think that's fine for most platforms. Apple would like this to only be enabled conditionally based on deployment target. 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
