[PATCH] D35788: [DAGCombiner] Extending pattern detection for vector shuffle.
Jatin Bhateja via llvm-commits
llvm-commits at lists.llvm.org
Mon Jul 24 22:00:10 PDT 2017
Review patch was generated against my sandbox which was last updated on
17/07/2017. Will rebase with trunk top.
This diff makes vector argument splitting efficient as it uses only minimal
required subvector needed to create a vector_shuffle.
On Tue, Jul 25, 2017 at 10:27 AM, Soft Dev <soft.devl81 at gmail.com> wrote:
> Hi Zvi,
> Review patch was generated against my sandbox which was last updated on
> 17/07/2017. Will rebase with trunk top.
> This diff makes vector argument splitting efficient as it uses only
> minimal required subvector needed to create a vector_shuffle.
> On Tue, Jul 25, 2017 at 1:41 AM, Zvi Rackover via Phabricator <
> reviews at reviews.llvm.org> wrote:
>> zvi added a comment.
>> See https://reviews.llvm.org/D35700 where pr33784 is handled a bit
>> Comment at: test/CodeGen/X86/pr33784-vector-shuffle.ll:1
>> +; NOTE: Assertions have been autogenerated by
>> RKSimon wrote:
>> > zvi wrote:
>> > > Can you please commit the test and re-apply the patch so that we can
>> see the change in the generated code?
>> > Please can you add this test to shuffle-vs-trunc-512.ll instead of
>> creating a new test file?
>> This case already exists in test/CodeGen/X86/shuffle-strid
>> Can you please verify you are rebased on top-of-trunk?
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the llvm-commits