[PATCH 1/2] New pass to partition single function into multiple sections (take 2)

Michael Meissner gnu@the-meissners.org
Fri Sep 19 07:32:00 GMT 2008


On Mon, Sep 08, 2008 at 03:43:20PM +0300, Revital1 Eres wrote:
> 
> Hello,
> 
> Attached is an updated version of the patch, following your review.
> (Please find below replies to your comments)
> 
> The second part of the patch, which contains
> support for debug and unwind info, was not reviewed yet
> (http://gcc.gnu.org/ml/gcc-patches/2008-08/msg02234.html), so I disabled
> -fpartition-functions-into-sections for those cases. (it should be.
> enabled once the second part will be accepted)
> 
> It passes bootstrap and regtest on powerpc64 and x86_64. On SPU it.
> passes regtest (c, c++, and fortran).
> 
> Does this patch OK for mainline?
> 
> Thanks again,
> Revital

It looks like it it answers my comments about the previous set of patches.
Just to be sure, have you verified that the hot/cold partitioning still works
the same on hosts that don't support bb splitting?

-- 
Michael Meissner
email: gnu@the-meissners.org
http://www.the-meissners.org



More information about the Gcc-patches mailing list