[PATCH] D148483: RISC-V Zvk (vector crypto) specification update to 0.4 (Zvbb/Zvbc/Zvkt/Zvkng/Zvksg)

[Eric Gouriou via Phabricator via llvm-commits]

ego marked 5 inline comments as done.
ego added inline comments.


================
Comment at: llvm/lib/Support/RISCVISAInfo.cpp:901
+
+  if ((Exts.count("zvkg") || Exts.count("zvkn") || Exts.count("zvkned") ||
+       Exts.count("zvkng") || Exts.count("zvknha") || Exts.count("zvks") ||
----------------
asb wrote:
> I'm not sure it makes sense to list composite extensions like zvkn and zvks and zvksg here, as the dependencies are going to be checked on the sub-extensions they imply anyway.
Thanks. I removed the combo extensions from the list.


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

https://reviews.llvm.org/D148483