mboehme added a comment.

In D145906#4198583 <https://reviews.llvm.org/D145906#4198583>, @PiotrZSL wrote:

> Regarding release-notes:
> I would probably wrote something like "fixed handling for designated 
> initializers", nonene will understand things like "sequence point".

Thanks, makes sense.

I've got another fix for the same checker in the pipeline anyway, so I've 
decided to roll this rewording into that patch: https://reviews.llvm.org/D145581

> If you decide to change this, the just commit this as an [NFC] without review.
> And checks in this section should be in alphabetical order (so this one 
> shouldn't be at the end), but that's a different story that can be corrected 
> later for all those entrys.

Right, I remember that too now...

As there are other entries that are also out of order, I'll just leave this 
where it is for now.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D145906

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

Reply via email to