[PATCH] D103788: [InstCombine] Eliminate casts to optimize ctlz operation
Datta Nagraj via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Jun 10 09:23:42 PDT 2021
datta.nagraj added a comment.
In D103788#2810722 <https://reviews.llvm.org/D103788#2810722>, @lebedev.ri wrote:
> If we look at the sequence with `trunc`, then preconditions are different.
> I'm not sure if we have *some* problematic combination of src bitwidth and bitwidth increase,
> but the hard cut-off that is currently there is not correct.
I am currently not sure of what to do here Sir, also I have no clue as to why some unit tests are failing, because they don't even contain trunc inst, and don't even hit my changes.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D103788/new/
https://reviews.llvm.org/D103788
More information about the llvm-commits
mailing list