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