Re: [RFC] Confusing fall through BB pc in conditional jump

2019-06-28 Thread Segher Boessenkool
Hi Kewen, On Fri, Jun 28, 2019 at 09:56:58AM +0800, Kewen.Lin wrote: > > It's probably a good idea to spend a line on this even in slim mode, it's > > really easy to miss and get confused and waste time without it. Nice :-) > > > >> The dumping will look like: > >> 6: NOTE_INSN_BASIC_BLOCK 2

Re: [RFC] Confusing fall through BB pc in conditional jump

2019-06-27 Thread Kewen.Lin
Hi Segher, Thanks a lot for the comments. on 2019/6/28 上午9:32, wrote: > Hi Kewen, > > On Thu, Jun 27, 2019 at 10:32:18AM +0800, Kewen.Lin wrote: >> 6: NOTE_INSN_BASIC_BLOCK 2 >> >>12: r135:CC=cmp(r122:DI,0) >>13: pc={(r135:CC!=0)?L52:pc} >> REG_DEAD r135:CC >>

Re: [RFC] Confusing fall through BB pc in conditional jump

2019-06-27 Thread Segher Boessenkool
Hi Kewen, On Thu, Jun 27, 2019 at 10:32:18AM +0800, Kewen.Lin wrote: > 6: NOTE_INSN_BASIC_BLOCK 2 > >12: r135:CC=cmp(r122:DI,0) >13: pc={(r135:CC!=0)?L52:pc} > REG_DEAD r135:CC > REG_BR_PROB 1041558836 >31: L31: >17: NOTE_INSN_BASIC_BLOCK 3 > > The above

[RFC] Confusing fall through BB pc in conditional jump

2019-06-26 Thread Kewen.Lin
Hi all, 6: NOTE_INSN_BASIC_BLOCK 2 12: r135:CC=cmp(r122:DI,0) 13: pc={(r135:CC!=0)?L52:pc} REG_DEAD r135:CC REG_BR_PROB 1041558836 31: L31: 17: NOTE_INSN_BASIC_BLOCK 3 The above RTL seq is from pass doloop dumping with -fdump-rtl-all-slim, I misunderstood