[PATCH] D99317: [RISCV] Add RISCVISD opcodes for CLZW and CTZW.

Alex Bradbury via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Mar 31 08:06:01 PDT 2021


asb accepted this revision.
asb added a comment.

LGTM, this seems the right direction to go in.

In theory I much prefer patterns in .td files wherever possible (shorter pattern description, harder for silly logic mistakes to hide within boilerplate etc). But the reality is obviously they can become brittle if above a certain level of complexity, as with many RV64 '*W' patterns.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D99317/new/

https://reviews.llvm.org/D99317



More information about the llvm-commits mailing list