<div class="gmail_quote">On Mon, Apr 16, 2012 at 11:25 PM, Jakob Stoklund Olesen <span dir="ltr"><<a href="mailto:stoklund@2pi.dk">stoklund@2pi.dk</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div style="word-wrap:break-word"><div><div class="h5"><br><div><div>On Apr 16, 2012, at 2:18 PM, Chandler Carruth <<a href="mailto:chandlerc@google.com" target="_blank">chandlerc@google.com</a>> wrote:</div><br><blockquote type="cite">
<div class="gmail_quote">On Mon, Apr 16, 2012 at 10:49 PM, Jakob Stoklund Olesen <span dir="ltr"><<a href="mailto:stoklund@2pi.dk" target="_blank">stoklund@2pi.dk</a>></span> wrote:<br><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">

Author: stoklund<br>
Date: Mon Apr 16 15:49:06 2012<br>
New Revision: 154857<br>
<br>
URL: <a href="http://llvm.org/viewvc/llvm-project?rev=154857&view=rev" target="_blank">http://llvm.org/viewvc/llvm-project?rev=154857&view=rev</a><br>
Log:<br>
Disable code placement for this test.<br>
<br>
It makes it less sensitive to small changes in heuristics.<br></blockquote><div><br></div><div>Cool, should I go back and do this to all the tests I had to update with the code placement flip? I wasn't sure it was worth it, and I wasn't aware we already had an option for this.</div>

</div>
</blockquote></div><br></div></div><div>I am not sure what is the right approach. Ideally, code placement should be less sensitive to small changes, and we wan't to keep you honest by having a bunch of tests fail when you randomly shuffle blocks.</div>
<div><br></div><div>How about disabling it only for the metastable tests? The ones that seem to flip between equally good solutions.</div></div></blockquote><div><br></div><div>That sounds good to me.</div><div><br></div>
<div>My particular flip was... not "equally good" but "radically different", but this should be a good rubrik for future changes and tweaks. =] Thanks!</div></div>