[PATCH] D113888: [SDAG] Use UnknownSize for masked load/store MMO size

[Dave Green via Phabricator via llvm-commits]

dmgreen added a comment.

In D113888#3152950 <https://reviews.llvm.org/D113888#3152950>, @yubing wrote:

> Hi, @dmgreen With this patch, the llc command crashed. Would you take a look?
> llc -mcpu=core-avx2 main.ll
>
>   define void @main() unnamed_addr #0 {
>   entry:
>     %P.i150.i.i = alloca [3 x [3 x double]], align 16
>     %0 = bitcast [3 x [3 x double]]* %P.i150.i.i to <8 x double>*
>     call void @llvm.masked.store.v8f64.p0v8f64(<8 x double> zeroinitializer, <8 x double>* %0, i32 8, <8 x i1> <i1 true, i1 true, i1 true, i1 true, i1 true, i1 true, i1 false, i1 false>)
>     ret void
>   }
>   
>   ; Function Attrs: argmemonly nofree nosync nounwind willreturn writeonly
>   declare void @llvm.masked.store.v8f64.p0v8f64(<8 x double>, <8 x double>*, i32 immarg, <8 x i1>)

Thanks. Seems that a masked store is turned back into an (unmasked) store, still with an unknown size. The assert seems superfluous, but I will try and fix the size when we promote the masked store to a store too.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D113888