[llvm-commits] [llvm] r146739 - in /llvm/trunk: docs/ReleaseNotes.html lib/Target/ARM/ARMConstantIslandPass.cpp

[Jakob Stoklund Olesen]

Author: stoklund
Date: Fri Dec 16 10:07:41 2011
New Revision: 146739

URL: http://llvm.org/viewvc/llvm-project?rev=146739&view=rev
Log:
Note ARM constant island alignment in the release notes.

The command line option should be removed, but not until the feature has
gotten a lot of testing. The ARMConstantIslandPass tends to have subtle
bugs that only show up after a while.

Modified:
    llvm/trunk/docs/ReleaseNotes.html
    llvm/trunk/lib/Target/ARM/ARMConstantIslandPass.cpp

Modified: llvm/trunk/docs/ReleaseNotes.html
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/docs/ReleaseNotes.html?rev=146739&r1=146738&r2=146739&view=diff
==============================================================================
--- llvm/trunk/docs/ReleaseNotes.html (original)
+++ llvm/trunk/docs/ReleaseNotes.html Fri Dec 16 10:07:41 2011
@@ -366,8 +366,8 @@
 <p>New features of the ARM target include:</p>
 
 <ul>
-  <li>....</li>
-</ul>
+  <li>The constant island pass now supports basic block and constant pool entry
+  alignments greater than 4 bytes.</li> </ul>
 </div>
 
 

Modified: llvm/trunk/lib/Target/ARM/ARMConstantIslandPass.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/ARM/ARMConstantIslandPass.cpp?rev=146739&r1=146738&r2=146739&view=diff
==============================================================================
--- llvm/trunk/lib/Target/ARM/ARMConstantIslandPass.cpp (original)
+++ llvm/trunk/lib/Target/ARM/ARMConstantIslandPass.cpp Fri Dec 16 10:07:41 2011
@@ -52,6 +52,7 @@
 AdjustJumpTableBlocks("arm-adjust-jump-tables", cl::Hidden, cl::init(true),
           cl::desc("Adjust basic block layout to better use TB[BH]"));
 
+// FIXME: This option should be removed once it has received sufficient testing.
 static cl::opt<bool>
 AlignConstantIslands("arm-align-constant-islands", cl::Hidden, cl::init(true),
           cl::desc("Align constant islands in code"));