diff options
author | Matthijs Kooijman <matthijs@stdin.nl> | 2008-06-05 08:57:20 +0000 |
---|---|---|
committer | Matthijs Kooijman <matthijs@stdin.nl> | 2008-06-05 08:57:20 +0000 |
commit | 47c6fd73172b07211ffaaad7189847ccd2416a73 (patch) | |
tree | b4ac9d823367bc2de5461c37d9b884e156489a4d /lib/Transforms/IPO | |
parent | 257da0a7fc2df19444e57e7c84a278c6c1b9b551 (diff) | |
download | external_llvm-47c6fd73172b07211ffaaad7189847ccd2416a73.zip external_llvm-47c6fd73172b07211ffaaad7189847ccd2416a73.tar.gz external_llvm-47c6fd73172b07211ffaaad7189847ccd2416a73.tar.bz2 |
Let StructRetPromotion check if all if its users are really calls or invokesn,
not other instructions. This fixes a crash with the added testcase.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@51992 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib/Transforms/IPO')
-rw-r--r-- | lib/Transforms/IPO/StructRetPromotion.cpp | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/lib/Transforms/IPO/StructRetPromotion.cpp b/lib/Transforms/IPO/StructRetPromotion.cpp index 97a532b..94bf4c6 100644 --- a/lib/Transforms/IPO/StructRetPromotion.cpp +++ b/lib/Transforms/IPO/StructRetPromotion.cpp @@ -165,6 +165,10 @@ bool SRETPromotion::isSafeToUpdateAllCallers(Function *F) { CallSite CS = CallSite::get(*FnUseI); Instruction *Call = CS.getInstruction(); + // The function is used by something else than a call or invoke instruction, + // we can't change it! + if (!Call) + return false; CallSite::arg_iterator AI = CS.arg_begin(); Value *FirstArg = *AI; |