<div dir="ltr">On 21 June 2013 18:09, Ye, Mei <span dir="ltr"><<a href="mailto:Mei.Ye@amd.com" target="_blank">Mei.Ye@amd.com</a>></span> wrote:<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 lang="EN-US" link="blue" vlink="purple">
<div>
<p class="MsoNormal"><span style="color:rgb(31,73,125);font-family:Calibri,sans-serif;font-size:11pt">So the question is:  if there is a requirement of proof, is there a standard on which architecture with what kind of machine configurations?</span></p>
</div></div></blockquote><div></div></div><br></div><div class="gmail_extra" style>You took of part my comment a bit too literally, and not the most important part, I'm afraid. It's not a matter of running "a" set of benchmarks on "a" list of configurations that will "prove" your patch is good, it's a matter of thinking how we can apply the transformation you need into LLVM without breaking unknown code on unknown targets.</div>
<div class="gmail_extra" style><br></div><div class="gmail_extra" style>As Evan said, you probably implemented the transformation in the wrong place, and what we have to do now is to understand what is the best place to put it, not run benchmarks.</div>
<div class="gmail_extra" style><br></div><div class="gmail_extra" style>Also, it should be specific to your target unless it benefits the majority of targets (here is where things get confusing), and even that, should be done slowly. If you implement in the back-end, or close to it, than it'll be easier to make it specific. If not, you'll have to create a compiler flag and leave it disabled by default. By Evan's comments, I guess you'll be safer with the former option.</div>
<div class="gmail_extra" style><br></div><div class="gmail_extra" style>cheers,</div><div class="gmail_extra" style>--renato</div></div>