Thanks for reporting. I'll fix them now. Dehao
On Thu, Sep 20, 2012 at 6:29 AM, Michael Meissner <meiss...@linux.vnet.ibm.com> wrote: > On Fri, Sep 14, 2012 at 11:03:34AM +0800, Dehao Chen wrote: >> Hi, >> >> I've integrated all the reviews from this thread (Thank you guys for >> helping refine this patch). >> >> Now the patch can pass all gcc testsuite as well as all spec2006 >> benchmarks (with LTO). Concerning memory consumption, for extreme >> benchmarks like tramp3d, this patch incurs around 2% peak memory >> overhead (mostly from the extra blocks that have been set NULL in the >> original implementation.) >> >> Attached is the new patch. Honza, could you help me try this on >> Mozzila lto to see if the error is gone? > > The current patch that was checked in breaks powerpc, because you did not > change INSN_LOCATOR to INSN_LOCATION in rs6000_final_prescan_insn in rs6000.c. > I also see INSN_LOCATOR in the arm, bfin, c6x, mep, mips, picochip, s390, sh, > and spu ports. > > -- > Michael Meissner, IBM > 5 Technology Place Drive, M/S 2757, Westford, MA 01886-3141, USA > meiss...@linux.vnet.ibm.com fax +1 (978) 399-6899 >
