[PATCH] D155690: [RISCV] Mask instructions in Zkt as constant-time

[Craig Topper via Phabricator via llvm-commits]

craig.topper added inline comments.


================
Comment at: llvm/lib/Target/RISCV/RISCVInstrInfoZb.td:335
 
 let Predicates = [HasStdExtZbbOrZbkb, IsRV64], IsSignExtendingOpW = 1 in {
 def ROLW  : ALUW_rr<0b0110000, 0b001, "rolw">,
----------------
craig.topper wrote:
> Are ROLW/RORW not in the constant time list? Surely that's a miss in the table in the spec since it says "The Zbkb, Zbkx and Zbkx extensions are included in their entirety."
PR for the spec https://github.com/riscv/riscv-crypto/pull/347


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D155690