aaronpuchert added a comment.
Yeah, we should get this over the line. I'm still not quite sure where to put
the check. Reading @rsmith's comment again, SemaInit might perhaps be
acceptable for now, except that I should add the additional tests (in case we
don't have them already).
I think we
shafik added a comment.
Herald added a project: All.
It looks like this is almost there, can we get it over the line?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D113838/new/
https://reviews.llvm.org/D113838
rsmith added a comment.
This is CWG issue 2351. Please add a corresponding test to
`tests/CXX/drs/dr23xx.cpp`.
Comment at: clang/lib/Sema/SemaInit.cpp:1311-1319
+ } else if (DeclType->isVoidType()) {
+// [expr.type.conv]p2: Otherwise, if the type is cv void and the
+
aaronpuchert added a subscriber: Tyker.
aaronpuchert added a comment.
CC @Tyker for the changes to `SemaCXX/attr-annotate.cpp`.
Comment at: clang/include/clang/Basic/DiagnosticSemaKinds.td:5848-5849
def err_illegal_initializer_type : Error<"illegal initializer type %0">;
+def
aaronpuchert created this revision.
aaronpuchert added a reviewer: rsmith.
aaronpuchert requested review of this revision.
Herald added a project: clang.
Herald added a subscriber: cfe-commits.
This is explicitly allowed via [expr.type.conv], if the initialization
list is empty.
Repository:
