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

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to