[PATCH] D23984: [X86] Fix bug in masked compress

Igor Breger via llvm-commits llvm-commits at lists.llvm.org
Mon Aug 29 01:52:51 PDT 2016


igorb added inline comments.

================
Comment at: lib/Target/X86/X86ISelLowering.cpp:18879
@@ -18877,2 +18878,3 @@
+                           Mask, LoadAddress, Subtarget, DAG);
     return DAG.getStore(Chain, dl, Compressed, Addr,
                         MemIntr->getMemOperand());
----------------
Chain should be changed to LoadAddress 

================
Comment at: test/CodeGen/X86/compress-maskz.ll:1
@@ +1,2 @@
+;RUN: llc < %s -O0 -mattr=+avx512vl -mtriple=x86_64-unknown | FileCheck %s --check-prefix=CHECK
+
----------------
could you please move the test to avx512vl-intrinsics.ll


https://reviews.llvm.org/D23984





More information about the llvm-commits mailing list