[PATCH] D118011: [RISCV] Adjust predicates and update intrinsic for clmul and clmulh in Zbkc extension

Xinlong Wu via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Jan 24 07:43:37 PST 2022


VincentWu added a comment.

Hi, Jimerlife

it is a great work to implement the intrinsic of `zbkb`.
however, I don't know whether you have noticed that there is already to patch https://reviews.llvm.org/D112774 and https://reviews.llvm.org/D102310.
these two patches have included almost all the work you did in your patch.

I think we should better avoid overlap as much as possible.

therefore, could you please comment on the difference between these two patches as you see it?
let's focus on only one version.

You are also welcome to join the bi-weekly LLVM RISC-V meetings to discuss with us if you want. You can subscribe to the calendar via this link https://calendar.google.com/calendar/embed?src=lowrisc.org_0n5pkesfjcnp0bh5hps1p0bd80%40group.calendar.google.com


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D118011



More information about the cfe-commits mailing list