https://github.com/zyn0217 requested changes to this pull request.
I wonder if we should fix the DR1344 implementation instead:
If you remove the parameter pack, clang would even not diagnose
`addition of default argument on redeclaration makes this constructor a
%select{default|copy|move}0 constructor`: https://gcc.godbolt.org/z/ePexa67hM
which IMO was somehow overlooked.
And the current approach your AI suggested seemed to make us even lose that
diagnose, which is worse.
https://github.com/llvm/llvm-project/pull/217386
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits