[PATCH] D139682: [DAG] Stop replace masked compressstore with normal masked store
Xiang Zhang via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Sun Dec 11 18:04:27 PST 2022
xiangzhangllvm added inline comments.
================
Comment at: llvm/test/CodeGen/X86/masked_compressstore_isel.ll:15-16
; CHECK-NEXT: %3:vk16wm = VPMOVW2MZ128rr killed %2
-; CHECK-NEXT: VPMOVDWZ256mrk $noreg, 1, $noreg, 0, $noreg, killed %3, %0 :: (store unknown-size into `i16* null`, align 16)
+; CHECK-NEXT: %4:vr128x = VPMOVDWZ256rr %0
+; CHECK-NEXT: VPCOMPRESSWZ128mrk $noreg, 1, $noreg, 0, $noreg, killed %3, killed %4 :: (store unknown-size into `i16* null`, align 16)
; CHECK-NEXT: RET 0
----------------
pengfei wrote:
> xiangzhangllvm wrote:
> > pengfei wrote:
> > > Can we optimizate it to truncated `compressstore`?
> > Do we have truncated compressstore?
> > May be can implement by targets.
> > But for target independent code we can not directly transfer to normal masted store.
> I think the truncated xxx is just a code concept, it actually means turning the trunction from value to the mask.
> So I think we should combine these 2 lines into `VPCOMPRESSWZ256mrk ...` just like mask store.
Can we first let it in and then try do the optimization ?
I wish to close a must fix bug first. thanks!
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D139682/new/
https://reviews.llvm.org/D139682
More information about the llvm-commits
mailing list