craig.topper added inline comments.
================
Comment at: llvm/lib/Support/RISCVISAInfo.cpp:827
+ {{"zvkb"}, {ImpliedExtsZve64x}},
+ {{"zvkg"}, {ImpliedExtsZve32x}},
+ {{"zvknha"}, {ImpliedExtsZve32x}},
----------------
ego wrote:
> What is the reasoning between 32 vs 64 for those sub-extensions?
> I do not think that extensions using 64bxN element groups are restricted to
> rv64.
>
> No matter what the end result is, I would welcome some comments explaining
> the encoded semantics.
The 32 and 64 refer to the ELEN. Zknhb requires SEW=64 so ELEN must be 64
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D138807/new/
https://reviews.llvm.org/D138807
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
