[PATCH] D124348: [1/2][RISCV]Add Intrinsics for B extension in Clang

craig.topper accepted this revision. craig.topper added a comment. This revision is now accepted and ready to land. LGTM, but please fix the title to not use "B extension" since the "B extension" was never ratified. Only Zba, Zbb, Zbc, Zbs were ratified and there will never be a B. Repository:

[PATCH] D124348: [1/2][RISCV]Add Intrinsics for B extension in Clang

joker881 updated this revision to Diff 425793. joker881 added a comment. Implement intrinsic ctz for bitmanip extension Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D124348/new/ https://reviews.llvm.org/D124348 Files: clang/include/clang/Basic/B

[PATCH] D124348: [1/2][RISCV]Add Intrinsics for B extension in Clang

joker881 added a comment. In D124348#3470612 , @craig.topper wrote: > The "B extension" terminology no longer exists. And I decide to ctz_32 and ctz_64 like clz. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.o

[PATCH] D124348: [1/2][RISCV]Add Intrinsics for B extension in Clang

kito-cheng added a comment. > And I want to know that do you have any documents about intrinsic of bitmanip > extension, like RISC-V Vector Extension Intrinsic Document. No, we didn't a formal document for that, I expect we will have one once this finalized https://github.com/riscv-non-isa/risc

[PATCH] D124348: [1/2][RISCV]Add Intrinsics for B extension in Clang

joker881 added a comment. In D124348#3470612 , @craig.topper wrote: > The "B extension" terminology no longer exists. Thank you for your comments. I will consider them carefully. And I want to know that do you have any documents about intrinsic of bitm

[PATCH] D124348: [1/2][RISCV]Add Intrinsics for B extension in Clang

craig.topper added a comment. The "B extension" terminology no longer exists. Comment at: clang/include/clang/Basic/BuiltinsRISCV.def:29 +TARGET_BUILTIN(__builtin_riscv_cpopw_64, "WiWi", "nc", "zbb,64bit") +TARGET_BUILTIN(__builtin_riscv_ctz_32, "ZiZi", "nc", "zbb") +TARGET_BUI

[PATCH] D124348: [1/2][RISCV]Add Intrinsics for B extension in Clang

joker881 created this revision. Herald added subscribers: sunshaoce, VincentWu, luke957, vkmr, frasercrmck, evandro, luismarques, apazos, sameer.abuasal, s.egerton, Jim, benna, psnobl, jocewei, PkmX, the_o, brucehoult, MartinMosbeck, rogfer01, edward-jones, zzheng, jrtc27, kito-cheng, niosHD, sa