[PATCH] D152628: [RISCV] Add __builtin_riscv_zip/unzip for Zbkb to match gcc.

Mon Jun 26 16:24:16 PDT 2023

craig.topper added a comment.
Herald added a subscriber: wangpc.

In D152628#4416794 <https://reviews.llvm.org/D152628#4416794>, @asb wrote:

> I'm not super familiar with these builtins so this might be a silly question why are the new builtins added in this patch LiLi (long int) rather than ZiZi (int32_t) like the old `_32` suffixed builtins?

In my head I was thinking this would be the name we would use for xlen for i32/i64 too so I was matching clmul. But maybe that's not a good idea. I think we just need to figure out the header file definition for these intrinsics and get both compilers to agree on that.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D152628