[llvm-commits] CVS: llvm/lib/Transforms/Instrumentation/RSProfiling.cpp

Reid Spencer reid at x10sys.com
Thu Jan 11 20:25:30 PST 2007



Changes in directory llvm/lib/Transforms/Instrumentation:

RSProfiling.cpp updated: 1.16 -> 1.17
---
Log message:

Implement review feedback for the ConstantBool->ConstantInt merge. Chris
recommended that getBoolValue be replaced with getZExtValue and that
get(bool) be replaced by get(const Type*, uint64_t). This implements
those changes.


---
Diffs of the changes:  (+2 -2)

 RSProfiling.cpp |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)


Index: llvm/lib/Transforms/Instrumentation/RSProfiling.cpp
diff -u llvm/lib/Transforms/Instrumentation/RSProfiling.cpp:1.16 llvm/lib/Transforms/Instrumentation/RSProfiling.cpp:1.17
--- llvm/lib/Transforms/Instrumentation/RSProfiling.cpp:1.16	Thu Jan 11 06:24:14 2007
+++ llvm/lib/Transforms/Instrumentation/RSProfiling.cpp	Thu Jan 11 22:24:45 2007
@@ -460,7 +460,7 @@
   //b:
   new BranchInst(cast<BasicBlock>(Translate(dst)), bbC);
   new BranchInst(dst, cast<BasicBlock>(Translate(dst)), 
-		 ConstantInt::get(true), bbCp);
+		 ConstantInt::get(Type::Int1Ty, true), bbCp);
   //c:
   {
     TerminatorInst* iB = src->getTerminator();
@@ -516,7 +516,7 @@
     TerminatorInst* T = F.getEntryBlock().getTerminator();
     ReplaceInstWithInst(T, new BranchInst(T->getSuccessor(0),
 			       cast<BasicBlock>(Translate(T->getSuccessor(0))),
-					  ConstantInt::get(true)));
+					 ConstantInt::get(Type::Int1Ty, true)));
     
     //do whatever is needed now that the function is duplicated
     c->PrepFunction(&F);






More information about the llvm-commits mailing list