diff options
author | Evan Cheng <evan.cheng@apple.com> | 2008-01-17 04:18:54 +0000 |
---|---|---|
committer | Evan Cheng <evan.cheng@apple.com> | 2008-01-17 04:18:54 +0000 |
commit | b2fc2a330a9106c67d50a2b6dbfffc679af4ce88 (patch) | |
tree | 0263fc377964a65a616176844a9b8db53a2fd728 /lib/Transforms | |
parent | 3de01e10aeb384fbb4ffc92fa046b10db2485261 (diff) | |
download | external_llvm-b2fc2a330a9106c67d50a2b6dbfffc679af4ce88.zip external_llvm-b2fc2a330a9106c67d50a2b6dbfffc679af4ce88.tar.gz external_llvm-b2fc2a330a9106c67d50a2b6dbfffc679af4ce88.tar.bz2 |
DAE bug fix. Don't lose parameter attributes on vararg arguments.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@46113 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib/Transforms')
-rw-r--r-- | lib/Transforms/IPO/DeadArgumentElimination.cpp | 14 |
1 files changed, 9 insertions, 5 deletions
diff --git a/lib/Transforms/IPO/DeadArgumentElimination.cpp b/lib/Transforms/IPO/DeadArgumentElimination.cpp index b877012..3a4c36f 100644 --- a/lib/Transforms/IPO/DeadArgumentElimination.cpp +++ b/lib/Transforms/IPO/DeadArgumentElimination.cpp @@ -590,15 +590,19 @@ void DAE::RemoveDeadArgumentsFromFunction(Function *F) { ParamAttrsVec.push_back(ParamAttrsWithIndex::get(Args.size(), Attrs)); } - // Reconstruct the ParamAttrsList based on the vector we constructed. - PAL = ParamAttrsList::get(ParamAttrsVec); - if (ExtraArgHack) Args.push_back(UndefValue::get(Type::Int32Ty)); - // Push any varargs arguments on the list - for (; AI != CS.arg_end(); ++AI) + // Push any varargs arguments on the list. Don't forget their attributes. + for (; AI != CS.arg_end(); ++AI) { Args.push_back(*AI); + uint16_t Attrs = PAL ? PAL->getParamAttrs(index++) : 0; + if (Attrs) + ParamAttrsVec.push_back(ParamAttrsWithIndex::get(Args.size(), Attrs)); + } + + // Reconstruct the ParamAttrsList based on the vector we constructed. + PAL = ParamAttrsList::get(ParamAttrsVec); Instruction *New; if (InvokeInst *II = dyn_cast<InvokeInst>(Call)) { |