Re: Merging tuples branch into mainline today
"Steven Bosscher" <[email protected]>
| Newsgroups | gmane.comp.gcc.fortran,gmane.comp.gcc.devel,gmane.comp.gcc.java.devel |
|---|---|
| Message-ID | <[email protected]> |
On Fri, Jul 25, 2008 at 6:19 PM, Mark Mitchell <[email protected]> wrote: > Diego Novillo wrote: > >>> Why not fix that before merging, then? >> >> Because it is not a pass that is run by default, it is not receiving >> active maintenance and the cost/benefit of doing it pre-merge is lower >> than fixing the pass post-merge. If that is a problem, we can keep >> delaying the merge until it's fixed, of course. > > Normally, we try to get major new functionality to meet the no-regression > test before merging. However, I can see that in the case of tuples it might > be easier for everyone to stop having to worry about tuplifying new code. > > I think that someone, though, should be committed to fixing this pass ASAP > after it's checked in; waiting until late August to fix it seems bad. Is > there someone else who can commit to working on it as a high priority after > the main tuples checkin? If this pass is effectively unmaintained, why not just remove it? Unmaintained passes that are not enabled at any optimization level are usually broken anyway. Gr. Steven