summaryrefslogtreecommitdiff
path: root/lib
diff options
context:
space:
mode:
authorJoerg Sonnenberger <joerg@bec.de>2011-12-12 20:18:31 +0000
committerJoerg Sonnenberger <joerg@bec.de>2011-12-12 20:18:31 +0000
commit127a669d09e21ddcd525f493c19dc399093bef35 (patch)
tree07a16d55d282531b1a1954a39877b486070d58d8 /lib
parent4ab406d7fc06b1272d02cd8be46f0c5ebe51a3da (diff)
downloadllvm-127a669d09e21ddcd525f493c19dc399093bef35.tar.gz
llvm-127a669d09e21ddcd525f493c19dc399093bef35.tar.bz2
llvm-127a669d09e21ddcd525f493c19dc399093bef35.tar.xz
Only replace fwrite with fputc, if the return value is unused.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@146411 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib')
-rw-r--r--lib/Transforms/Scalar/SimplifyLibCalls.cpp3
1 files changed, 2 insertions, 1 deletions
diff --git a/lib/Transforms/Scalar/SimplifyLibCalls.cpp b/lib/Transforms/Scalar/SimplifyLibCalls.cpp
index 075adc0187..f3184ecc86 100644
--- a/lib/Transforms/Scalar/SimplifyLibCalls.cpp
+++ b/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);