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

Tue Aug 29 05:46:46 PDT 2023

wangpc added a comment.

In D112921#4624141 <https://reviews.llvm.org/D112921#4624141>, @nikic wrote:

> FYI this resulted in some pretty wild code size swings, in particular between -10% and -15% for tramp3d-v4 (http://llvm-compile-time-tracker.com/compare.php?from=6cde64a94986165547ae5237ac7dd4bddfc9f2a7&to=2916b125f686115deab2ba573dcaff3847566ab9&stat=size-text). Not sure whether that's an expected result of this change or not.

I'm surprised. From the tests diff in this patch, it should increase code size I think.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D112921