[PATCH] D29119: [ImplicitNullCheck] NFC isSuitableMemoryOp cleanup

[Serguei Katkov via Phabricator via llvm-commits]

skatkov added a comment.

Will appload a new version soon. I'm ok with all comments.



================
Comment at: lib/CodeGen/ImplicitNullChecks.cpp:493
+      return false;
+    if ((SRResult == SR_OK) &&
         canHoistLoadInst(&MI, PointerReg, InstsSeenSoFar, NullSucc,
----------------
sanjoy wrote:
> The braces around `SRResult == SR_OK` is unnecessary.
To me it is more readable.


https://reviews.llvm.org/D29119