<div dir="ltr">I'm dropping the patch for now. The argument for making this as a pass is to allow it to be shared by some client other than inliner (LoopUnroller? something else?). But then, tying it to the cost model used by inliner doesn't make sense. I can't rewrite this patch as a library now before switching to the new pass manager, so I'll drop this now and revisit this later. <div><br></div><div><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Oct 27, 2015 at 3:55 PM, <a href="mailto:hfinkel@anl.gov">hfinkel@anl.gov</a> <span dir="ltr"><<a href="mailto:hfinkel@anl.gov" target="_blank">hfinkel@anl.gov</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">hfinkel added inline comments.<br>
<br>
================<br>
Comment at: include/llvm/Analysis/InlineCost.h:165<br>
@@ +164,3 @@<br>
+  // not.<br>
+  const bool Inlining;<br>
+<br>
----------------<br>
I like the fact that the inlining-specific behavior can be explicitly disabled using this flag, but this needs to be better documented. At least give a few examples here of what kind of inlining-specific behavior this enables.<br>
<div class="HOEnZb"><div class="h5"><br>
<br>
Repository:<br>
  rL LLVM<br>
<br>
<a href="http://reviews.llvm.org/D13755" rel="noreferrer" target="_blank">http://reviews.llvm.org/D13755</a><br>
<br>
<br>
<br>
</div></div></blockquote></div><br></div>