XUANTIE-RV / thead-extension-spec

XuanTie vendor extension Instruction Set spec
Apache License 2.0
30 stars 20 forks source link

`XTheadFMemIdx`: No parens? #7

Closed a4lg closed 1 year ago

a4lg commented 1 year ago

As I pointed out in #6, there are some inconsistencies in XTheadMemIdx.

XTheadFMemIdx does not have such inconsistencies in the manual but replacing rs1 with (rs1) may be an option to keep consistency with other RISC-V instructions.

Is not having parens around rs1 is intentional? If not, we should fix both the manual and GNU Binutils (I can handle the GNU Binutils part).

Cooper-Qu commented 1 year ago

As I mentioned in the https://github.com/T-head-Semi/thead-extension-spec/issues/6

I will fix the document. And if you can fix in the GNU Binutils, I will would really appreciate it.