diff options
author | James Molloy <james.molloy@arm.com> | 2014-06-12 15:18:33 +0000 |
---|---|---|
committer | James Molloy <james.molloy@arm.com> | 2014-06-12 15:18:33 +0000 |
commit | 5eba90a861ff19cc2c49c7074565541978cfc83f (patch) | |
tree | 3e7ee0647a1fcfb486ce4d480c95a966963ac2af /lib | |
parent | 1f4c755c2cb31f5b812734428a28bb11fd27639a (diff) | |
download | llvm-5eba90a861ff19cc2c49c7074565541978cfc83f.tar.gz llvm-5eba90a861ff19cc2c49c7074565541978cfc83f.tar.bz2 llvm-5eba90a861ff19cc2c49c7074565541978cfc83f.tar.xz |
Disable the load/store optimization pass for Thumb-1.
Moritz's changes have improved codegen a lot, but further testing showed significant correctness problems. Disable by default until these have been worked out.
Patch by Moritz Roth!
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@210789 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib')
-rw-r--r-- | lib/Target/ARM/ARMTargetMachine.cpp | 10 |
1 files changed, 7 insertions, 3 deletions
diff --git a/lib/Target/ARM/ARMTargetMachine.cpp b/lib/Target/ARM/ARMTargetMachine.cpp index 6ef2ea4e10..c1e866be6e 100644 --- a/lib/Target/ARM/ARMTargetMachine.cpp +++ b/lib/Target/ARM/ARMTargetMachine.cpp @@ -265,7 +265,8 @@ bool ARMPassConfig::addInstSelector() { } bool ARMPassConfig::addPreRegAlloc() { - if (getOptLevel() != CodeGenOpt::None) + // FIXME: Temporarily disabling Thumb-1 pre-RA Load/Store optimization pass + if (getOptLevel() != CodeGenOpt::None && !getARMSubtarget().isThumb1Only()) addPass(createARMLoadStoreOptimizationPass(true)); if (getOptLevel() != CodeGenOpt::None && getARMSubtarget().isCortexA9()) addPass(createMLxExpansionPass()); @@ -280,8 +281,11 @@ bool ARMPassConfig::addPreRegAlloc() { bool ARMPassConfig::addPreSched2() { if (getOptLevel() != CodeGenOpt::None) { - addPass(createARMLoadStoreOptimizationPass()); - printAndVerify("After ARM load / store optimizer"); + // FIXME: Temporarily disabling Thumb-1 post-RA Load/Store optimization pass + if (!getARMSubtarget().isThumb1Only()) { + addPass(createARMLoadStoreOptimizationPass()); + printAndVerify("After ARM load / store optimizer"); + } if (getARMSubtarget().hasNEON()) addPass(createExecutionDependencyFixPass(&ARM::DPRRegClass)); |