commit 4d06391c477b922484d4bd979bfb6c2ea15c4c4a
author Dale Johannesen <dalej@apple.com> Wed Oct 24 20:14:50 2007 +0000
committer Dale Johannesen <dalej@apple.com> Wed Oct 24 20:14:50 2007 +0000
tree 68bdc01f66cc7c74b16d16dfd90f2b777420f4c5
parent b2f525dabca0e7ddde443ac429e4be7fb6e30379

Fix off by 1 bug in printf->puts lowering.

llvm-svn: 43309

llvm/lib/Transforms/IPO/SimplifyLibCalls.cpp

diff --git a/llvm/lib/Transforms/IPO/SimplifyLibCalls.cpp b/llvm/lib/Transforms/IPO/SimplifyLibCalls.cpp
index 717a5a8..0904c4c6 100644
--- a/llvm/lib/Transforms/IPO/SimplifyLibCalls.cpp
+++ b/llvm/lib/Transforms/IPO/SimplifyLibCalls.cpp
@@ -1211,8 +1211,10 @@
       new CallInst(SLC.get_puts(), GV, "", CI);
 
       if (CI->use_empty()) return ReplaceCallWith(CI, 0);
+      // The return value from printf includes the \n we just removed, so +1.
       return ReplaceCallWith(CI,
-                             ConstantInt::get(CI->getType(), FormatStr.size()));
+                             ConstantInt::get(CI->getType(), 
+                                              FormatStr.size()+1));
     }