On Mon, Dec 11, 2017 at 2:16 PM, Stephen Crane <s...@immunant.com> 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 <tmsri...@google.com> wrote: >> On Thu, Nov 9, 2017 at 9:04 PM, Cary Coutant <ccout...@gmail.com> wrote: >>>> include/ChangeLog: >>>> 2017-11-09 Stephen Crane <s...@immunant.com> >>>> >>>> * 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 <s...@immunant.com> >>>> >>>> * 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
