<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Thu, Oct 24, 2013 at 1:15 PM, Shankar Easwaran <span dir="ltr"><<a href="mailto:shankare@codeaurora.org" target="_blank">shankare@codeaurora.org</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div>On 10/24/2013 3:10 PM, Chandler Carruth wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
On Thu, Oct 24, 2013 at 1:04 PM, Nick Kledzik <<a href="mailto:kledzik@apple.com" target="_blank">kledzik@apple.com</a>> wrote:<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
There is a catch-22 here.  Without the new passes it is hard for anyone to<br>
fix the other problems in lld that were causing the test case failures.<br>
<br>
Can we do this in steps:<br>
1) commit the code changes with the lines that use the two new passes<br>
commented out<br>
2) owners of the broken areas can uncomment the code (enabling the passes)<br>
and fix their issues<br>
3) the final commit is to enable the two new passes when all test cases<br>
pass<br>
<br>
</blockquote>
If folks want this, cool. I usually use a macro to enable/disable.<br>
<br>
That said, I think that Shankar should make a reasonable effort to fix the<br>
other targets. I don't think its likely that hard. He hadn't even asked for<br>
help due to it being harder than expected when he landed this patch.<br>
</blockquote></div></div>
I am going to have the two lines as mentioned by Nick. The problem which I see is there are parts of code that are unimplemented, which needs to be fixed(which I have highlighted in my earlier mail).<br>
<br>
I would leave that to the owners of each flavor to handle in whichever form is needed as it requires thorough ABI understanding of how it runs(PECOFF/Darwin).<br>
<br>
To summarize I would like to have the owners of each flavor fix those areas.</blockquote><div><br></div><div>Can you let each flavor opt in to the behavior, rather than breaking them?</div><div><br></div><div>It's also not clear to me that the code review finished.</div>

</div></div></div>