[PATCH] D66718: [DAGCombiner] (insert_vector_elt (vector_shuffle X, Y), (extract_vector_elt X, N), IdxC) -> (vector_shuffle X, Y)

[Roman Lebedev via Phabricator via llvm-commits]

lebedev.ri added a comment.

In D66718#1646859 <https://reviews.llvm.org/D66718#1646859>, @deadalnix wrote:

> In D66718#1645956 <https://reviews.llvm.org/D66718#1645956>, @lebedev.ri wrote:
>
> > Looks like the link was incorrect, this didn't auto-close?
>
>
> Why should this auto close?
>
> It wasn't landed yet. I was wondering if the creation of vector shuffles should be normalized. I think it may be worth doing.


Sorry, i got confused by "commits" tab


Repository:
  rL LLVM

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

https://reviews.llvm.org/D66718