From: Richard Henderson <richard.hender...@linaro.org> > On 9/10/21 4:26 AM, Luis Pires wrote: > > +&Z22_bf_fra bf fra dm > > +@Z22_bf_fra ...... bf:3 .. fra:5 dm:6 ......... . &Z22_bf_fra > > + > > +%z22_frap 17:4 !function=times_2 > > +@Z22_bf_frap ...... bf:3 .. ....0 dm:6 ......... . &Z22_bf_fra > fra=%z22_frap > > How confusing. There's a typo in the manual for these insns, with the minor > opcode (XO) field at the wrong location. It's correct in the summary of > instruction formats at the beginning of the manual.
Right! This was also the case for dscli[q]/dscri[q]. I've reported this, and it should be fixed in the next release of the ISA. > Functional change: you're no longer storing nip. It does seem wrong, but that > fix should be broken out to a separate patch. Thanks, I've moved the nip update changes to a separate patch in the new series. -- Luis Pires Instituto de Pesquisas ELDORADO Aviso Legal - Disclaimer <https://www.eldorado.org.br/disclaimer.html>