This is the mail archive of the mailing list for the GCC project.

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH][i386]Fix PR 57756

On Wed, Oct 16, 2013 at 6:06 PM, David Edelsohn <> wrote:
> On Wed, Oct 16, 2013 at 7:23 PM, Sriraman Tallam <> wrote:
>> I was unable to build a native powerpc compiler. I checked for
>> build_target_node and build_optimization_node throughout and changed
>> rs6000 because it had references. I did not realize
>> function_specific_save and function_specific_restore have to be
>> changed. Sorry for breaking it.
> As Mike replied, gcc110 is available.  Richard Biener's approval was
> dependent upon successful bootstrap and passing the regression
> testsuite, which you did not even attempt, nor did you try to build a
> cross-compiler.  You also did not contact the rs6000 maintainer (me)
> nor the last person who changed the code in question (Mike).

Yes, I should have done this.

> How is Google going to change its patch commit policies to ensure that
> this does not happen again?

Looking at our best practices document, it points to and mentions the compile farm. So,
this is my fault alone for not testing  powerpc & mips.


> Thanks, David

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]