On Friday 01 July 2005 01:05, Vladimir Makarov wrote: > Andrey Belevantsev wrote: > > Vladimir Makarov wrote: > >> I'll look at this PR today. > > > > We've looked today at this issue. We think the problem is that > > proposed patch of sched_get_condition() treats conditional jumps > > likely to COND_EXECs, but it doesn't fix other places in sched-deps, > > where COND_EXECs are considered. Maxim Kuvyrkov proposed the attached > > patch, which allows gcc to bootstrap on ia64 and fixes the testcase in > > PR. > > It seems like processing cond-exec was added long ago in 2000 with > keeping Itanium only in mind. But other targets with conditional > execution have other forms of jump insns. So Richard Earnshaw made a > quick fix a year ago.
No. In fact Richard fixed code that can't possibly have worked. Ever. And when he fixed it, sched_get_condition started working for the first time, which of course uncovered a bunch of bugs latent up till then. Gr. Steven
