This revision was automatically updated to reflect the committed changes.
Closed by commit rC320401: P0620 follow-up: deducing `auto` from
braced-init-list in new expr (authored by lichray).
Changed prior to commit:
https://reviews.llvm.org/D39451?vs=126092&id=126400#toc
Repository:
rC Clang
rsmith accepted this revision.
rsmith added inline comments.
This revision is now accepted and ready to land.
Comment at: include/clang/Basic/DiagnosticSemaKinds.td:1992
+def ext_auto_new_list_init : Extension<
+ "ISO C++ standards before C++17 does not allow new expression for
lichray added a comment.
Ping?
Repository:
rC Clang
https://reviews.llvm.org/D39451
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
lichray updated this revision to Diff 126092.
lichray added a comment.
Keep a pedantic Extension warning.
Repository:
rC Clang
https://reviews.llvm.org/D39451
Files:
include/clang/Basic/DiagnosticSemaKinds.td
lib/Sema/SemaExprCXX.cpp
test/CXX/expr/expr.unary/expr.new/p2-cxx0x.cpp
tes
rsmith added a comment.
EDG and MSVC do not appear to treat this as a defect resolution; I suspect this
is an oversight in GCC rather than an intentional extension. Let's convert the
error to an (off by default) pedantic `Extension` (ISO C++11 does not allow
...), and suppress the extension war
lichray added a comment.
Ping. Just accept this as a DR, like what GCC does, I guess?
https://reviews.llvm.org/D39451
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
lichray created this revision.
This is a side-effect brought in by p0620r0, which allows other placeholder
types (derived from `auto` and `decltype(auto)`) to be usable in a `new`
expression with a single-clause //braced-init-list// as its initializer (8.3.4
[expr.new]/2). N3922 defined its se
