This is an archive of the discontinued LLVM Phabricator instance.

[NFC][llvm-mca] Fix compiler warning
ClosedPublic

Authored by pguo on Feb 3 2021, 8:24 PM.

Details

Summary

Fix clang compiler warning from -Wrange-loop-analysis.

Diff Detail

Event Timeline

pguo created this revision.Feb 3 2021, 8:24 PM
pguo requested review of this revision.Feb 3 2021, 8:24 PM
Herald added a project: Restricted Project. · View Herald TranscriptFeb 3 2021, 8:24 PM

This is a duplicate of D95321.
No idea why it is taking so long for that other patch to go in.

pguo updated this revision to Diff 321487.Feb 4 2021, 9:29 AM
pguo retitled this revision from [llvm-mca] Fix compiler warning to [NFC][llvm-mca] Fix compiler warning.

Update commit message.

pguo updated this revision to Diff 321491.Feb 4 2021, 9:36 AM

Update commit message.

This revision was not accepted when it landed; it landed in state Needs Review.Feb 4 2021, 9:47 AM
This revision was landed with ongoing or failed builds.
This revision was automatically updated to reflect the committed changes.
pguo added a comment.Feb 4 2021, 10:12 AM

@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.

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.

pguo added a comment.Feb 4 2021, 1:44 PM

Thanks. I had a wrong impression that you approved this before, maybe I am confused by some flags from Harbormaster.

No problem. Your change was good.
Thanks for fixing the warning.