[PATCH] D79800: [Sema] Implement DR2233
Raul Tambre via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Sat May 30 03:09:02 PDT 2020
tambre abandoned this revision.
tambre added a comment.
In D79800#2058985 <https://reviews.llvm.org/D79800#2058985>, @rsmith wrote:
> Unfortunately, I think this approach basically can't work, because we need to consider inheriting default arguments from a previous (or subsequent!) declaration before dropping default arguments preceding a pack. I think we will instead need to detect this situation when issuing the diagnostic for a parameter with a default argument followed by a parameter without one, and will need to make sure that all the parts of Clang that look at default arguments can cope with them being discontiguous.
Thanks for reviewing!
Abandoning, as I lack the motivation to implement the proper solution you laid out.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D79800/new/
https://reviews.llvm.org/D79800
More information about the cfe-commits
mailing list