https://github.com/Sirraide commented:

Implementation seems fine, but this is still missing a release note (in 
`clang/docs/ReleaseNotes.rst`, probably somewhere in the ‘Potentially Breaking 
Changes’ section).

CC @AaronBallman @cor3ntin Do we want to make this a warning that defaults to 
an error instead? Imo it’s probably fine as-is because I don’t imagine too many 
people would write code that abuses us not diagnosing this...

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

Reply via email to