[PATCH] D71811: [PowerPC] Fixes -Wrange-loop-analysis warnings
Aaron Ballman via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Sun Dec 22 09:11:17 PST 2019
aaron.ballman accepted this revision.
aaron.ballman added a comment.
This revision is now accepted and ready to land.
Herald added a subscriber: wuzish.
LGTM with a request to drop top-level `const`s
================
Comment at: llvm/lib/Target/PowerPC/PPCLoopInstrFormPrep.cpp:553
// times, and we need to add it the right number of times.
- for (const auto &PI : predecessors(Header)) {
+ for (const auto PI : predecessors(Header)) {
if (PI != LoopPredecessor)
----------------
Drop top-level `const` here and elsewhere?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D71811/new/
https://reviews.llvm.org/D71811
More information about the llvm-commits
mailing list