[llvm] r345395 - [X86][SSE] Move 2-input limit up from getFauxShuffleMask to resolveTargetShuffleInputs

[Eric Christopher via llvm-commits]

On Mon, Oct 29, 2018 at 8:43 AM Greg Bedwell <gregbedwell at gmail.com> wrote:

> On Sat, 27 Oct 2018 at 09:31, Simon Pilgrim via llvm-commits <
> llvm-commits at lists.llvm.org> wrote:
>
>> I've reverted this at rL345451 - but if you can send me an example of a
>> failing test (was it an assert or bad codegen?) that'd be great - my own
>> fuzz tests haven't found anything and I need to fix this to support
>> combining shuffle with subvectors.
>>
>
> I'm not sure whether this is related to the failure that Eric saw, but
> I've raised http://llvm.org/PR39483 for the assertion failure related to
> this commit that we found in our testing.
>

Asserts weren't on in our testing, so how about we go with this for now and
I can try again after :)

-eric
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20181029/88ad6a8a/attachment.html>