Re: [PATCH] backends: add abi_cfi and register_info callbacks for RISC-V

2018-07-26 Thread Mark Wielaard
On Tue, 2018-07-24 at 16:13 +0200, Andreas Schwab wrote: > How about this, does it look reasonable?  It doesn't handle SET6/SUB6 > yet, and eu-readelf -wframe doesn't agree with readelf -wf, so this > is only a RFC. I don't really like adding not really existing types to the translation types ELF_

Re: [PATCH] backends: add abi_cfi and register_info callbacks for RISC-V

2018-07-24 Thread Andreas Schwab
How about this, does it look reasonable? It doesn't handle SET6/SUB6 yet, and eu-readelf -wframe doesn't agree with readelf -wf, so this is only a RFC. Andreas. diff --git a/backends/riscv_symbol.c b/backends/riscv_symbol.c index dce8e3586b..866be8f093 100644 --- a/backends/riscv_symbol.c +++ b/

Re: [PATCH] backends: add abi_cfi and register_info callbacks for RISC-V

2018-07-11 Thread Andreas Schwab
On Jun 20 2018, Mark Wielaard wrote: > aha, ok, but it looks like RISCV_(ADD|SUB)(8|16|32|64) should not be > too hard to handle. The other ones (RISCV_SETX) might be a little > harder. The SETX relocs are simple absolute relocs (SET32 is the same as 32, but the dynamic linker does not need to h

Re: [PATCH] backends: add abi_cfi and register_info callbacks for RISC-V

2018-06-20 Thread Mark Wielaard
On Wed, 2018-06-20 at 15:55 +0200, Andreas Schwab wrote: > It's probably because there are more than just simple relocations on > debug sections, more than reloc_simple_type can handle. aha, ok, but it looks like RISCV_(ADD|SUB)(8|16|32|64) should not be too hard to handle. The other ones (RISCV_S

Re: [PATCH] backends: add abi_cfi and register_info callbacks for RISC-V

2018-06-20 Thread Mark Wielaard
On Wed, 2018-06-20 at 14:38 +0200, Andreas Schwab wrote: > On Jun 20 2018, Mark Wielaard wrote: > > > > FAIL: run-low_high_pc.sh > > > > > > lowpc: 220, highpc: 220lx > > > ../../elfutils/src/size.c: [c84] 'handle_elf' highpc <= lowpc > > > > > > lowpc: 41c, highpc: 41clx > > > ../../elfutils/s

Re: [PATCH] backends: add abi_cfi and register_info callbacks for RISC-V

2018-06-20 Thread Andreas Schwab
On Jun 20 2018, Mark Wielaard wrote: >> FAIL: run-low_high_pc.sh >> >> lowpc: 220, highpc: 220lx >> ../../elfutils/src/size.c: [c84] 'handle_elf' highpc <= lowpc >> >> lowpc: 41c, highpc: 41clx >> ../../elfutils/src/strip.c: [1c00] 'update_section_size' highpc <= >> lowpc > > That is odd. For w

Re: [PATCH] backends: add abi_cfi and register_info callbacks for RISC-V

2018-06-20 Thread Mark Wielaard
Hi Andreas, On Mon, 2018-06-18 at 09:59 +0200, Andreas Schwab wrote: > On Jun 15 2018, Mark Wielaard wrote: > > > How does the result of make check look now on a native riscv > > system? > > FAIL: run-native-test.sh > > return_value_location is missing Sadly DWARF doesn't describe how return

Re: [PATCH] backends: add abi_cfi and register_info callbacks for RISC-V

2018-06-18 Thread Andreas Schwab
On Jun 15 2018, Mark Wielaard wrote: > How does the result of make check look now on a native riscv system? FAIL: run-native-test.sh return_value_location is missing FAIL: run-low_high_pc.sh lowpc: 220, highpc: 220lx ../../elfutils/src/size.c: [c84] 'handle_elf' highpc <= lowpc lowpc: 41c, h

Re: [PATCH] backends: add abi_cfi and register_info callbacks for RISC-V

2018-06-14 Thread Mark Wielaard
On Wed, Jun 13, 2018 at 12:28:00PM +0200, Andreas Schwab wrote: > From > > and GCC source. This looks good as far as I can see. How does the result of make check look now on a native riscv system? Could you provid

[PATCH] backends: add abi_cfi and register_info callbacks for RISC-V

2018-06-13 Thread Andreas Schwab
From and GCC source. Signed-off-by: Andreas Schwab --- backends/Makefile.am | 2 +- backends/riscv_cfi.c | 75 ++ backends/riscv_init.c | 4 + backends/riscv_regs.c | 177 +++