[PATCH] D115462: [SLP]Improve shuffles cost estimation where possible.

[Alexey Bataev via Phabricator via llvm-commits]

ABataev added a comment.

In D115462#3613570 <https://reviews.llvm.org/D115462#3613570>, @akhuang wrote:

> I saw another crash on chromium builds - not sure if this is related to the previous mentioned crash but here's a creduced repro:
>
>   struct f {
>     float g;
>     float h;
>   };
>   struct j {
>     j() = default;
>     j(float k, float l) : c(k), d(l) {}
>     j operator*(j k) const { return j(k.a + c, b + k.d); }
>     float a = 1.0f;
>     float b = 0.0f;
>     float c = 0.0f;
>     float d = 1.0f;
>     float e = 0.0f;
>   };
>   struct m {
>     j n() const;
>     f o;
>     j p;
>   };
>   j m::n() const {
>     if (o.g || o.h)
>       return j();
>     j a;
>     return p * a;
>   }
>
> build with
>
>   clang -cc1 -O2 -vectorize-slp -emit-llvm -fno-delete-null-pointer-checks t.cpp

Yes, the cause is the same but need to adjust the patch D128680 <https://reviews.llvm.org/D128680> a bit.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D115462