[PATCH] D122918: [RISCV][CodeGen] Support Zfinx, Zdinx, Zhinx, Zhinxmin codegen

Fri Apr 21 08:18:14 PDT 2023

asb added a comment.

In D122918#4285895 <https://reviews.llvm.org/D122918#4285895>, @sunshaoce wrote:

> We plan to split this patch into the following four parts:
>
> - zfinx
> - zdinx on rv32
> - zdinx on rv64
> - zhinx, zhinxmin
>
> And, we will address Jessica’s comments in the new patch. Thanks!

Thank you - I think this is a piece of work where you need to have implemented everything to be sure that your approach works, but splitting to allowing reviewing it in smaller chunks will be much better.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D122918