[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