From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 25490 invoked by alias); 28 Nov 2018 22:49:17 -0000 Mailing-List: contact gcc-patches-help@gcc.gnu.org; run by ezmlm Precedence: bulk List-Id: List-Archive: List-Post: List-Help: Sender: gcc-patches-owner@gcc.gnu.org Received: (qmail 25473 invoked by uid 89); 28 Nov 2018 22:49:17 -0000 Authentication-Results: sourceware.org; auth=none X-Spam-SWARE-Status: No, score=-1.9 required=5.0 tests=BAYES_00,RCVD_IN_DNSWL_NONE autolearn=ham version=3.3.2 spammy=there! X-HELO: gate.crashing.org Received: from gate.crashing.org (HELO gate.crashing.org) (63.228.1.57) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Wed, 28 Nov 2018 22:49:15 +0000 Received: from gate.crashing.org (localhost.localdomain [127.0.0.1]) by gate.crashing.org (8.14.1/8.14.1) with ESMTP id wASMnA3g010240; Wed, 28 Nov 2018 16:49:10 -0600 Received: (from segher@localhost) by gate.crashing.org (8.14.1/8.14.1/Submit) id wASMn3OX010215; Wed, 28 Nov 2018 16:49:03 -0600 Date: Wed, 28 Nov 2018 22:49:00 -0000 From: Segher Boessenkool To: Michael Ploujnikov Cc: Martin Jambor , Richard Biener , GCC Patches , Jan Hubicka Subject: Re: [PATCH v3] Make function clone name numbering independent. Message-ID: <20181128224902.GT23873@gate.crashing.org> References: <85bd8119-0bce-a06d-df3f-1a1a6ed88187@oracle.com> <3167e521-aa5b-e47c-6d9b-956a1af2a886@oracle.com> <078bf036-f02b-3bdb-545d-3f5ad67bc786@oracle.com> <3f9e8c2a-6112-b595-a932-137958e61f5e@oracle.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.4.2.3i X-IsSubscribed: yes X-SW-Source: 2018-11/txt/msg02379.txt.bz2 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 > > * 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 * config/rs6000/rs6000.c (make_resolver_func): Generate resolver symbol with clone_function_name instead of clone_function_name_numbered. Segher