This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [tuples] Convert loop header copying


Hi,

> >this patch converts loop header copying, and related cfg manipulation
> >and code size estimation functions.  Also, it makes us avoid using
> >labels in GIMPLE_COND statements once cfg is build as in mainline
> >(restores the parts of the code in cfg building and pretty-printing
> >that got lost -- cfg manipulation and rtl expansion, and probably
> >all the other places already expect that this is the case).
> 
> Thanks.  The patch is fine, however, there is some coordination we need 
> to do:
> 
> 1- Bill has a patch to estimate_num_insns that I had already approved 
> before he had a chance to post it.
> 
> 2- Aldy is now doing a merge which is giving him some grief because of 
> some OMP changes in mainline.
> 
> So, I will ask folks to refrain from committing stuff to the branch 
> until Aldy is done with the merge.

OK

> Also, Zdenek could you commit your 
> patch after Bill's change?  Your patch is a superset of Bill's, so once 
> his stuff is in, your final patch should be smaller.

Actually, if that is the case, it perhaps would make more sense to
commit just my patch (I would avoid having to resolve the conflicts, and
estimate_num_insns would be rewritten anyway)?  But I'm just guessing,
let Bill post his patch and we may discuss it then,

Zdenek


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]