[PATCH] D115750: [SLP]Further improvement of the cost model for scalars used in buildvectors.

[Alexey Bataev via Phabricator via llvm-commits]

ABataev added a comment.

In D115750#3501860 <https://reviews.llvm.org/D115750#3501860>, @aeubanks wrote:

> another crash even with https://reviews.llvm.org/rGcce80bd8b74d54deb82b1b6ae0cbec1ab53c1dbb
>
>   target datalayout = "e-m:e-p270:32:32-p271:32:32-p272:64:64-i64:64-f80:128-n8:16:32:64-S128"
>   target triple = "x86_64-unknown-linux-gnu"
>   
>   define internal void @0() {
>   .lr.ph.i41:
>     br label %.lr.ph.split.us.i
>   
>   .lr.ph.split.us.i:                                ; preds = %.lr.ph.split.us.i, %.lr.ph.i41
>     %0 = fadd float 0.000000e+00, 0.000000e+00
>     %1 = fadd float 0.000000e+00, 0.000000e+00
>     %2 = fadd float %0, 0.000000e+00
>     %3 = fadd float %1, 0.000000e+00
>     %.sroa.3.8.vec.insert.i.us.i = insertelement <2 x float> zeroinitializer, float %2, i64 0
>     %.sroa.3.12.vec.insert.i.us.i = insertelement <2 x float> %.sroa.3.8.vec.insert.i.us.i, float %3, i64 1
>     %.sroa.025.4.vec.insert.us.i = insertelement <2 x float> %.sroa.3.12.vec.insert.i.us.i, float %0, i64 0
>     br label %.lr.ph.split.us.i
>   }

Thanks, will add it to fixed version of the patch


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D115750