Re: option GPROF on riscv64

2023-07-23 Thread Jeremie Courreges-Anglas
On Sat, Jul 22 2023, Scott Cheloha wrote: [...] > Is there any difference if you duplicate the contents of > intr_disable/intr_restore in MCOUNT_ENTER/MCOUNT_EXIT directly? > Unlikely, but maybe the compiler is shoving an _mcount() call into > intr_disable/intr_restore, causing infinite

Re: option GPROF on riscv64

2023-07-22 Thread Scott Cheloha
On Fri, Jul 21, 2023 at 08:41:32PM +0200, Jeremie Courreges-Anglas wrote: > > Spotted while testing a diff from cheloha@, option GPROF doesn't build > on riscv64 because MCOUNT_ENTER/MCOUNT_EXIT from > riscv64/include/profile.h haven't been adapted for riscv64. > > riscv64

option GPROF on riscv64

2023-07-21 Thread Jeremie Courreges-Anglas
Spotted while testing a diff from cheloha@, option GPROF doesn't build on riscv64 because MCOUNT_ENTER/MCOUNT_EXIT from riscv64/include/profile.h haven't been adapted for riscv64. riscv64 /sys/arch/riscv64/compile/GPROF.MP$ doas -u build make cc -g -Werror -Wall -Wimplicit-function-declaration