diff options
| author | Nick Lewycky <nicholas@mxc.ca> | 2012-09-25 01:55:59 +0000 |
|---|---|---|
| committer | Nick Lewycky <nicholas@mxc.ca> | 2012-09-25 01:55:59 +0000 |
| commit | 24ab21c691fb98040318b9521cd1247c3e0b79cc (patch) | |
| tree | 562d5355ffcdb7e1eba9d3423c60d5be34c44e90 /lib/Transforms/Scalar | |
| parent | e5510db57c86a618cbf9c7513e03f34e7e40a1fd (diff) | |
| download | external_llvm-24ab21c691fb98040318b9521cd1247c3e0b79cc.zip external_llvm-24ab21c691fb98040318b9521cd1247c3e0b79cc.tar.gz external_llvm-24ab21c691fb98040318b9521cd1247c3e0b79cc.tar.bz2 | |
Don't forget that strcpy and friends return a pointer to the destination, so
it's not a dead store if that pointer is used. Whoops!
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@164583 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib/Transforms/Scalar')
| -rw-r--r-- | lib/Transforms/Scalar/DeadStoreElimination.cpp | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/lib/Transforms/Scalar/DeadStoreElimination.cpp b/lib/Transforms/Scalar/DeadStoreElimination.cpp index f96309e..301ee2f 100644 --- a/lib/Transforms/Scalar/DeadStoreElimination.cpp +++ b/lib/Transforms/Scalar/DeadStoreElimination.cpp @@ -266,8 +266,8 @@ static bool isRemovable(Instruction *I) { } } - if (CallSite(I)) // If we assume hasMemoryWrite(I) is true, - return true; // then there's nothing left to check. + if (CallSite CS = I) + return CS.getInstruction()->use_empty(); return false; } |