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
