[PATCH] D71814: [Hexagon] Fixes -Wrange-loop-analysis warnings

Aaron Ballman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Dec 22 09:11:18 PST 2019


aaron.ballman accepted this revision.
aaron.ballman added a comment.
This revision is now accepted and ready to land.

LGTM with a request to drop top-level `const`s



================
Comment at: llvm/lib/Target/Hexagon/HexagonLoopIdiomRecognition.cpp:2338
         return false;
-      for (const auto &K : In.users()) {
+      for (const auto K : In.users()) {
         Instruction *UseI = dyn_cast<Instruction>(K);
----------------
Drop top-level `const`?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D71814





More information about the llvm-commits mailing list