[PATCH] D37017: [DAGCombiner] fold assertzexts separated by trunc

[Sanjay Patel via Phabricator via llvm-commits]

spatel added a comment.

In https://reviews.llvm.org/D37017#874263, @spatel wrote:

> In https://reviews.llvm.org/D37017#874229, @craig.topper wrote:
>
> > PR28540 is fixed now.
>
>
> Thanks - so now for the questionable test, we'll get movl+negq rather than negq+movq (no changes to the output from this patch).


That comment wasn't clear. I meant we still get the same DAG nodes for the questionable test from this combine:

      t16: i32 = AssertZext t2, ValueType:ch:i1
    t18: i64 = any_extend t16
  t15: i64 = sign_extend_inreg t18, ValueType:ch:i1


https://reviews.llvm.org/D37017