[libcxx-commits] [PATCH] D156642: libunwind: riscv: disable vector test when __riscv_v not defined

Alex Bradbury via Phabricator via libcxx-commits libcxx-commits at lists.llvm.org
Sun Jul 30 23:47:46 PDT 2023


asb requested changes to this revision.
asb added a comment.

Thanks for the patch @xobs - you're right there's a problem here, but I think your current approach needs a slight tweak. `__riscv_v` corresponds to a specific set of the vector extension - you'd be better off checking for `__riscv_zicsr` as you suggest earlier in the thread, or `__riscv_vector`, which is defined if V or any of the Zve* extensions is available (see here <https://github.com/riscv-non-isa/riscv-c-api-doc/blob/master/riscv-c-api.md#preprocessor-definitions>).

On a sidenote: all the other registers are copied into `Registers_riscv` and then read from there - is it correct that vlenb is read directly through a CSR read on demand?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D156642



More information about the libcxx-commits mailing list