<div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Jan 14, 2015 at 5:46 PM, Brad Smith <span dir="ltr"><<a href="mailto:brad@comstyle.com" target="_blank">brad@comstyle.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On 01/14/15 14:45, Chandler Carruth wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<span class="il">Brad</span>, I think it is completely unacceptable to change the default<br>
assembler behavior of every OS that delegates to <span class="il">Generic_GCC</span> without any<br>
warning, heads up email, or discussion on the lists.<br>
</blockquote>
<br></span>
It was on the list for everyone to see. It isn't as if its hiding.</blockquote><div><br></div><div>You submitted it without review, which means no one had a chance to see and comment. I specifically asked for a discussion on the mailing lists. If you want to change the behavior of all the OSes using this, then you at least need to *ask* whether the maintainers there want the change.</div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class=""><br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
It is especially unacceptable to do so when there are active problems on<br>
build bots and the tests aren't passing. This wasn't the first time this<br>
patch caused a problem either, and you are forcing several other<br>
developers to chase down build bot failures.<br>
</blockquote>
<br></span>
What build bot failures? I haven't seen any and with the first patch I<br>
did.</blockquote></div><br>Duncan cited bot failures when he reverted the patch the first time and Rafael pointed to an openbsd build bot failure. While I don't have the links handy, I don't think they were making them up.</div></div>