diff options
| author | Dan Gohman <gohman@apple.com> | 2010-04-02 03:04:37 +0000 |
|---|---|---|
| committer | Dan Gohman <gohman@apple.com> | 2010-04-02 03:04:37 +0000 |
| commit | 3ed470da2b05e027282e2f8c67b3739dad11745c (patch) | |
| tree | c46f1b1e7a1c636246d07f7377a16114624145ec /lib/Transforms | |
| parent | 583bd1720b42702d7c02c197916919107f683d27 (diff) | |
| download | external_llvm-3ed470da2b05e027282e2f8c67b3739dad11745c.zip external_llvm-3ed470da2b05e027282e2f8c67b3739dad11745c.tar.gz external_llvm-3ed470da2b05e027282e2f8c67b3739dad11745c.tar.bz2 | |
Revert the recent alignment changes. They're broken for -Os because,
in particular, they end up aligning strings at 16-byte boundaries, and
there's no way for GlobalOpt to check OptForSize.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@100172 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib/Transforms')
| -rw-r--r-- | lib/Transforms/IPO/GlobalOpt.cpp | 8 |
1 files changed, 1 insertions, 7 deletions
diff --git a/lib/Transforms/IPO/GlobalOpt.cpp b/lib/Transforms/IPO/GlobalOpt.cpp index c09e78e..ddff5ef 100644 --- a/lib/Transforms/IPO/GlobalOpt.cpp +++ b/lib/Transforms/IPO/GlobalOpt.cpp @@ -1880,7 +1880,6 @@ 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++; @@ -1890,16 +1889,11 @@ 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()) |