[PATCH] D55722: [DAGCombiner] scalarize binop followed by extractelement

Ulrich Weigand via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Dec 18 02:30:16 PST 2018


uweigand added a comment.

In D55722#1334123 <https://reviews.llvm.org/D55722#1334123>, @jonpa wrote:

> I tried reverting my patch for the DAG type legalizer, and found that vec-trunc-to-i1.ll still fails with this new patch applied. It fails during type legalization, while this patch at least in this case gets enabled only after type legalization. So I think the test still serves its purpose.


Ah, good.  Thanks for checking!


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

https://reviews.llvm.org/D55722





More information about the llvm-commits mailing list