From 9f87fca49cec50c76c1d28b58e43414e00007b11 Mon Sep 17 00:00:00 2001
From: Evan Cheng <evan.cheng@apple.com>
Date: Thu, 1 Feb 2007 01:09:47 +0000
Subject: [PATCH] Pessmistically assume the .align 2 before the first constpool
 entry adds two bytes padding.

llvm-svn: 33734
---
 lib/Target/ARM/ARMConstantIslandPass.cpp | 14 ++++++++++++--
 1 file changed, 12 insertions(+), 2 deletions(-)

diff --git a/lib/Target/ARM/ARMConstantIslandPass.cpp b/lib/Target/ARM/ARMConstantIslandPass.cpp
index 93115288b92..e54f7f97725 100644
--- a/lib/Target/ARM/ARMConstantIslandPass.cpp
+++ b/lib/Target/ARM/ARMConstantIslandPass.cpp
@@ -339,6 +339,13 @@ void ARMConstantIslands::InitialFunctionScan(MachineFunction &Fn,
           break;
         }
     }
+
+    // In thumb mode, if this block is a constpool island, pessmisticly assume
+    // it needs to be padded by two byte so it's aligned on 4 byte boundary.
+    if (AFI->isThumbFunction() &&
+        MBB.begin()->getOpcode() == ARM::CONSTPOOL_ENTRY)
+      MBBSize += 2;
+
     BBSizes.push_back(MBBSize);
   }
 }
@@ -465,8 +472,11 @@ bool ARMConstantIslands::CPEIsInRange(MachineInstr *MI, MachineInstr *CPEMI,
                                       unsigned MaxDisp) {
   unsigned PCAdj      = AFI->isThumbFunction() ? 4 : 8;
   unsigned UserOffset = GetOffsetOf(MI) + PCAdj;
-  unsigned CPEOffset  = GetOffsetOf(CPEMI);
-  
+  // In thumb mode, pessmisticly assumes the .align 2 before the first CPE
+  // in the island adds two byte padding.
+  unsigned AlignAdj   = AFI->isThumbFunction() ? 2 : 0;
+  unsigned CPEOffset  = GetOffsetOf(CPEMI) + AlignAdj;
+
   DEBUG(std::cerr << "User of CPE#" << CPEMI->getOperand(0).getImm()
                   << " max delta=" << MaxDisp
                   << " at offset " << int(UserOffset-CPEOffset) << "\t"