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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
