Delete a comment that makes no sense to me. The statement that moving a CPE
authorBob Wilson <bob.wilson@apple.com>
Mon, 12 Oct 2009 22:49:05 +0000 (22:49 +0000)
committerBob Wilson <bob.wilson@apple.com>
Mon, 12 Oct 2009 22:49:05 +0000 (22:49 +0000)
before its reference is only supported on ARM has not been true for a while.
In fact, until recently, that was only supported for Thumb.  Besides that,
CPEs are always a multiple of 4 bytes in size, so inserting a CPE should have
no effect on Thumb alignment.

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@83916 91177308-0d34-0410-b5e6-96231b3b80d8

lib/Target/ARM/ARMConstantIslandPass.cpp

index 3016565ccaac8e5d79eb02fcc5e9c4d900632037..d9d7763eda2766de467ba45003ab69b195a8f765 100644 (file)
@@ -757,8 +757,7 @@ bool ARMConstantIslands::WaterIsInRange(unsigned UserOffset,
                        BBSizes[Water->getNumber()];
 
   // If the CPE is to be inserted before the instruction, that will raise
-  // the offset of the instruction.  (Currently applies only to ARM, so
-  // no alignment compensation attempted here.)
+  // the offset of the instruction.
   if (CPEOffset < UserOffset)
     UserOffset += U.CPEMI->getOperand(2).getImm();