diff options
author | Bob Wilson <bob.wilson@apple.com> | 2009-05-18 20:55:32 +0000 |
---|---|---|
committer | Bob Wilson <bob.wilson@apple.com> | 2009-05-18 20:55:32 +0000 |
commit | e6abdffe06d293b68c498251b5dc9f5dba78dece (patch) | |
tree | 7ee21e789b91333f2b9b00314ccfa763554e081e | |
parent | e2b201bac382464496758d789cddefa50690fbe3 (diff) |
Fix pr4202: Disable CodePlacementOpt for ARM. The ARMConstantIslandPass has
to run last because it needs to know the exact size and position of every
basic block. Currently CodePlacementOpt is set up to run last. It might be
worthwhile to investigate reordering these passes, but for now, let's just
make it work.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@72037 91177308-0d34-0410-b5e6-96231b3b80d8
-rw-r--r-- | lib/Target/ARM/ARMISelLowering.cpp | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/lib/Target/ARM/ARMISelLowering.cpp b/lib/Target/ARM/ARMISelLowering.cpp index d3cf6675f2..e06846559c 100644 --- a/lib/Target/ARM/ARMISelLowering.cpp +++ b/lib/Target/ARM/ARMISelLowering.cpp @@ -288,7 +288,10 @@ ARMTargetLowering::ARMTargetLowering(TargetMachine &TM) setIfCvtDupBlockSizeLimit(Subtarget->isThumb() ? 0 : 2); maxStoresPerMemcpy = 1; //// temporary - rewrite interface to use type - benefitFromCodePlacementOpt = true; + // Do not enable CodePlacementOpt for now: it currently runs after the + // ARMConstantIslandPass and messes up branch relaxation and placement + // of constant islands. + // benefitFromCodePlacementOpt = true; } const char *ARMTargetLowering::getTargetNodeName(unsigned Opcode) const { |