aaron.ballman added a comment.

In D151162#4369717 <https://reviews.llvm.org/D151162#4369717>, @SlaterLatiao 
wrote:

> In D151162#4369584 <https://reviews.llvm.org/D151162#4369584>, @aaron.ballman 
> wrote:
>
>> Please be sure to add a release note for the change, and it'd probably be 
>> good to have a test case that shows this triggers under `-Wall` (the 
>> modified test case explicitly names the diagnostic).
>
> Thank you, Aaron. I'll add the test case. Should this change go under 
> `Improvements to Clang’s diagnostics` in the release note?

Yes, that sounds like a good place for the note. Thank you!


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D151162/new/

https://reviews.llvm.org/D151162

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to