[PATCH] D13088: [InlineCost] Adjust inlining cost for implicit null checks

Chen Li via llvm-commits llvm-commits at lists.llvm.org
Wed Sep 23 12:22:36 PDT 2015


chenli added inline comments.

================
Comment at: lib/Analysis/InlineCost.cpp:606
@@ +605,3 @@
+  // Implicit null checks act as unconditional branches, and
+  // they should be free. See the comments in visitBranchInst.
+  if (I.getMetadata(LLVMContext::MD_make_implicit))
----------------
reames wrote:
> It looks like this might be a partial diff?  I don't see any changes in visitBranchInst?
Apparently I uploaded the wrong diff. Will update soon.


http://reviews.llvm.org/D13088





More information about the llvm-commits mailing list