[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