diff options
| author | Torok Edwin <edwintorok@gmail.com> | 2009-09-24 09:47:18 +0000 |
|---|---|---|
| committer | Torok Edwin <edwintorok@gmail.com> | 2009-09-24 09:47:18 +0000 |
| commit | f95a450ef9d6d106af171c4f34b2427c32dfc91f (patch) | |
| tree | b024e29c898bcaa3c6310ffc86596cd5016659ab /llvm/lib/Transforms/IPO/IPConstantPropagation.cpp | |
| parent | a5840dbcb55dd6757916446b442a97f12ff154ab (diff) | |
| download | bcm5719-llvm-f95a450ef9d6d106af171c4f34b2427c32dfc91f.tar.gz bcm5719-llvm-f95a450ef9d6d106af171c4f34b2427c32dfc91f.zip | |
Don't constant propagate byval pointers, since they are not really pointers, but
rather structs passed by value.
This fixes PR5038.
llvm-svn: 82689
Diffstat (limited to 'llvm/lib/Transforms/IPO/IPConstantPropagation.cpp')
| -rw-r--r-- | llvm/lib/Transforms/IPO/IPConstantPropagation.cpp | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/llvm/lib/Transforms/IPO/IPConstantPropagation.cpp b/llvm/lib/Transforms/IPO/IPConstantPropagation.cpp index bb2448610f2..95c3f4e3548 100644 --- a/llvm/lib/Transforms/IPO/IPConstantPropagation.cpp +++ b/llvm/lib/Transforms/IPO/IPConstantPropagation.cpp @@ -130,7 +130,8 @@ bool IPCP::PropagateConstantsIntoArguments(Function &F) { Function::arg_iterator AI = F.arg_begin(); for (unsigned i = 0, e = ArgumentConstants.size(); i != e; ++i, ++AI) { // Do we have a constant argument? - if (ArgumentConstants[i].second || AI->use_empty()) + if (ArgumentConstants[i].second || AI->use_empty() || + (AI->hasByValAttr() && isa<PointerType>(AI->getType()))) continue; Value *V = ArgumentConstants[i].first; |
