[PATCH] D145906: [clang-tidy] Correctly handle evaluation order of designated initializers.

Martin Böhme via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Mar 20 02:55:23 PDT 2023


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



More information about the cfe-commits mailing list