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

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to