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

Wang Pengcheng via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Aug 29 05:32:18 PDT 2023


wangpc added a comment.

In D112921#4624488 <https://reviews.llvm.org/D112921#4624488>, @thakis wrote:

> Looks like this breaks tests on windows: http://45.33.8.238/win/83485/step_7.txt
>
> Please take a look and revert for now if it takes a while to fix.
>
> (Also, if the patch doesn't already do it, it probably shouldn't change defaults in clang-cl mode?)

The two failed cases are about `Interpreter`, which is not related to `clang-cl` mode I think? Is it because the windows environment doesn't support sized deallocation?
I don't know how to fix it, can someone help me?


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