This is the mail archive of the gcc-patches@gcc.gnu.org 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] |
On 2018-11-28 5:49 p.m., Segher Boessenkool wrote: > Hi! > > On Wed, Nov 28, 2018 at 04:09:14PM -0500, Michael Ploujnikov wrote: >> I've also included a small change to rs6000 which I'm pretty sure is >> safe, but I have no way of testing. > > Do you have an account on the GCC Compile Farm? > https://cfarm.tetaneutral.net/ > There are some nice Power machines in there! > > Does this patch dependent on the rest of the series? > > If it tests okay, it is okay for trunk of course. Thanks! > > One comment about your changelog: > >> 2018-11-28 Michael Ploujnikov <michael.ploujnikov@oracle.com> >> >> * config/rs6000/rs6000.c (make_resolver_func): Generate >> resolver symbol with clone_function_name instead of >> clone_function_name_numbered. > > Those last two lines should not start with the spaces. It should be: > > 2018-11-28 Michael Ploujnikov <michael.ploujnikov@oracle.com> > > * config/rs6000/rs6000.c (make_resolver_func): Generate > resolver symbol with clone_function_name instead of > clone_function_name_numbered. > > > Segher > Thanks for the review and suggestion to use the cfarm! I've successfully regtested this on power9 and committed the stand-alone change to rs6000.c after fixing the changelog. - Michael
Attachment:
signature.asc
Description: OpenPGP digital signature
Index Nav: | [Date Index] [Subject Index] [Author Index] [Thread Index] | |
---|---|---|
Message Nav: | [Date Prev] [Date Next] | [Thread Prev] [Thread Next] |