[llvm] r256502 - Refactor: Simplify boolean conditional return statements in lib/Transforms/ObjCARC

Alexander Kornienko via llvm-commits llvm-commits at lists.llvm.org
Mon Dec 28 08:19:08 PST 2015


Author: alexfh
Date: Mon Dec 28 10:19:08 2015
New Revision: 256502

URL: http://llvm.org/viewvc/llvm-project?rev=256502&view=rev
Log:
Refactor: Simplify boolean conditional return statements in lib/Transforms/ObjCARC

Summary: Use clang-tidy to simplify boolean conditional return statements

Reviewers: craig.topper, bkramer, chandlerc, gottesmm

Subscribers: llvm-commits

Patch by Richard Thomson!

Differential Revision: http://reviews.llvm.org/D9999

Modified:
    llvm/trunk/lib/Transforms/ObjCARC/ObjCARCOpts.cpp

Modified: llvm/trunk/lib/Transforms/ObjCARC/ObjCARCOpts.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Transforms/ObjCARC/ObjCARCOpts.cpp?rev=256502&r1=256501&r2=256502&view=diff
==============================================================================
--- llvm/trunk/lib/Transforms/ObjCARC/ObjCARCOpts.cpp (original)
+++ llvm/trunk/lib/Transforms/ObjCARC/ObjCARCOpts.cpp Mon Dec 28 10:19:08 2015
@@ -2010,10 +2010,7 @@ HasSafePathToPredecessorCall(const Value
 
   // Check that the call is a regular call.
   ARCInstKind Class = GetBasicARCInstKind(Call);
-  if (Class != ARCInstKind::CallOrUser && Class != ARCInstKind::Call)
-    return false;
-
-  return true;
+  return Class == ARCInstKind::CallOrUser || Class == ARCInstKind::Call;
 }
 
 /// Find a dependent retain that precedes the given autorelease for which there




More information about the llvm-commits mailing list