On 01/14/15 22:08, Chandler Carruth wrote:

On Wed, Jan 14, 2015 at 7:04 PM, Brad Smith <[email protected]
<mailto:[email protected]>> 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.

_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to