Re: A question about df_get_live_in

2009-07-14 Thread Kaz Kojima
Eric Botcazou wrote: > Patch attached. I'll give it a whirl on SPARC but not immediately so, Kaz, > if > you can test it on SH in the meantime, you can apply it on all branches. > > > 2009-07-14 Eric Botcazou > > PR rtl-optimization/40710 > * resource.c (mark_target_live_regs)

Re: A question about df_get_live_in

2009-07-14 Thread Eric Botcazou
> > Why does find_basic_block ignore NOTE_INSN_BASIC_BLOCK notes? > > The same question arises to me. That's explained in the head comment of find_basic_block: the CFG is destroyed by the DBR pass in some controlled way so the strategy is to recompute the liveness info starting from data that ar

Re: A question about df_get_live_in

2009-07-13 Thread Jeff Law
Steven Bosscher wrote: On Mon, Jul 13, 2009 at 11:46 PM, Kaz Kojima wrote: Hi, I hope DF/middle-end experts will comment about this. PR target/40710 http://gcc.gnu.org/bugzilla/show_bug.cgi?id=40710 is a wrong code problem on SH. A delayed slot of a conditional branch insn is wrongly fill

Re: A question about df_get_live_in

2009-07-13 Thread Kaz Kojima
Steven Bosscher wrote: > OK, so isn't the bug obvious then? You said: "mark_target_live_regs > uses df_get_live_in to get live regs for the basic block including the > opposite_thread insn which is insn 32.". And you had insn 32 in basic > block 3. So find_basic_block returns the wrong basic block

Re: A question about df_get_live_in

2009-07-13 Thread Eric Botcazou
> I doubt he can help you with this one... When your problem concerns > reorg, you should talk to people like Eric Botcazou or Richard > Sandiford or HP Nillson. I've added Eric to the CC, to make this a > happier crowd. :-) Thank you. I was about to leave for vacation but I'll stay for this on

Re: A question about df_get_live_in

2009-07-13 Thread Steven Bosscher
On Tue, Jul 14, 2009 at 1:17 AM, Kaz Kojima wrote: > [I'd like to add kenny to the CC list.] I doubt he can help you with this one... When your problem concerns reorg, you should talk to people like Eric Botcazou or Richard Sandiford or HP Nillson. I've added Eric to the CC, to make this a happi

Re: A question about df_get_live_in

2009-07-13 Thread Kaz Kojima
[I'd like to add kenny to the CC list.] Steven Bosscher wrote: > So when the CFG is still valid, r15 is live-out in basic block 2 and > live-in in basic block 3 (which contains insns 32, whatever that means > for an invalid CFG). For which bb does mark_target_live_regs call > df_get_live_in? gdb

Re: A question about df_get_live_in

2009-07-13 Thread Steven Bosscher
On Mon, Jul 13, 2009 at 11:46 PM, Kaz Kojima wrote: > Hi, > > I hope DF/middle-end experts will comment about this. > > PR target/40710 > http://gcc.gnu.org/bugzilla/show_bug.cgi?id=40710 > > is a wrong code problem on SH.  A delayed slot of a conditional > branch insn is wrongly filled with an ins