diff options
| author | Dan Gohman <gohman@apple.com> | 2010-04-02 00:14:16 +0000 |
|---|---|---|
| committer | Dan Gohman <gohman@apple.com> | 2010-04-02 00:14:16 +0000 |
| commit | c2873722e6025a93391e7d8460ec2be480b899a4 (patch) | |
| tree | a1c763659947749b089714167a10339a7b56862d | |
| parent | ad54cc547d78395ea0171c1a334b298a070595fd (diff) | |
| download | external_llvm-c2873722e6025a93391e7d8460ec2be480b899a4.zip external_llvm-c2873722e6025a93391e7d8460ec2be480b899a4.tar.gz external_llvm-c2873722e6025a93391e7d8460ec2be480b899a4.tar.bz2 | |
Make globalopt refine global variable alignment.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@100160 91177308-0d34-0410-b5e6-96231b3b80d8
| -rw-r--r-- | lib/Transforms/IPO/GlobalOpt.cpp | 8 | ||||
| -rw-r--r-- | test/Transforms/GlobalOpt/gv-align.ll | 6 |
2 files changed, 13 insertions, 1 deletions
diff --git a/lib/Transforms/IPO/GlobalOpt.cpp b/lib/Transforms/IPO/GlobalOpt.cpp index ddff5ef..c09e78e 100644 --- a/lib/Transforms/IPO/GlobalOpt.cpp +++ b/lib/Transforms/IPO/GlobalOpt.cpp @@ -1880,6 +1880,7 @@ bool GlobalOpt::OptimizeFunctions(Module &M) { bool GlobalOpt::OptimizeGlobalVars(Module &M) { bool Changed = false; + TargetData *TD = getAnalysisIfAvailable<TargetData>(); for (Module::global_iterator GVI = M.global_begin(), E = M.global_end(); GVI != E; ) { GlobalVariable *GV = GVI++; @@ -1889,11 +1890,16 @@ bool GlobalOpt::OptimizeGlobalVars(Module &M) { // Simplify the initializer. if (GV->hasInitializer()) if (ConstantExpr *CE = dyn_cast<ConstantExpr>(GV->getInitializer())) { - TargetData *TD = getAnalysisIfAvailable<TargetData>(); Constant *New = ConstantFoldConstantExpression(CE, TD); if (New && New != CE) GV->setInitializer(New); } + // Refine the alignment value. + if (TD && GV->hasDefinitiveInitializer()) { + unsigned Align = TD->getPreferredAlignment(GV); + if (Align > GV->getAlignment()) + GV->setAlignment(Align); + } // Do more involved optimizations if the global is internal. if (!GV->isConstant() && GV->hasLocalLinkage() && GV->hasInitializer()) diff --git a/test/Transforms/GlobalOpt/gv-align.ll b/test/Transforms/GlobalOpt/gv-align.ll new file mode 100644 index 0000000..f99e8a6 --- /dev/null +++ b/test/Transforms/GlobalOpt/gv-align.ll @@ -0,0 +1,6 @@ +; RUN: opt -globalopt -S < %s | grep {, align 16} + +; Globalopt should refine the alignment for global variables. + +target datalayout = "e-p:64:64:64" +@a = global [4 x i32] [i32 2, i32 3, i32 4, i32 5 ] |