[C++14] Enable sized deallocation by default #60061
Assignees
Labels
Comments
|
@llvm/issue-subscribers-c-14 |
EugeneZelenko
added
the
clang:driver
|
@llvm/issue-subscribers-clang-driver |
|
Hi @wangpc-pp, Do you have any status on enabling this by default which you last tried this summer (https://reviews.llvm.org/D112921) ? |
|
This feels like a duplicate of: #56505 We should try and keep one main bug and updates and queries should go to that. |
There were some buildbot failures (Windows, AMDGPU) and I don't have such environments, so I suspended the progress. |
wangpc-pp
added a commit
that referenced
this issue
Apr 26, 2024
Since C++14 has been released for about nine years and most standard libraries have implemented sized deallocation functions, it's time to make this feature default again. This is another try of https://reviews.llvm.org/D112921. Fixes #60061
|
Reverted with #90299 |
wangpc-pp
added a commit
to wangpc-pp/llvm-project
that referenced
this issue
Apr 28, 2024
Since C++14 has been released for about nine years and most standard libraries have implemented sized deallocation functions, it's time to make this feature default again. This is another try of https://reviews.llvm.org/D112921. The original commit cf5a8b4 was reverted by 2e5035a due to some failures (see llvm#83774). Fixes llvm#60061
wangpc-pp
added a commit
to wangpc-pp/llvm-project
that referenced
this issue
May 22, 2024
Since C++14 has been released for about nine years and most standard libraries have implemented sized deallocation functions, it's time to make this feature default again. This is another try of https://reviews.llvm.org/D112921. The original commit cf5a8b4 was reverted by 2e5035a due to some failures (see llvm#83774). Fixes llvm#60061
wangpc-pp
added a commit
that referenced
this issue
May 22, 2024
#90373) Since C++14 has been released for about nine years and most standard libraries have implemented sized deallocation functions, it's time to make this feature default again. This is another try of https://reviews.llvm.org/D112921. The original commit cf5a8b4 was reverted by 2e5035a due to some failures (see #83774). Fixes #60061
vitalybuka
pushed a commit
that referenced
this issue
Jun 12, 2024
#90373) Since C++14 has been released for about nine years and most standard libraries have implemented sized deallocation functions, it's time to make this feature default again. This is another try of https://reviews.llvm.org/D112921. The original commit cf5a8b4 was reverted by 2e5035a due to some failures (see #83774). Fixes #60061
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Sized deallocation is a C++14 feature and it was disabled in clang due to potential ABI breaking (https://reviews.llvm.org/D8467). There was guys trying to enable it by default 2 years ago (https://reviews.llvm.org/D112921) but the work suspended later.
The text was updated successfully, but these errors were encountered: