RKSimon added a comment.
Close this? It appears to have been committed, including the auto upgrade tests
requested by Craig.
https://reviews.llvm.org/D20359
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
craig.topper requested changes to this revision.
craig.topper added a comment.
This revision now requires changes to proceed.
Actually can you add autoupgrade support?
http://reviews.llvm.org/D20359
___
cfe-commits mailing list
craig.topper accepted this revision.
craig.topper added a comment.
This revision is now accepted and ready to land.
LGTM
http://reviews.llvm.org/D20359
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
RKSimon added a subscriber: RKSimon.
RKSimon added a comment.
Should AVX512 store support (non-temporal or otherwise) be added to
X86FastISel::X86FastEmitStore ?
http://reviews.llvm.org/D20359
___
cfe-commits mailing list
m_zuckerman added a comment.
Yes, we can delete it. We don't need it any more .
http://reviews.llvm.org/D20359
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
m_zuckerman updated this revision to Diff 58201.
http://reviews.llvm.org/D20359
Files:
include/llvm/IR/IntrinsicsX86.td
lib/Target/X86/X86ISelLowering.cpp
lib/Target/X86/X86IntrinsicsInfo.h
test/CodeGen/X86/avx512-intrinsics.ll
Index: lib/Target/X86/X86IntrinsicsInfo.h
craig.topper added inline comments.
Comment at: lib/Target/X86/X86IntrinsicsInfo.h:279
@@ -278,3 +278,1 @@
X86_INTRINSIC_DATA(avx512_scattersiv8_si, SCATTER, X86::VPSCATTERDDZ256mr,
0),
- X86_INTRINSIC_DATA(avx512_storent_pd_512, STOREANT, ISD::DELETED_NODE, 0),
-
