Hi Youling, On Tue, 2023-05-09 at 09:32 +0800, Youling Tang wrote: > Thanks for pointing out that there is no need to mask the lower 32 bits > here. > > It can be modified as follows, > --- a/backends/loongarch_initreg.c > +++ b/backends/loongarch_initreg.c > @@ -79,11 +79,8 @@ loongarch_set_initial_registers_tid (pid_t tid > __attribute__ ((unused)), > if (ptrace (PTRACE_GETREGSET, tid, NT_FPREGSET, &iovec) != 0) > return false; > > - Dwarf_Word dwarf_fregs[32]; > - for (int r = 0; r < 32; r++) > - dwarf_fregs[r] = fregs.fpr[r] & 0xFFFFFFFF; > - > - if (! setfunc (32, 32, dwarf_fregs, arg)) > + /* $f0-$f31 */ > + if (! setfunc (32, 32, &fregs.fpr[0], arg)) > > Do I need to send the v2 patchset again?
Please just sent this patch rebased against the current git branch. I already pushed the other 4 patches since they looked good. Thanks, Mark
