This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH][i386]Fix PR 57756
- From: Sriraman Tallam <tmsriram at google dot com>
- To: David Edelsohn <dje dot gcc at gmail dot com>
- Cc: Michael Meissner <meissner at linux dot vnet dot ibm dot com>, GCC Patches <gcc-patches at gcc dot gnu dot org>, David Edelsohn <dje at gcc dot gnu dot org>, David Li <davidxl at google dot com>, Richard Biener <richard dot guenther at gmail dot com>
- Date: Wed, 16 Oct 2013 18:53:48 -0700
- Subject: Re: [PATCH][i386]Fix PR 57756
- Authentication-results: sourceware.org; auth=none
- References: <CAAs8HmztsmmpUNucXECgDd-GDjKGv-2CWwFMFC5=LUNeyasaZg at mail dot gmail dot com> <CAAs8Hmz1qS4ZvpN-jU96Y4h2QhpWwOAQ2iEuwyjHUXOoPGMJ5w at mail dot gmail dot com> <CAFiYyc2y5-6j5wqhgz30y4ZvZgMvekPwxTdxy7kt_0s=rpD+CA at mail dot gmail dot com> <CAAs8HmzaRuskG1AcVEmiYuVQCNzXp8kjY5oMTTdhhDhG5juJ5g at mail dot gmail dot com> <CAAs8Hmy7bU2fMRqtRUiX76EEe4Ma8NBdnPsbWCAJz5zAof8oLg at mail dot gmail dot com> <CAAs8Hmy0H4VB-0mJx_4=OE1eyMbSm7Lo40kcWi42CQzymtYU4Q at mail dot gmail dot com> <CAFULd4YguDhSbUtoEU+Qw60HoKkw+64s9hEJTcAv-9dGxdOy+Q at mail dot gmail dot com> <CAAs8Hmxt8TEN8ZWF1WuhHkTY9faa1wH12FtHUJOVa-UTm54D0Q at mail dot gmail dot com> <20131016055411 dot GJ20756 at bubble dot grove dot modra dot org> <CAAs8HmyL=7ZgsiPGK=ZUh6j3PWudi3i_FObR3_0Yr3YkBfVBXw at mail dot gmail dot com> <20131016231314 dot GA16300 at ibm-tiger dot the-meissners dot org> <CAAs8HmzdmE1w9hGzR4BVo9dFw9JkuUiJ7j1SNXYYXrAh+9J50Q at mail dot gmail dot com> <CAGWvny=gRGugwLbFCHpSJhdFgTYOUKbDk6jXaOb=P+UfY+RyFg at mail dot gmail dot com>
On Wed, Oct 16, 2013 at 6:06 PM, David Edelsohn <dje.gcc@gmail.com> wrote:
> On Wed, Oct 16, 2013 at 7:23 PM, Sriraman Tallam <tmsriram@google.com> 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
http://gcc.gnu.org/contribute.html and mentions the compile farm. So,
this is my fault alone for not testing powerpc & mips.
Sri
>
> Thanks, David