[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