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.ploujni...@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.ploujni...@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
signature.asc
Description: OpenPGP digital signature
