On 8/8/19 7:04 AM, Martin Liška wrote: > Hi. > > As requested by Richi, I'm suggesting to use new dump_printf > optimization info infrastructure. > > Patch can bootstrap on x86_64-linux-gnu and survives regression tests. > > Ready to be installed? > Thanks, > Martin > > gcc/ChangeLog: > > 2019-08-08 Martin Liska <mli...@suse.cz> > > * ipa-icf.c (sem_function::merge): Define AUTO_DUMP_SCOPE and > use dump_printf to report optimization. > (sem_variable::merge): Likwise. > (sem_item_optimizer::merge_classes): Use dump_printf to report > ICF hits. > > gcc/testsuite/ChangeLog: > > 2019-08-08 Martin Liska <mli...@suse.cz> > > * g++.dg/ipa/ipa-icf-2.C: Add -optimized to -fdump-ipa-icf. > * g++.dg/ipa/ipa-icf-3.C: Likewise. > * g++.dg/ipa/ipa-icf-4.C: Likewise. > * g++.dg/ipa/ipa-icf-6.C: Likewise. > * gcc.dg/ipa/ipa-icf-1.c: Likewise. > * gcc.dg/ipa/ipa-icf-10.c: Likewise. > * gcc.dg/ipa/ipa-icf-11.c: Likewise. > * gcc.dg/ipa/ipa-icf-12.c: Likewise. > * gcc.dg/ipa/ipa-icf-13.c: Likewise. > * gcc.dg/ipa/ipa-icf-16.c: Likewise. > * gcc.dg/ipa/ipa-icf-18.c: Likewise. > * gcc.dg/ipa/ipa-icf-2.c: Likewise. > * gcc.dg/ipa/ipa-icf-20.c: Likewise. > * gcc.dg/ipa/ipa-icf-21.c: Likewise. > * gcc.dg/ipa/ipa-icf-23.c: Likewise. > * gcc.dg/ipa/ipa-icf-25.c: Likewise. > * gcc.dg/ipa/ipa-icf-26.c: Likewise. > * gcc.dg/ipa/ipa-icf-27.c: Likewise. > * gcc.dg/ipa/ipa-icf-3.c: Likewise. > * gcc.dg/ipa/ipa-icf-35.c: Likewise. > * gcc.dg/ipa/ipa-icf-36.c: Likewise. > * gcc.dg/ipa/ipa-icf-37.c: Likewise. > * gcc.dg/ipa/ipa-icf-38.c: Likewise. > * gcc.dg/ipa/ipa-icf-39.c: Likewise. > * gcc.dg/ipa/ipa-icf-5.c: Likewise. > * gcc.dg/ipa/ipa-icf-7.c: Likewise. > * gcc.dg/ipa/ipa-icf-8.c: Likewise. > * gcc.dg/ipa/ipa-icf-merge-1.c: Likewise. > * gcc.dg/ipa/pr64307.c: Likewise. > * gcc.dg/ipa/pr90555.c: Likewise. OK. As are any adjustments you may need to make if there's conflicts or dependencies on the on the other ICF work you've got in the works.
This also seems straighforward enough that other patches that convert to using the new infrastructure are pre-approved, just post them for archival purposes. jeff
