[PATCH] D71364: Fix no-assert ununsed variable.

Florian Hahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Dec 11 10:13:55 PST 2019


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

LGTM thanks, in case you are waiting for approval in Phabricator. But as mentioned earlier, there's no need for patches like this to go through review.



================
Comment at: llvm/lib/Analysis/LoopInfo.cpp:369
   BasicBlock *Preheader = getLoopPreheader();
   BasicBlock *Latch = getLoopLatch();
+  (void)Latch;
----------------
FWIW if Latch is not used elsewhere, it would be easier to just call getLoopLatch in the assert, rather than adding an unused variable to the scope.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D71364





More information about the llvm-commits mailing list