<div dir="ltr">On Fri, Jan 25, 2013 at 1:43 PM, Jakob Stoklund Olesen <span dir="ltr"><<a href="mailto:stoklund@2pi.dk" target="_blank" class="cremed">stoklund@2pi.dk</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 style="word-wrap:break-word"><div class="im"><br><div><div>On Jan 25, 2013, at 12:00 PM, Chandler Carruth <<a href="mailto:chandlerc@gmail.com" target="_blank" class="cremed">chandlerc@gmail.com</a>> wrote:</div>
<br><blockquote type="cite"><div style="font-family:Menlo;font-size:medium;font-style:normal;font-variant:normal;font-weight:normal;letter-spacing:normal;line-height:normal;text-align:-webkit-auto;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px">
The presence of a 'trunc' in the conditional basic block shouldn't be the thing that enables or disables this. All I wanted out of this patch was improved stability and consistency on x86.</div></blockquote><br>
</div></div><div>It basically sounds like you are already on your way down the path Duncan warned about.</div><div><br></div><div>This is not the right way to do it.</div></div></blockquote><div><br></div><div style>I really am not...</div>
<div style><br></div><div style><br></div><div style>Clearly this commit is more controversial than I had ever anticipated (including when I mentioned it to Chris on IRC), so I'm happy to back it out... It'll be a bit, as I can't just revert it at this point -- there were other patches here that fixed somewhat serious bugs -- but I'll get to it soon.</div>
<div style><br></div><div style><br></div><div style>Anyways, I think there is a lot of confusion, miscommunication, and misunderstanding in this thread (and the other on a related subject, but which had *nothing* to do with my actual commits)... So I hope we can get back to discussing the actual technical issues at hand here. Note that while I've reverted this patch, SimplifyCFG still uses TTI, and fixing that is a bigger task -- that was introduced with the lookup table optimization.</div>
<div style><br></div></div></div></div>