<div dir="ltr">Hi Chandler,<div><br><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><span class=""><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><span><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div>Regarding your patch, I think the simple loop heuristic may be a reasonable incremental improvement, but you really need to wait for Chandler's pass manager change instead of doing all the ordering tricks.</div></div></div></div></blockquote><div><br></div></span><div>Loop analysis is critical for my patch and it is expensive. The ordering tricks (3.b) is only for compile-time reduction. As I mentioned in replying Yin's comments, (3.b) itself can actually reduce compile-time a lot (~5% or more for llvm bootstrap) without changing performance generally. If I don't add (3.b), we would see ~10% compile-time regression. I don't think it is acceptable. So now the only difference is how we update loop info. i.e. use my current naive solution, or use new pass manager to help it?</div><div><br></div><div>So do you know when Chandler's pass manager change can be ready for inlining use?</div></div></div></div></blockquote><div><br></div></span><div>It will be done in a very near future, but Chandler can give you more precise info. </div><span class=""><div> <br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div> Why we can't just use this native method first, and I can add //FIXME to highlight it, and then when Chandler's great new pass manager is ready for inlining use, we just switch to it? Eventually we would be able to see big compile-time reduction.</div></div></div></div></blockquote><div><br></div></span><div>You need to discuss with Chandler or other inliner maintainers.  It is probably possible but not in the current form of patch. IMO, the patch need to be in the current form (because of the hack to workaround limitations).</div><div><br></div></div></div></div></blockquote><div><br></div><div>How do you think of this?</div><div><br></div><div>Appreciate your response in advance!</div><div><br></div><div>Thanks,</div><div>-Jiangning</div></div></div></div></div>