Skip to content

Commit

Permalink
[NFC] Move some asserts out of Expensive Checks
Browse files Browse the repository at this point in the history
This was done by reviewer's request, but in fact without them we
skip very nasty bugs. Unless it is a REAL problem, I'd keep them
in default setup.
  • Loading branch information
xortator committed Feb 14, 2023
1 parent 94676cf commit 3849dc1
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion llvm/lib/Transforms/Scalar/SimpleLoopUnswitch.cpp
Expand Up @@ -3021,7 +3021,7 @@ injectPendingInvariantConditions(NonTrivialUnswitchCandidate Candidate, Loop &L,
MSSAU->applyUpdates(DTUpdates, DT);
L.addBasicBlockToLoop(CheckBlock, LI);

#ifdef EXPENSIVE_CHECKS
#ifndef NDEBUG
DT.verify();
LI.verify(DT);
if (MSSAU && VerifyMemorySSA)
Expand Down

0 comments on commit 3849dc1

Please sign in to comment.