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

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Jul 30 21:59:03 PDT 2023


MaskRay added a comment.

In D156642#4545571 <https://reviews.llvm.org/D156642#4545571>, @xobs wrote:

> The other issue is that without this change, libunwind cannot compile on targets such as rv32imac since that instruction set does not include `csrr` as an opcode.
>
> If you like (and if it's possible to do so), I can update the description to reflect that fact. Or we can gate it on `__riscv_zicsr` instead, as suggested in https://reviews.llvm.org/D136264#4545515

The question is related to what configurations libunwind needs to support. I think this is unfortunately a bit vague.
Can you describe your configuration by providing detailed cmake command lines?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D156642



More information about the llvm-commits mailing list