MitalAshok wrote:

Looking at other places, it looks like init-list stuff is guarded behind 
`getLangOpts().CPlusPlus11`, so I'll add that check.

It looks like this DR is CD5 (after C++17, applies to C++17), but `void{}` in 
C++11/14 without a warning seems fine.

As for "regular tests", do you mean in 
[`/clang/test/CXX`](https://github.com/llvm/llvm-project/tree/main/clang/test/CXX)?
 Should I also add tests there?

I also found this: https://stackoverflow.com/q/41131806 that `void({})` 
apparently used to (erroneously) be accepted in Clang 3.8 but I can't find any 
tests for it

https://github.com/llvm/llvm-project/pull/78060
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to