From 9ea0d8c38fc58ecec2e2f62f3b65e99fcbc2ec4c Mon Sep 17 00:00:00 2001 From: Florian Hahn Date: Wed, 15 Jul 2020 11:49:24 +0100 Subject: [LoopRotate] Remove unnecessary verifyMemorySSA calls. The actual rotation happens in processLoop, so the second removed call to verifyMemorySSA was unnecessary. In fact, processLoop/rotateLoop already verify MemorySSA before and after transforming each loop. Hence, both calls can be removed. Pointed out by @lebedev.ri post-commit D51718. --- llvm/lib/Transforms/Utils/LoopRotationUtils.cpp | 5 ----- 1 file changed, 5 deletions(-) (limited to 'llvm/lib/Transforms/Utils/LoopRotationUtils.cpp') diff --git a/llvm/lib/Transforms/Utils/LoopRotationUtils.cpp b/llvm/lib/Transforms/Utils/LoopRotationUtils.cpp index 8804bba..da1c09e 100644 --- a/llvm/lib/Transforms/Utils/LoopRotationUtils.cpp +++ b/llvm/lib/Transforms/Utils/LoopRotationUtils.cpp @@ -740,12 +740,7 @@ bool llvm::LoopRotation(Loop *L, LoopInfo *LI, const TargetTransformInfo *TTI, const SimplifyQuery &SQ, bool RotationOnly = true, unsigned Threshold = unsigned(-1), bool IsUtilMode = true) { - if (MSSAU && VerifyMemorySSA) - MSSAU->getMemorySSA()->verifyMemorySSA(); LoopRotate LR(Threshold, LI, TTI, AC, DT, SE, MSSAU, SQ, RotationOnly, IsUtilMode); - if (MSSAU && VerifyMemorySSA) - MSSAU->getMemorySSA()->verifyMemorySSA(); - return LR.processLoop(L); } -- cgit v1.1