diff options
| author | Joerg Sonnenberger <joerg@bec.de> | 2011-12-12 20:18:31 +0000 |
|---|---|---|
| committer | Joerg Sonnenberger <joerg@bec.de> | 2011-12-12 20:18:31 +0000 |
| commit | 45c41641664d8ff44c03f6205312d347081882b0 (patch) | |
| tree | 53776dd203edb525bde6e940880266add8277b78 /llvm/lib/Transforms | |
| parent | 27a7489a0323873dc0afe23d570f64517b261614 (diff) | |
| download | bcm5719-llvm-45c41641664d8ff44c03f6205312d347081882b0.tar.gz bcm5719-llvm-45c41641664d8ff44c03f6205312d347081882b0.zip | |
Only replace fwrite with fputc, if the return value is unused.
llvm-svn: 146411
Diffstat (limited to 'llvm/lib/Transforms')
| -rw-r--r-- | llvm/lib/Transforms/Scalar/SimplifyLibCalls.cpp | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/llvm/lib/Transforms/Scalar/SimplifyLibCalls.cpp b/llvm/lib/Transforms/Scalar/SimplifyLibCalls.cpp index 075adc01873..f3184ecc860 100644 --- a/llvm/lib/Transforms/Scalar/SimplifyLibCalls.cpp +++ b/llvm/lib/Transforms/Scalar/SimplifyLibCalls.cpp @@ -1293,7 +1293,8 @@ struct FWriteOpt : public LibCallOptimization { return ConstantInt::get(CI->getType(), 0); // If this is writing one byte, turn it into fputc. - if (Bytes == 1) { // fwrite(S,1,1,F) -> fputc(S[0],F) + // This optimisation is only valid, if the return value is unused. + if (Bytes == 1 && CI->use_empty()) { // fwrite(S,1,1,F) -> fputc(S[0],F) Value *Char = B.CreateLoad(CastToCStr(CI->getArgOperand(0), B), "char"); EmitFPutC(Char, CI->getArgOperand(3), B, TD); return ConstantInt::get(CI->getType(), 1); |
