Re: [PATCH 1/2] SH epilogue unwind, dwarf2 pass changes

2012-06-11 Thread Richard Henderson
On 2012-06-11 01:34, Chung-Lin Tang wrote: > Ping? > > On 2012/6/1 06:24 PM, Chung-Lin Tang wrote: >> On 12/5/23 1:46 AM, Richard Henderson wrote: >>> On 05/18/12 03:48, Chung-Lin Tang wrote: @@ -2401,6 +2401,7 @@ scan_trace (dw_trace_info *trace) { /* Propagate across fallt

Re: [PATCH 1/2] SH epilogue unwind, dwarf2 pass changes

2012-06-11 Thread Chung-Lin Tang
Ping? On 2012/6/1 06:24 PM, Chung-Lin Tang wrote: > On 12/5/23 1:46 AM, Richard Henderson wrote: >> On 05/18/12 03:48, Chung-Lin Tang wrote: >>> @@ -2401,6 +2401,7 @@ scan_trace (dw_trace_info *trace) >>> { >>> /* Propagate across fallthru edges. */ >>> dwarf2out_flush_queued_reg_

Re: [PATCH 1/2] SH epilogue unwind, dwarf2 pass changes

2012-06-01 Thread Chung-Lin Tang
On 12/5/23 1:46 AM, Richard Henderson wrote: > On 05/18/12 03:48, Chung-Lin Tang wrote: >> @@ -2401,6 +2401,7 @@ scan_trace (dw_trace_info *trace) >> { >>/* Propagate across fallthru edges. */ >>dwarf2out_flush_queued_reg_saves (); >> + def_cfa_1 (&this_cfa); >>ma

Re: [PATCH 1/2] SH epilogue unwind, dwarf2 pass changes

2012-05-22 Thread Richard Henderson
On 05/18/12 03:48, Chung-Lin Tang wrote: > @@ -2401,6 +2401,7 @@ scan_trace (dw_trace_info *trace) > { > /* Propagate across fallthru edges. */ > dwarf2out_flush_queued_reg_saves (); > + def_cfa_1 (&this_cfa); > maybe_record_trace_start (insn, NULL); > b

Re: [PATCH 1/2] SH epilogue unwind, dwarf2 pass changes

2012-05-22 Thread Richard Henderson
On 05/18/12 03:48, Chung-Lin Tang wrote: > Hi, > > I found a few changes were needed to the dwarf2 pass when trying to > implement epilogue unwind for SH, mainly that the current handling of > annulled-taken branches does not seem correct; the delay slot insn > should be handled in a manner simila

[PATCH 1/2] SH epilogue unwind, dwarf2 pass changes

2012-05-18 Thread Chung-Lin Tang
Hi, I found a few changes were needed to the dwarf2 pass when trying to implement epilogue unwind for SH, mainly that the current handling of annulled-taken branches does not seem correct; the delay slot insn should be handled in a manner similar to an insn in the fallthru block. Cross-tested on