mirror of
https://github.com/c64scene-ar/llvm-6502.git
synced 2025-01-09 10:31:14 +00:00
Stop trying to have instcombine preserve LCSSA form: this was not
effective in avoiding recomputation of LCSSA form; the widespread use of instsimplify (which looks through phi nodes) means it was not preserving LCSSA form anyway; and instcombine is no longer scheduled in the middle of the loop passes so this doesn't matter anymore. git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@130301 91177308-0d34-0410-b5e6-96231b3b80d8
This commit is contained in:
parent
104f4adeac
commit
089a246333
@ -70,7 +70,6 @@ class LLVM_LIBRARY_VISIBILITY InstCombiner
|
||||
: public FunctionPass,
|
||||
public InstVisitor<InstCombiner, Instruction*> {
|
||||
TargetData *TD;
|
||||
bool MustPreserveLCSSA;
|
||||
bool MadeIRChange;
|
||||
public:
|
||||
/// Worklist - All of the instructions that need to be simplified.
|
||||
|
@ -774,9 +774,6 @@ Instruction *InstCombiner::SliceUpIllegalIntegerPHI(PHINode &FirstPhi) {
|
||||
// PHINode simplification
|
||||
//
|
||||
Instruction *InstCombiner::visitPHINode(PHINode &PN) {
|
||||
// If LCSSA is around, don't mess with Phi nodes
|
||||
if (MustPreserveLCSSA) return 0;
|
||||
|
||||
if (Value *V = SimplifyInstruction(&PN, TD))
|
||||
return ReplaceInstUsesWith(PN, V);
|
||||
|
||||
|
@ -76,7 +76,6 @@ INITIALIZE_PASS(InstCombiner, "instcombine",
|
||||
"Combine redundant instructions", false, false)
|
||||
|
||||
void InstCombiner::getAnalysisUsage(AnalysisUsage &AU) const {
|
||||
AU.addPreservedID(LCSSAID);
|
||||
AU.setPreservesCFG();
|
||||
}
|
||||
|
||||
@ -1637,7 +1636,6 @@ bool InstCombiner::DoOneIteration(Function &F, unsigned Iteration) {
|
||||
|
||||
|
||||
bool InstCombiner::runOnFunction(Function &F) {
|
||||
MustPreserveLCSSA = mustPreserveAnalysisID(LCSSAID);
|
||||
TD = getAnalysisIfAvailable<TargetData>();
|
||||
|
||||
|
||||
|
Loading…
x
Reference in New Issue
Block a user