LWG4113: Disallow has_unique_object_representations<Incomplete[]>
#118350
Labels
c++26
clang:frontend
Language frontend issues, e.g. anything involving "Sema"
libc++
libc++ C++ Standard Library. Not GNU libstdc++. Not libc++abi.
lwg-issue
this is an issue that was filed to the Library Working Group
Comments
|
It seems better to let compiler frontends to handle this. |
|
@llvm/issue-subscribers-clang-frontend Author: Louis Dionne (ldionne)
**Link:** https://wg21.link/LWG4113
|
True, but we should add tests for it still. We're actually lacking any kind of test coverage to ensure we're rejecting incorrect type traits uses properly. |
cor3ntin
added a commit
to cor3ntin/llvm-project
that referenced
this issue
May 2, 2025
…ete[] This implements [LWG4113](https://cplusplus.github.io/LWG/issue411) This is technically a breaking change, but it's a fix and I think anyone who relies on this to day is in a world of hurt. Fixes llvm#118350
cor3ntin
added a commit
that referenced
this issue
May 2, 2025
…ete[] (#138291) This implements [LWG4113](https://cplusplus.github.io/LWG/issue411) This is technically a breaking change, but it's a fix, and I think anyone who relies on this today is in a world of hurt. Fixes #118350
|
@philnik777 I modified the builtin, I don't know if you want to add more tests on the libc++ side |
|
@cor3ntin yeah, I think we should keep this open until we've added tests on the libc++ side. |
IanWood1
pushed a commit
to IanWood1/llvm-project
that referenced
this issue
May 6, 2025
…ete[] (llvm#138291) This implements [LWG4113](https://cplusplus.github.io/LWG/issue411) This is technically a breaking change, but it's a fix, and I think anyone who relies on this today is in a world of hurt. Fixes llvm#118350
IanWood1
pushed a commit
to IanWood1/llvm-project
that referenced
this issue
May 6, 2025
…ete[] (llvm#138291) This implements [LWG4113](https://cplusplus.github.io/LWG/issue411) This is technically a breaking change, but it's a fix, and I think anyone who relies on this today is in a world of hurt. Fixes llvm#118350
IanWood1
pushed a commit
to IanWood1/llvm-project
that referenced
this issue
May 6, 2025
…ete[] (llvm#138291) This implements [LWG4113](https://cplusplus.github.io/LWG/issue411) This is technically a breaking change, but it's a fix, and I think anyone who relies on this today is in a world of hurt. Fixes llvm#118350
GeorgeARM
pushed a commit
to GeorgeARM/llvm-project
that referenced
this issue
May 7, 2025
…ete[] (llvm#138291) This implements [LWG4113](https://cplusplus.github.io/LWG/issue411) This is technically a breaking change, but it's a fix, and I think anyone who relies on this today is in a world of hurt. Fixes llvm#118350
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Link: https://wg21.link/LWG4113
The text was updated successfully, but these errors were encountered: