[PATCH] D112921: [clang] Enable sized deallocation by default in C++14 onwards

Reid Kleckner via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Nov 12 09:36:49 PST 2021


rnk added a comment.

Let's not bring back the weak function thunk approach. That approach caused problems described in my commit, D8467 <https://reviews.llvm.org/D8467>, which is why the code was removed.

The next steps are to sort out right defaults for Apple and understand the libc++ test failures. Would it be reasonable to take a shortcut here, leave the feature disabled for Apple targets, and defer those details to those that own the target?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D112921



More information about the cfe-commits mailing list