diff options
author | Michael Gottesman <mgottesman@apple.com> | 2013-01-29 21:07:53 +0000 |
---|---|---|
committer | Michael Gottesman <mgottesman@apple.com> | 2013-01-29 21:07:53 +0000 |
commit | 23cda0cd39ba17f02fbe1ff6d54c73c29797e707 (patch) | |
tree | 2b1c6c6a27c73f0b9ad3ad76fcef1d7653e7ab8a /llvm/lib/Transforms | |
parent | 53fd20bdbd700835d9786b88f08b99cea88db527 (diff) | |
download | bcm5719-llvm-23cda0cd39ba17f02fbe1ff6d54c73c29797e707.tar.gz bcm5719-llvm-23cda0cd39ba17f02fbe1ff6d54c73c29797e707.zip |
Fixed a few debug messages and some 80+ violations.
llvm-svn: 173832
Diffstat (limited to 'llvm/lib/Transforms')
-rw-r--r-- | llvm/lib/Transforms/ObjCARC/ObjCARCOpts.cpp | 19 |
1 files changed, 10 insertions, 9 deletions
diff --git a/llvm/lib/Transforms/ObjCARC/ObjCARCOpts.cpp b/llvm/lib/Transforms/ObjCARC/ObjCARCOpts.cpp index 068da13fc04..c082701f8a5 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<const Value *, 4> 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<PHINode>(UUser) || isa<SelectInst>(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; } |