Re: [PATCH] dwarf2cfi: Improve cfa_reg comparisons [PR103619]

2021-12-14 Thread Jeff Law via Gcc-patches
On 12/14/2021 3:27 PM, Jakub Jelinek wrote: On Tue, Dec 14, 2021 at 03:05:37PM -0700, Jeff Law wrote: 2021-12-14 Jakub Jelinek PR debug/103619 * dwarf2cfi.c (dwf_cfa_reg): Remove gcc_assert. (operator==, operator!=): New overloaded operators. (dwarf2out_fra

Re: [PATCH] dwarf2cfi: Improve cfa_reg comparisons [PR103619]

2021-12-14 Thread Jakub Jelinek via Gcc-patches
On Tue, Dec 14, 2021 at 03:05:37PM -0700, Jeff Law wrote: > > 2021-12-14 Jakub Jelinek > > > > PR debug/103619 > > * dwarf2cfi.c (dwf_cfa_reg): Remove gcc_assert. > > (operator==, operator!=): New overloaded operators. > > (dwarf2out_frame_debug_adjust_cfa, dwarf2out_frame_debug

Re: [PATCH] dwarf2cfi: Improve cfa_reg comparisons [PR103619]

2021-12-14 Thread Jeff Law via Gcc-patches
On 12/14/2021 1:18 PM, Jakub Jelinek wrote: On Tue, Dec 14, 2021 at 10:32:21AM -0700, Jeff Law wrote: I think the attached testcase should trigger on c6x with -mbig-endian -O2 -g Thanks. Finally I see what's going on. c6x doesn't really need the CFA with span > 1 (and I bet neither does ar

[PATCH] dwarf2cfi: Improve cfa_reg comparisons [PR103619]

2021-12-14 Thread Jakub Jelinek via Gcc-patches
On Tue, Dec 14, 2021 at 10:32:21AM -0700, Jeff Law wrote: > I think the attached testcase should trigger on c6x with -mbig-endian -O2 -g Thanks. Finally I see what's going on. c6x doesn't really need the CFA with span > 1 (and I bet neither does armbe), the only reason why dwf_cfa_reg is called