[PATCH] D121670: [RISCV] Add zihintntl instructions

asb added a comment. I've created a PR on the ISA manual to try to clarify the confusing overloaded use of the term "reserved" https://github.com/riscv/riscv-isa-manual/pull/990 Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D121670/new/

[PATCH] D121670: [RISCV] Add zihintntl instructions

asb added a comment. In D121670#4192158 , @joshua-arch1 wrote: > It seems that llvm implementation is different from Binutils/GCC. Binutils > didn't regard ntl instructions as aliases of add since encoding space is > reserved for HINT instructions.

[PATCH] D121670: [RISCV] Add zihintntl instructions

joshua-arch1 added a comment. Herald added subscribers: jobnoorman, luke. It seems that llvm implementation is different from Binutils/GCC. Binutils didn't regard ntl instructions as aliases of add since encoding space is reserved for HINT instructions. Repository: rG LLVM Github Monorepo

[PATCH] D121670: [RISCV] Add zihintntl instructions

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG7167a4207ee2: [RISCV] Add zihintntl instructions (authored by sunshaoce). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D121670: [RISCV] Add zihintntl instructions

kito-cheng accepted this revision. kito-cheng added a comment. This revision is now accepted and ready to land. LGTM Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D121670/new/ https://reviews.llvm.org/D121670

[PATCH] D121670: [RISCV] Add zihintntl instructions

sunshaoce updated this revision to Diff 438563. sunshaoce added a comment. Herald added a subscriber: shiva0217. Rebase Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D121670/new/ https://reviews.llvm.org/D121670 Files:

[PATCH] D121670: [RISCV] Add zihintntl instructions

s updated this revision to Diff 415703. s added a comment. Herald added a project: clang. Herald added a subscriber: cfe-commits. Add preprocessor test Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D121670/new/ https://reviews.llvm.org/D121670