From 23cda0cd39ba17f02fbe1ff6d54c73c29797e707 Mon Sep 17 00:00:00 2001 From: Michael Gottesman Date: Tue, 29 Jan 2013 21:07:53 +0000 Subject: [PATCH] Fixed a few debug messages and some 80+ violations. llvm-svn: 173832 --- llvm/lib/Transforms/ObjCARC/ObjCARCOpts.cpp | 19 ++++++++++--------- 1 file changed, 10 insertions(+), 9 deletions(-) diff --git a/llvm/lib/Transforms/ObjCARC/ObjCARCOpts.cpp b/llvm/lib/Transforms/ObjCARC/ObjCARCOpts.cpp index 068da13..c082701 100644 --- a/llvm/lib/Transforms/ObjCARC/ObjCARCOpts.cpp +++ b/llvm/lib/Transforms/ObjCARC/ObjCARCOpts.cpp @@ -177,14 +177,14 @@ static const Value *FindSingleUseIdentifiedObject(const Value *Arg) { /// argument to a call is not considered an escape. /// static bool DoesRetainableObjPtrEscape(const User *Ptr) { - DEBUG(dbgs() << "DoesRetainableObjPtrEscape: Target: " << *Ptr << "\n"); // Walk the def-use chains. SmallVector Worklist; Worklist.push_back(Ptr); // If Ptr has any operands add them as well. - for (User::const_op_iterator I = Ptr->op_begin(), E = Ptr->op_end(); I != E; ++I) { + for (User::const_op_iterator I = Ptr->op_begin(), E = Ptr->op_end(); I != E; + ++I) { Worklist.push_back(*I); } @@ -210,8 +210,8 @@ static bool DoesRetainableObjPtrEscape(const User *Ptr) { case IC_StoreStrong: case IC_Autorelease: case IC_AutoreleaseRV: { - DEBUG(dbgs() << "DoesRetainableObjPtrEscape: User copies pointer arguments. " - "Block Escapes!\n"); + DEBUG(dbgs() << "DoesRetainableObjPtrEscape: User copies pointer " + "arguments. Pointer Escapes!\n"); // These special functions make copies of their pointer arguments. return true; } @@ -223,11 +223,12 @@ static bool DoesRetainableObjPtrEscape(const User *Ptr) { isa(UUser) || isa(UUser)) { if (!VisitedSet.insert(UUser)) { - DEBUG(dbgs() << "DoesRetainableObjPtrEscape: User copies value. Escapes " - "if result escapes. Adding to list.\n"); + DEBUG(dbgs() << "DoesRetainableObjPtrEscape: User copies value. " + "Ptr escapes if result escapes. Adding to list.\n"); Worklist.push_back(UUser); } else { - DEBUG(dbgs() << "DoesRetainableObjPtrEscape: Already visited node.\n"); + DEBUG(dbgs() << "DoesRetainableObjPtrEscape: Already visited node." + "\n"); } continue; } @@ -244,13 +245,13 @@ static bool DoesRetainableObjPtrEscape(const User *Ptr) { continue; } // Otherwise, conservatively assume an escape. - DEBUG(dbgs() << "DoesRetainableObjPtrEscape: Assuming block escapes.\n"); + DEBUG(dbgs() << "DoesRetainableObjPtrEscape: Assuming ptr escapes.\n"); return true; } } while (!Worklist.empty()); // No escapes found. - DEBUG(dbgs() << "DoesRetainableObjPtrEscape: Block does not escape.\n"); + DEBUG(dbgs() << "DoesRetainableObjPtrEscape: Ptr does not escape.\n"); return false; } -- 2.7.4