[PATCH] D155669: [RISCV] Remove zvk uimm constraints

Eric Gouriou via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jul 19 07:14:43 PDT 2023


ego added a comment.

LGTM. 
I was initially surprised however the current spec makes all values valid by prescribing how seemingly "out of range" immediate field values get re-interpreted as being in-range. This wasn't the case in early version of the spec.

Thanks for bringing the behavior in line with the spec.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D155669



More information about the llvm-commits mailing list