Fix clang compiler warning from -Wrange-loop-analysis.
Details
Details
Diff Detail
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
Comment Actions
This is a duplicate of D95321.
No idea why it is taking so long for that other patch to go in.
Comment Actions
@andreadb could you approve this again? I made minor changes to commit message, and it invalidates the approval status. Sorry to bother you again. Thanks.
Comment Actions
Not sure I understand why you would need an approval now. You have already committed this change.
See: https://reviews.llvm.org/rG91e7a17133324ac4beaf6ed45c170436c2d91c98
P.s.: I never approved it. However it is fine since it fixes a warning, and it is identical to D95321 which is already approved.
Comment Actions
Thanks. I had a wrong impression that you approved this before, maybe I am confused by some flags from Harbormaster.