alextsao1999 added a comment. In D118011#3265374 <https://reviews.llvm.org/D118011#3265374>, @Jimerlife wrote:
> In D118011#3265071 <https://reviews.llvm.org/D118011#3265071>, @craig.topper > wrote: > >> Will you also be updating clang's RISCVBuiltins.def? >> >> Can you add tests? > > I update BuiltinsRISCV.def and add "__builtin_riscv_clmul_kc" intrinsic for > zbkc extension, but I am not sure if the name is appropriate. https://github.com/rvkrypto/rvkrypto-fips/blob/main/rvkintrin.md Is the name as `__builtin_riscv_clmul_32` or `__builtin_riscv_clmul_64` better for conflict resolution? 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
