[PATCH] D9999: Refactor: Simplify boolean conditional return statements in lib/Transforms/ObjCARC

Richard via llvm-commits llvm-commits at lists.llvm.org
Sat Oct 24 10:02:06 PDT 2015


LegalizeAdulthood updated this revision to Diff 38307.
LegalizeAdulthood added a comment.

Update to latest


http://reviews.llvm.org/D9999

Files:
  lib/Transforms/ObjCARC/ObjCARCOpts.cpp

Index: lib/Transforms/ObjCARC/ObjCARCOpts.cpp
===================================================================
--- lib/Transforms/ObjCARC/ObjCARCOpts.cpp
+++ lib/Transforms/ObjCARC/ObjCARCOpts.cpp
@@ -2010,10 +2010,7 @@
 
   // 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


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D9999.38307.patch
Type: text/x-patch
Size: 588 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20151024/db169c29/attachment.bin>


More information about the llvm-commits mailing list