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

Joseph Huber via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Aug 29 05:49:22 PDT 2023


jhuber6 added a subscriber: sivachandra.
jhuber6 added a comment.

In D112921#4624580 <https://reviews.llvm.org/D112921#4624580>, @wangpc wrote:

> It seems that the linker can't find sized deallocation (no support in the environment or AMDGPU libraries?).

We should have some implementations here I thought https://github.com/llvm/llvm-project/blob/main/libc/src/__support/CPP/new.cpp, maybe @sivachandra can elucidate on that. I actually don't know what the expected behavior is here, since we don't really have any of this support on GPU targets. I'm wondering why we're currently fine with compiling `delete` but not with the sized version, since their definitions should both be present in that file.


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