Re: [PATCH, SMS 2/2] Support instructions with REG_INC_NOTE (second try)

2011-09-30 Thread Revital Eres
Hello, > OK, with the following comments: > > Make sure reg_moves are generated for the correct (result, not addr) > register, in generate_reg_moves(). > > "been">>"being" (multiple appearances). > > Add a note that autoinc_var_is_used_p (rtx def_insn, rtx use_insn) > doesn't need to consider the

Re: [PATCH, SMS 2/2] Support instructions with REG_INC_NOTE (second try)

2011-09-27 Thread Revital Eres
Hello, > Ok, this does have two anti-dep edges. But still, only a single true > dependence(?) ... can you see why? The intra edge [3681 -(T,2,0)-> 3682] was created by haifa-sched and I guess that because both of the expected true-dep edges (one for the target and one for the address) are identic

Re: [PATCH, SMS 2/2] Support instructions with REG_INC_NOTE (second try)

2011-09-27 Thread Ayal Zaks
On Tue, Sep 27, 2011 at 9:47 AM, Revital Eres wrote: > Hello, > >> ok, so if we have an auto-inc'ing insn which defines (auto-inc's) an >> addr register and another (say, result) register, we want to allow the >> result register to have life ranges in excess of ii (by eliminating >> anti-dep edges

Re: [PATCH, SMS 2/2] Support instructions with REG_INC_NOTE (second try)

2011-09-26 Thread Revital Eres
Hello, > ok, so if we have an auto-inc'ing insn which defines (auto-inc's) an > addr register and another (say, result) register, we want to allow the > result register to have life ranges in excess of ii (by eliminating > anti-dep edges of distance 1 from uses to def, and generating > reg_moves i

Re: [PATCH, SMS 2/2] Support instructions with REG_INC_NOTE (second try)

2011-09-26 Thread Ayal Zaks
On Mon, Sep 26, 2011 at 7:31 AM, Revital Eres wrote: > Hello, > > This patch extends the implementation to support instructions with > REG_INC notes. > It addresses the comments from the previous submission: > http://gcc.gnu.org/ml/gcc-patches/2011-08/msg01299.html. > ok, so if we have an auto-in

[PATCH, SMS 2/2] Support instructions with REG_INC_NOTE (second try)

2011-09-25 Thread Revital Eres
Hello, This patch extends the implementation to support instructions with REG_INC notes. It addresses the comments from the previous submission: http://gcc.gnu.org/ml/gcc-patches/2011-08/msg01299.html. btw, regarding your previous question about the usage of the address register been auto inc, ap