[libcxx-commits] [libcxx] [libcxx] Align `__recommend() + 1` by __endian_factor (PR #90292)

[Louis Dionne via libcxx-commits]

ldionne wrote:

> Is there a reason why gcc has this on by default, but clang doesn't?

Basically, we had trouble with rolling it out. It's a wide-reaching change to deploy in the wild because it's effectively an ABI change (call sites that used to call `operator delete(void*)` will now sometimes call `operator delete(void*, size_t)`), and it just took a long time for us to be comfortable and able to make it the default. For example, making `-fsized-deallocation` enabled by default required many rounds of fixing tests and other stuff just within LLVM. Or at least that's my understanding.

https://github.com/llvm/llvm-project/pull/90292