[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