<div dir="ltr">Sorry for the delay. The stated intention is to keep the legality part in CallAnalyzer and move the benefit part to InlineCostCallAnalyzer.  But if you consider any of the visit methods in CallAnalyzer, they actually compute the benefit - tells whether the instruction can be simplified or removed by inlining the callee. So what is the division of work between CallAnalyzer and InlineCostCallAnalyzer?</div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Jan 6, 2020 at 9:07 AM David Li via Phabricator <<a href="mailto:reviews@reviews.llvm.org">reviews@reviews.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">davidxl added a comment.<br>
<br>
this looks good to me. Easwaran, do you have any comments on the refactoring?<br>
<br>
<br>
Repository:<br>
  rG LLVM Github Monorepo<br>
<br>
CHANGES SINCE LAST ACTION<br>
  <a href="https://reviews.llvm.org/D71733/new/" rel="noreferrer" target="_blank">https://reviews.llvm.org/D71733/new/</a><br>
<br>
<a href="https://reviews.llvm.org/D71733" rel="noreferrer" target="_blank">https://reviews.llvm.org/D71733</a><br>
<br>
<br>
<br>
</blockquote></div>