[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