[PATCH] D142085: [Clang][RISCV] Add `__riscv_` prefix for all RVV intrinsics
Yueh-Ting (eop) Chen via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Fri Jan 20 23:53:40 PST 2023
eopXD added a comment.
> Not related to this patch, but this enum should probably be using double underscores for its names.
Thanks for mentioning this, probably should start a PR under rvv-intrinsic-doc before having a patch here.
@craig.topper @kito-cheng I don't have a preference here, is any of `__riscv_RVV_CSR`/ `__riscv_rvv_csr_type` / `__riscv_rvv_csr_t` suitable?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D142085/new/
https://reviews.llvm.org/D142085
More information about the cfe-commits
mailing list