[PATCH] D130141: [RISCV] Add part support of RISCV Zc Extension (Zca, Zcb, Zcmp)

Xinlong Wu via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jul 20 01:45:41 PDT 2022


VincentWu added a comment.

In D130141#3664635 <https://reviews.llvm.org/D130141#3664635>, @craig.topper wrote:

> Where is it documented that these extensions are ratified?

sorry for my wrong statements, I have corrected my description

> Putting 3 extensions in one patch is a guaranteed way to make it take longer to approved. The smaller you can make patches the quicker individual pieces can be reviewed and approved. I would also recommend putting new optimization passes in separate patches. MC layer and CodeGen should be separate patches.
>
> Please run clang-format on patches.

I will split this patch as much as passable soon.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D130141



More information about the llvm-commits mailing list