<div dir="ltr">Since this is just for testing the feature, maybe it should be -cc1-only? Right now you allow it in the driver but don't forward it from driver to frontend. (Move the option definition from Options.td to CC1Options.td to fix this.)</div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Oct 23, 2014 at 2:47 PM, Nick Lewycky <span dir="ltr"><<a href="mailto:nlewycky@google.com" target="_blank">nlewycky@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Sorry, I failed to "svn add" the new test and include it in the patch. Here's an updated patch with the test in it.</div><div class="HOEnZb"><div class="h5"><div class="gmail_extra"><br><div class="gmail_quote">On 23 October 2014 14:39, Nick Lewycky <span dir="ltr"><<a href="mailto:nlewycky@google.com" target="_blank">nlewycky@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">The attached patch adds -fmerge-functions which is in turn attached to PassManagerBuilder::MergeFunctions. This is provided for testing it out, and we can decide what we want the default to be after we have experience. Note that it does bad things to debug info which doesn't currently have a way to represent which function got called if two functions were merged.<div><br></div><div>Please review!<span><font color="#888888"><div><br></div><div>Nick</div><div><br></div></font></span></div></div>
</blockquote></div><br></div>
</div></div><br>_______________________________________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
<br></blockquote></div><br></div>