r225958 - Use the integrated assembler by default on 32-bit PowerPC and SPARC.

Brad Smith brad at comstyle.com
Wed Jan 14 19:18:29 PST 2015


On 01/14/15 22:08, Chandler Carruth wrote:
>
> On Wed, Jan 14, 2015 at 7:04 PM, Brad Smith <brad at comstyle.com
> <mailto:brad at comstyle.com>> wrote:
>
>         FWIW, I couldn't find any of these code reviews through simple
>         searching. They're probably there and I've just missed them, but
>         I think
>         others may have missed them as well and also been confused. It would
>         have been nice to let people know this was landing now. Switching
>         between the integrated assembler and the system assembler is
>         very often
>         a disruptive process. Doing it a few hours before 3.6 branches seems
>         like a *really* bad idea under any circumstances. This is the
>         kind of
>         thing that should happen right after a release branches rather than
>         right before. We should at least make sure that 3.6 doesn't
>         include this
>         change.
>
>
>     I switched it because all of the BSD's have already been using this
>     on the 32-bit PowerPC side of the house for quite awhile now and that's
>     where more or less all the testing and development has been going on
>     for 32-bit PowerPC as far as I can see. The SPARC backend is still
>     quite experimental and has rough edges. I could understand the concern
>     if either backend was more or less 100% perfect or close to it but
>     they're not. There is still a lot of work going on to make the relevant
>     architectures usable and by usable I mean building a whole OS not
>     just a few userland programs here and there.
>
>
> FWIW, I still don't think it's reasonable to change strategy like this
> just before cutting a release. I think that 3.6 shouldn't have this
> change so that we can prepare for it in the run up to 3.7.

The goal was to have it in for the release but I don't have a problem
delaying it to right after the branch.


-- 
This message has been scanned for viruses and
dangerous content by MailScanner, and is
believed to be clean.




More information about the cfe-commits mailing list