<html><head><meta http-equiv="Content-Type" content="text/html charset=iso-8859-1"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><br><div><div>On Dec 11, 2012, at 7:51 PM, Eric Christopher <<a href="mailto:echristo@gmail.com">echristo@gmail.com</a>> wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><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="auto"><div>Thanks for the review, I'll add a testcase.</div>
</div></blockquote><div><br></div><div>Awesome.</div></div></div></blockquote>Attached a patch for the test case (separate patch as this one will go into clang).<br><blockquote type="cite"><div class="gmail_extra"><div class="gmail_quote"><div> </div><blockquote class="gmail_quote" style="margin: 0px 0px 0px 0.8ex; border-left-width: 1px; border-left-color: rgb(204, 204, 204); border-left-style: solid; padding-left: 1ex; position: static; z-index: auto; "><div dir="auto"><div>Regarding the performance impact, tell me what do you want me to measure/run.</div>
<div><br></div></div></blockquote><div><br></div><div> Since it probably only affects -Oz input a timing/size run of projects/test-suite with Oz before and after should be sufficient to show some size wins and what kind of performance impact we're looking at. You may also want to run SPEC as well from the extras repo.</div></div></div></blockquote><div><br></div><div>Thanks, I'll look at that.</div><div><br></div><div>-Quentin</div><div><br></div><div></div></div></body></html>