Details
Details
Diff Detail
Diff Detail
Event Timeline
llvm/test/CodeGen/CSKY/intrinsic.ll | ||
---|---|---|
25 | It would be better to also test the condition that the second argument is zero |
llvm/test/CodeGen/CSKY/intrinsic.ll | ||
---|---|---|
12 | BTW. Because counting zeros is not expensive, it's a little opt to set isCheapToSpeculateCttz and isCheapToSpeculateCtlz as other targets. So that this sequence can be same as ctlz_1. Anyway, it's can be another patch. |
llvm/test/CodeGen/CSKY/intrinsic.ll | ||
---|---|---|
12 | Thanks. I have made another patch https://reviews.llvm.org/D156780 for that. |
BTW. Because counting zeros is not expensive, it's a little opt to set isCheapToSpeculateCttz and isCheapToSpeculateCtlz as other targets. So that this sequence can be same as ctlz_1. Anyway, it's can be another patch.