Hello, thank you for the trick in ipa-split.c. It really helped! I prepared 2 tests for Inkscape, first was just with my function reordering pass. And for the second, I enable also -freorder-blocks-and-partition (note: files ending with _blocks.xxx in attached tar).
Touched pages: just reorder functions: 1108 with -freorder-blocks-and-partition: 1120 Please see dumps at: https://drive.google.com/file/d/0B0pisUJ80pO1R19zSXR6U1Q4NmM/edit?usp=sharing Note: I put all function to a single section (for easier layout orientation). If I'm correct, there a small chunk of about 10 functions after the boundary of untouched functions and a single miss at the end of binary: __libc_csu_init. If you look at the graph line, it looks really well. I will prepare patch for mailing list as a phase 2. Martin On 5 December 2013 14:38, Jan Hubicka <hubi...@ucw.cz> wrote: >> Can you, please, send me the -flto systemtaps for gimp and/or inkscape so we >> can decide >> on the patch? We should enable it earlier in stage3 rather than later. > > I see, the PDF was included within the tar file. Was this with > -freorder-blocks-and-partition? If so, the patch is OK. > I still think we should put cold parts of hot/normal function into a > subsection different > from unlikely section, but lets handle that incrementally. > > Thanks, > Honza >> >> Honza
