[PATCH] D109817: [AggressiveInstCombine] Add `phi` nodes support to `TruncInstCombine`

[Anton Afanasyev via Phabricator via llvm-commits]

anton-afanasyev added a comment.

In D109817#3184760 <https://reviews.llvm.org/D109817#3184760>, @lebedev.ri wrote:

> I would recommend to proceed with this.



In D109817#3188058 <https://reviews.llvm.org/D109817#3188058>, @RKSimon wrote:

> None of those size changes look like alarming - why do you want to stop work on this patch?

The patch changes previous logic without undoubted positive impact. I would expect no code size increase of any test for such change. But ok, I'm to investigate what is a real cause of  it and proceed with this.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D109817/new/

https://reviews.llvm.org/D109817