[PATCH] D118376: [x86] try harder to scalarize a vector load with extracted integer op uses

Fri Mar 4 08:53:44 PST 2022

RKSimon added a comment.

In D118376#3360031 <https://reviews.llvm.org/D118376#3360031>, @manojgupta wrote:

> In D118376#3360001 <https://reviews.llvm.org/D118376#3360001>, @manojgupta wrote:
>
>> We are noticing some failures in internal unit tests with this change.
>>
>> Tests pass at 588f121ada6d541 <https://reviews.llvm.org/rG588f121ada6d5418290e271bcaf2d1bada500912> but start failing at b4b97ec813a0258 <https://reviews.llvm.org/rGb4b97ec813a02585000f30ac7d532dda74e8bfda>.
>
> Oh, the patch was already reverted at 7b037250978 <https://reviews.llvm.org/rG7b03725097872fbd3369a7213c1d98b372aa2d78>. I'll test it again and report back if there are issues.

@manojgupta Please can you confirm that everything is OK after the patch was recommitted at rGc486b82cfbe59929a80e5f29bab82112555c8bf4 <https://reviews.llvm.org/rGc486b82cfbe59929a80e5f29bab82112555c8bf4>

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D118376