[PATCH] D152250: [Clang][RISCV] Add test cases for intrinsics clz/ctz codegen when has extension zbb/xtheadbb

2023-06-07 Thread Yunze Zhu(Thead) via Phabricator via cfe-commits
Yunzezhu abandoned this revision. Yunzezhu added a comment. Abandon this revision because https://reviews.llvm.org/D151867 is abandoned. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D152250/new/ https://reviews.llvm.org/D152250 ___

[PATCH] D151867: [Clang][RISCV] Make generic clz/ctz builtins defined for zero on RISCV targets.

2023-06-07 Thread Yunze Zhu(Thead) via Phabricator via cfe-commits
Yunzezhu abandoned this revision. Yunzezhu added a comment. In D151867#4401987 , @craig.topper wrote: > In D151867#4401952 , @Yunzezhu > wrote: > >> In D151867#4400255 ,

[PATCH] D151867: [Clang][RISCV] Make generic clz/ctz builtins defined for zero on RISCV targets.

2023-06-06 Thread Yunze Zhu(Thead) via Phabricator via cfe-commits
Yunzezhu added a comment. In D151867#4400255 , @craig.topper wrote: > From the C language perspective with this change, __builtin_clz/ctz is still > considered undefined for 0 and code that uses it is ill-formed. > `isCLZForZeroUndef` is only intended

[PATCH] D151867: [Clang][RISCV] Make generic clz/ctz builtins defined for zero on RISCV targets.

2023-06-06 Thread Yunze Zhu(Thead) via Phabricator via cfe-commits
Yunzezhu added a comment. In D151867#4398483 , @asb wrote: > Could you please post a separate patch that has a test that will show the > codegen change (and demonstrate how it is unchanged when zbb or xtheadbb)? Sure. I post a patch to demonstrate curre

[PATCH] D152250: [Clang][RISCV] Add test cases for intrinsics clz/ctz codegen when has extension zbb/xtheadbb

2023-06-06 Thread Yunze Zhu(Thead) via Phabricator via cfe-commits
Yunzezhu created this revision. Yunzezhu added reviewers: asb, craig.topper, kito-cheng. Yunzezhu added a project: clang. Herald added subscribers: jobnoorman, luke, VincentWu, vkmr, frasercrmck, luismarques, apazos, sameer.abuasal, s.egerton, Jim, benna, psnobl, jocewei, PkmX, the_o, brucehoult,

[PATCH] D151867: [Clang][RISCV] Make generic clz/ctz builtins defined for zero on RISCV targets.

2023-06-06 Thread Yunze Zhu(Thead) via Phabricator via cfe-commits
Yunzezhu updated this revision to Diff 528698. Yunzezhu added a comment. I made the flag isCLZForZeroUndef set to false only when target support extension zbb or xtheadbb, and this will prevent making codegen worse when target does not support abb or xtheadbb. Repository: rG LLVM Github Mono

[PATCH] D151867: [Clang][RISCV] Make generic clz/ctz builtins defined for zero on RISCV targets.

2023-06-01 Thread Yunze Zhu(Thead) via Phabricator via cfe-commits
Yunzezhu created this revision. Yunzezhu added reviewers: asb, craig.topper, kito-cheng. Yunzezhu added a project: clang. Herald added subscribers: jobnoorman, luke, VincentWu, vkmr, frasercrmck, luismarques, apazos, sameer.abuasal, s.egerton, Jim, benna, psnobl, jocewei, PkmX, the_o, brucehoult,

[PATCH] D149401: [Clang][RISCV] Set native half type to legal when has zfh extension

2023-04-27 Thread Yunze Zhu(Thead) via Phabricator via cfe-commits
Yunzezhu abandoned this revision. Yunzezhu added a comment. In D149401#4304053 , @craig.topper wrote: > Is this the same as https://reviews.llvm.org/D145071 Yes and I will abandon this one. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACT

[PATCH] D149401: [Clang][RISCV] Set native half type to legal when has zfh extension

2023-04-27 Thread Yunze Zhu(Thead) via Phabricator via cfe-commits
Yunzezhu created this revision. Yunzezhu added reviewers: asb, kito-cheng, craig.topper. Herald added subscribers: jobnoorman, luke, VincentWu, vkmr, frasercrmck, evandro, luismarques, apazos, sameer.abuasal, s.egerton, Jim, benna, psnobl, jocewei, PkmX, the_o, brucehoult, MartinMosbeck, rogfer01