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

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun May 3 02:39:43 PDT 2020


RKSimon added a comment.

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.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D79231





More information about the llvm-commits mailing list