[PATCH] D79231: [X86] Custom legalize v16i64->v16i8 truncate with avx512.

Sun May 3 10:05:18 PDT 2020

craig.topper added a comment.

In D79231#2016648 <https://reviews.llvm.org/D79231#2016648>, @RKSimon wrote:

> In D79231#2016102 <https://reviews.llvm.org/D79231#2016102>, @craig.topper wrote:
>
> > In D79231#2014842 <https://reviews.llvm.org/D79231#2014842>, @RKSimon wrote:
> >
> > > Does anything need to be done for truncstore cases?
> >
> >
> > It looks like previously we would form a truncstore using vmovdb after type legalization. After this patch we no longer do that.
>
>
> Is it still a perf gain if we lose the truncstore? I can't see any truncstore test changes in the patch, but we might just be missing test coverage.

I don’t think the store unit does the truncate. I think its still does the truncate in the shuffle unit and then does a separate store. I’ll double check uops.info.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D79231