diff options
author | Eric Christopher <echristo@apple.com> | 2010-03-27 01:54:00 +0000 |
---|---|---|
committer | Eric Christopher <echristo@apple.com> | 2010-03-27 01:54:00 +0000 |
commit | 6fde0bd39b1b431b7be30aaf9221ce6c60b0a5f0 (patch) | |
tree | a7b5e12e2821e3cb348dc6a6740c4a760c38619d /lib/Transforms/IPO/ArgumentPromotion.cpp | |
parent | e6b293b78d8eb3eb749771d2201ace3ecd076010 (diff) | |
download | llvm-6fde0bd39b1b431b7be30aaf9221ce6c60b0a5f0.tar.gz llvm-6fde0bd39b1b431b7be30aaf9221ce6c60b0a5f0.tar.bz2 llvm-6fde0bd39b1b431b7be30aaf9221ce6c60b0a5f0.tar.xz |
When we promote a load of an argument make sure to take the alignment
of the previous load - it's usually important. For example, we don't want
to blindly turn an unaligned load into an aligned one.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@99699 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib/Transforms/IPO/ArgumentPromotion.cpp')
-rw-r--r-- | lib/Transforms/IPO/ArgumentPromotion.cpp | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/lib/Transforms/IPO/ArgumentPromotion.cpp b/lib/Transforms/IPO/ArgumentPromotion.cpp index 00b147e885..40a87e880d 100644 --- a/lib/Transforms/IPO/ArgumentPromotion.cpp +++ b/lib/Transforms/IPO/ArgumentPromotion.cpp @@ -687,7 +687,11 @@ CallGraphNode *ArgPromotion::DoPromotion(Function *F, Ops.clear(); AA.copyValue(OrigLoad->getOperand(0), V); } - Args.push_back(new LoadInst(V, V->getName()+".val", Call)); + // Since we're replacing a load make sure we take the alignment + // of the previous load. + LoadInst *newLoad = new LoadInst(V, V->getName()+".val", Call); + newLoad->setAlignment(OrigLoad->getAlignment()); + Args.push_back(newLoad); AA.copyValue(OrigLoad, Args.back()); } } |