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

Thu Feb 3 05:05:31 PST 2022

spatel added a comment.

In D118376#3292090 <https://reviews.llvm.org/D118376#3292090>, @haowei wrote:

> Hi, we found out that this patch breaks following tests in the Fuchsia's stage-2 mac clang builders:
>
>   Clang :: Refactor/Extract/ExtractExprIntoFunction.cpp
>   Clang :: Refactor/Extract/ExtractionSemicolonPolicy.cpp
>   Clang :: Refactor/Extract/ExtractionSemicolonPolicy.m
>   Clang :: Refactor/Extract/FromMethodToFunction.cpp
>   Clang :: Refactor/Extract/ObjCProperty.m
>
> An example of the failure: https://ci.chromium.org/ui/p/fuchsia/builders/prod/clang-mac-x64/b8823769353085181809/overview . And test error messages can be found at https://ci.chromium.org/ui/p/fuchsia/builders/prod/clang-mac-x64/b8823769353085181809/test-results . These failures only happens in stage 2 build so it took us some time to bisect and confirm it. Could you take a look please? And if it takes a long time to fix, could you revert this change first please? Thanks.

Thanks for letting me know. I don't have a system set up to debug a stage 2 failure, so this is going to take a while unless someone else has a way to diagnose/reduce the errors more quickly. I will revert within a day if there is no update/progress.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D118376