[llvm-commits] [llvm] r146690 - /llvm/trunk/lib/Target/ARM/ARMConstantIslandPass.cpp

[Chris Lattner]

woot, can the cl::opt be removed now? :)

Is this worth a comment in the release notes?

-Chris

On Dec 15, 2011, at 2:14 PM, Jakob Stoklund Olesen wrote:

> Author: stoklund
> Date: Thu Dec 15 16:14:45 2011
> New Revision: 146690
> 
> URL: http://llvm.org/viewvc/llvm-project?rev=146690&view=rev
> Log:
> Enable proper constant island alignment by default.
> 
> The code size increase is tiny (< 0.05%) because so little code uses
> 16-byte constant pool entries.
> 
> Modified:
>    llvm/trunk/lib/Target/ARM/ARMConstantIslandPass.cpp
> 
> Modified: llvm/trunk/lib/Target/ARM/ARMConstantIslandPass.cpp
> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/ARM/ARMConstantIslandPass.cpp?rev=146690&r1=146689&r2=146690&view=diff
> ==============================================================================
> --- llvm/trunk/lib/Target/ARM/ARMConstantIslandPass.cpp (original)
> +++ llvm/trunk/lib/Target/ARM/ARMConstantIslandPass.cpp Thu Dec 15 16:14:45 2011
> @@ -53,7 +53,7 @@
>           cl::desc("Adjust basic block layout to better use TB[BH]"));
> 
> static cl::opt<bool>
> -AlignConstantIslands("arm-align-constant-islands", cl::Hidden,
> +AlignConstantIslands("arm-align-constant-islands", cl::Hidden, cl::init(true),
>           cl::desc("Align constant islands in code"));
> 
> /// UnknownPadding - Return the worst case padding that could result from
> 
> 
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits