Skip to content
This repository has been archived by the owner. It is now read-only.
Permalink
Browse files

Revert "Add the NullCheckElimination pass to the default pass list"

This reverts commit e1af108 as the pass
causes misoptimizations in certain situations.
  • Loading branch information...
dotdash committed Dec 1, 2015
1 parent cde1ed3 commit 3564439515985dc1cc0d77057ed00901635a80ad
Showing with 0 additions and 4 deletions.
  1. +0 −4 lib/Transforms/IPO/PassManagerBuilder.cpp
@@ -234,8 +234,6 @@ void PassManagerBuilder::populateModulePassManager(
MPM.add(createEarlyCSEPass()); // Catch trivial redundancies
MPM.add(createJumpThreadingPass()); // Thread jumps.
MPM.add(createCorrelatedValuePropagationPass()); // Propagate conditionals
// Specific to the rust-lang llvm branch:
MPM.add(createNullCheckEliminationPass()); // Eliminate null checks
MPM.add(createCFGSimplificationPass()); // Merge & remove BBs
MPM.add(createInstructionCombiningPass()); // Combine silly seq's
addExtensionsToPM(EP_Peephole, MPM);
@@ -278,8 +276,6 @@ void PassManagerBuilder::populateModulePassManager(
addExtensionsToPM(EP_Peephole, MPM);
MPM.add(createJumpThreadingPass()); // Thread jumps
MPM.add(createCorrelatedValuePropagationPass());
// Specific to the rust-lang llvm branch:
MPM.add(createNullCheckEliminationPass()); // Eliminate null checks
MPM.add(createDeadStoreEliminationPass()); // Delete dead stores
MPM.add(createLICMPass());

0 comments on commit 3564439

Please sign in to comment.
You can’t perform that action at this time.