From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 81022 invoked by alias); 11 Dec 2017 22:33:32 -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 80998 invoked by uid 89); 11 Dec 2017 22:33:31 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-2.6 required=5.0 tests=AWL,BAYES_00,RCVD_IN_DNSWL_NONE,SPF_PASS,T_RP_MATCHES_RCVD autolearn=ham version=3.3.2 spammy=Hx-languages-length:2322, town X-HELO: mail-vk0-f53.google.com Received: from mail-vk0-f53.google.com (HELO mail-vk0-f53.google.com) (209.85.213.53) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Mon, 11 Dec 2017 22:33:30 +0000 Received: by mail-vk0-f53.google.com with SMTP id s197so5218985vke.11 for ; Mon, 11 Dec 2017 14:33:29 -0800 (PST) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:in-reply-to:references:from:date :message-id:subject:to:cc; bh=nhUjDO2U+/j4n5zRlq5oFQjjykCCEkcm8E1wSokMfUg=; b=m0dFDCDJxPZLuOcez2i+DDMBPX4g21VgOccQYQ/36CaNyQdib2MMS4m5gSEKVVDdHm wbdjvzTWqJmVNCZB6XcjJ3ktEU63gCzzzGcnrUuP5kszTFuTaPZbG+MdI8D7B05ur1v9 15MZMlgYkrhe3ELtiXyITuf8n5ML3TgQnwrT5tM3+x4io5ygDTSJoJ3kOX8ghhGkMtpw sGvNPFoV4i/s8CLZ+JGZ2H9LhO/RFAEk51eX9t46xXDgtWu2KUKnzi5xBvEY+wu5Y5zm R3e2SbZVBJwTEACUaXNbYOqqET0cBx8/8cNqDnEs/NBC8bFa6Uzawf+3BIVqziqudy5s oX6Q== X-Gm-Message-State: AKGB3mJvWma3PJ5BRYpb6rf9tR8bW8sDC3bDPI4qriFcCRK+HJvhCO/N onzK+0FI1WWAq2AsfLQjfhalEC4fqqlwBLZ8zA1vmQ== X-Google-Smtp-Source: ACJfBosNhHtCSJc9X4LiUkRPKo1jpz7H6sW5bF83NYZ3n0/J9XhwwVGyD1df45hnODoRNzqJpy2/EO1KEVpmEHV9D+0= X-Received: by 10.31.52.196 with SMTP id b187mr2052232vka.23.1513031607723; Mon, 11 Dec 2017 14:33:27 -0800 (PST) MIME-Version: 1.0 Received: by 10.31.148.3 with HTTP; Mon, 11 Dec 2017 14:33:27 -0800 (PST) In-Reply-To: References: <87376dlcb9.fsf_-_@immunant.com> <87r2t7j584.fsf@immunant.com> From: "Sriraman Tallam via gcc-patches" Reply-To: Sriraman Tallam Date: Mon, 11 Dec 2017 22:33:00 -0000 Message-ID: Subject: Re: [PATCH] [GOLD] Add plugin API for processing plugin-added input files To: Stephen Crane Cc: Cary Coutant , Binutils , GCC Patches Content-Type: text/plain; charset="UTF-8" X-IsSubscribed: yes X-SW-Source: 2017-12/txt/msg00663.txt.bz2 On Mon, Dec 11, 2017 at 2:16 PM, Stephen Crane wrote: > Thanks for committing the GCC portion and following up on this. I had > been meaning to write and ask. I don't have commit privs for binutils, > so either you or Cary will have to commit the binutils patch as well, > if it's not too much trouble. I think much has changed to need a > rebase? Ok, let me apply your patch. I will get back if there are inconsistencies. Thanks Sri > > Thanks, > Stephen > > On Mon, Dec 11, 2017 at 2:10 PM, Sriraman Tallam wrote: >> On Thu, Nov 9, 2017 at 9:04 PM, Cary Coutant wrote: >>>> include/ChangeLog: >>>> 2017-11-09 Stephen Crane >>>> >>>> * plugin-api.h: Add new plugin hook to allow processing of input >>>> files added by a plugin. >>>> (ld_plugin_new_input_handler): New funcion hook type. >>>> (ld_plugin_register_new_input): New interface. >>>> (LDPT_REGISTER_NEW_INPUT_HOOK): New enum val. >>>> (tv_register_new_input): New member. >>>> >>>> >>>> gold/ChangeLog: >>>> 2017-11-09 Stephen Crane >>>> >>>> * plugin.cc (Plugin::load): Include hooks for register_new_input >>>> in transfer vector. >>>> (Plugin::new_input): New function. >>>> (register_new_input): New function. >>>> (Plugin_manager::claim_file): Call Plugin::new_input if in >>>> replacement phase. >>>> * plugin.h (Plugin::set_new_input_handler): New function. >>>> * testsuite/plugin_new_section_layout.c: New plugin to test >>>> new_input plugin API. >>>> * testsuite/plugin_final_layout.sh: Add new input test. >>>> * testsuite/Makefile.am (plugin_layout_new_file): New test case. >>>> * testsuite/Makefile.in: Regenerate. >>> >>> These are OK. Thanks! >>> >>> Sri, I'm out of town through 11/18, and won't be able to commit the >>> include/ patch to GCC before Stage 1 ends. Can you take care of it? >>> (If not, I'll take care of it when I get back -- it was approved >>> during Stage 1, so I think it's OK to commit early in Stage 3, >>> especially since it's nothing but new declarations.) >> >> Stephen, I was looking at binutils and realized this patch has not >> been committed yet. I only committed the GCC portion, plugin-api.h. >> >> Thanks >> Sri >> >>> >>> -cary