[llvm-commits] CVS: llvm/lib/Transforms/IPO/SimplifyLibCalls.cpp

[unknown_user at cs.uiuc.edu]

Changes in directory llvm/lib/Transforms/IPO:

SimplifyLibCalls.cpp updated: 1.39 -> 1.40
---
Log message:

Make sure ... arguments are casted to sbyte* where needed.


---
Diffs of the changes:  (+12 -13)

 SimplifyLibCalls.cpp |   25 ++++++++++++-------------
 1 files changed, 12 insertions(+), 13 deletions(-)


Index: llvm/lib/Transforms/IPO/SimplifyLibCalls.cpp
diff -u llvm/lib/Transforms/IPO/SimplifyLibCalls.cpp:1.39 llvm/lib/Transforms/IPO/SimplifyLibCalls.cpp:1.40
--- llvm/lib/Transforms/IPO/SimplifyLibCalls.cpp:1.39	Fri May 20 19:23:23 2005
+++ llvm/lib/Transforms/IPO/SimplifyLibCalls.cpp	Fri May 20 19:39:30 2005
@@ -1237,6 +1237,16 @@
   }
 } PowOptimizer;
 
+/// CastToCStr - Return V if it is an sbyte*, otherwise cast it to sbyte*,
+/// inserting the cast before IP, and return the cast.
+/// @brief Cast a value to a "C" string.
+static Value *CastToCStr(Value *V, Instruction &IP) {
+  const Type *SBPTy = PointerType::get(Type::SByteTy);
+  if (V->getType() != SBPTy)
+    return new CastInst(V, SBPTy, V->getName(), &IP);
+  return V;
+}
+
 /// This LibCallOptimization will simplify calls to the "fprintf" library 
 /// function. It looks for cases where the result of fprintf is not used and the
 /// operation can be reduced to something simpler.
@@ -1292,7 +1302,7 @@
           return false;
       }
 
-      // fprintf(file,fmt) -> fwrite(fmt,strlen(fmt),1file) 
+      // fprintf(file,fmt) -> fwrite(fmt,strlen(fmt),file) 
       const Type* FILEptr_type = ci->getOperand(1)->getType();
       Function* fwrite_func = SLC.get_fwrite(FILEptr_type);
       if (!fwrite_func)
@@ -1335,7 +1345,7 @@
         if (!fwrite_func)
           return false;
         std::vector<Value*> args;
-        args.push_back(ci->getOperand(3));
+        args.push_back(CastToCStr(ci->getOperand(3), *ci));
         args.push_back(ConstantUInt::get(SLC.getIntPtrType(),len));
         args.push_back(ConstantUInt::get(SLC.getIntPtrType(),1));
         args.push_back(ci->getOperand(1));
@@ -1366,17 +1376,6 @@
   }
 } FPrintFOptimizer;
 
-/// CastToCStr - Return V if it is an sbyte*, otherwise cast it to sbyte*,
-/// inserting the cast before IP, and return the cast.
-/// @brief Cast a value to a "C" string.
-static Value *CastToCStr(Value *V, Instruction &IP) {
-  const Type *SBPTy = PointerType::get(Type::SByteTy);
-  if (V->getType() != SBPTy)
-    return new CastInst(V, SBPTy, V->getName(), &IP);
-  return V;
-}
-
-
 /// This LibCallOptimization will simplify calls to the "sprintf" library 
 /// function. It looks for cases where the result of sprintf is not used and the
 /// operation can be reduced to something simpler.