[PATCH] D119530: [RISCV] Add combination crypto extensions in ISAInfo

Craig Topper via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Feb 12 23:46:13 PST 2022


craig.topper added inline comments.


================
Comment at: llvm/lib/Support/RISCVISAInfo.cpp:754
 static const char *ImpliedExtsZvl64b[] = {"zvl32b"};
 static const char *ImpliedExtsZk[] = {"zkn", "zkt", "zkr"};
 static const char *ImpliedExtsZkn[] = {"zbkb", "zbkc", "zbkx", "zkne", "zknd", "zknh"};
----------------
VincentWu wrote:
> i'm not sure did you notice that?
> 
> could you explain what's different about this patch?
I believe this patch is inferring Zk when all of zkn, zkr, and zkt are listed in march. This is needed so that __riscv_zk gets defined in the frontend preprocessor when the individual extensions are listed individually.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D119530



More information about the llvm-commits mailing list