[PATCH v3] Make function clone name numbering independent.

Michael Ploujnikov michael.ploujnikov@oracle.com
Thu Nov 29 14:13:00 GMT 2018


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

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 488 bytes
Desc: OpenPGP digital signature
URL: <http://gcc.gnu.org/pipermail/gcc-patches/attachments/20181129/f21d8fd7/attachment.sig>


More information about the Gcc-patches mailing list