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

Philip Reames via llvm-commits llvm-commits at lists.llvm.org
Wed Sep 23 12:19:02 PDT 2015


reames 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))
----------------
It looks like this might be a partial diff?  I don't see any changes in visitBranchInst?


http://reviews.llvm.org/D13088





More information about the llvm-commits mailing list