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