Skip to content

Commit b76eb1d

Browse files
authored
[clang][CodeGen] Remove SimplifyCFGPass preceding RemoveTrapsPass (#84852)
There is no performance difference after switching to `llvm.experimental.hot`.
1 parent 90453f4 commit b76eb1d

File tree

1 file changed

+0
-6
lines changed

1 file changed

+0
-6
lines changed

clang/lib/CodeGen/BackendUtil.cpp

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -84,7 +84,6 @@
8484
#include "llvm/Transforms/Scalar/EarlyCSE.h"
8585
#include "llvm/Transforms/Scalar/GVN.h"
8686
#include "llvm/Transforms/Scalar/JumpThreading.h"
87-
#include "llvm/Transforms/Scalar/SimplifyCFG.h"
8887
#include "llvm/Transforms/Utils/Debugify.h"
8988
#include "llvm/Transforms/Utils/EntryExitInstrumenter.h"
9089
#include "llvm/Transforms/Utils/ModuleUtils.h"
@@ -753,11 +752,6 @@ static void addSanitizers(const Triple &TargetTriple,
753752
// from `buildInlinerPipeline`, which called after profile matching.
754753
PB.registerScalarOptimizerLateEPCallback(
755754
[](FunctionPassManager &FPM, OptimizationLevel Level) {
756-
// RemoveTrapsPass expects trap blocks preceded by conditional
757-
// branches, which usually is not the case without SimplifyCFG.
758-
// TODO: Remove `SimplifyCFGPass` after switching to dedicated
759-
// intrinsic.
760-
FPM.addPass(SimplifyCFGPass());
761755
FPM.addPass(RemoveTrapsPass());
762756
});
763757
}

0 commit comments

Comments
 (0)