eopXD added inline comments.

================
Comment at: clang/lib/Sema/SemaChecking.cpp:4576
+  case RISCVVector::BI__builtin_rvv_vaeskf1_vi_ta:
+  case RISCVVector::BI__builtin_rvv_vsm4k_vi_ta:
+    return SemaBuiltinConstantArgRange(TheCall, 1, 0, 31);
----------------
Valid range of `vaeskf1`, `vaeskf2` seems to be 0 to 15. [0]
Valid range of `vsm4k` seems to be 0 to 7 [1].



[0] 
https://github.com/riscv/riscv-crypto/blob/master/doc/vector/insns/vaeskf1.adoc 
[1] 
https://github.com/riscv/riscv-crypto/blob/master/doc/vector/insns/vsm4k.adoc


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D138810

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to