[PATCH] D13769: [X86][AVX512DQ] add scalar fpclass

Elena Demikhovsky via llvm-commits llvm-commits at lists.llvm.org
Sun Oct 18 00:23:02 PDT 2015


delena added inline comments.

================
Comment at: ../commit/llvm_clean/lib/Target/X86/X86ISelLowering.cpp:16301
@@ +16300,3 @@
+                                      Subtarget, DAG);
+      return DAG.getNode(ISD::SIGN_EXTEND, dl, MVT::i8,
+                     DAG.getNode(ISD::ANY_EXTEND, dl, MVT::i8, FPclassMask),
----------------
I did not mean that you put SIGN_EXTEND on ANY_EXTEND. It is just meaningless. 
Is this possible:
return DAG.getNode(ISD::SIGN_EXTEND, dl, MVT::i8,FPclassMask ) ?


http://reviews.llvm.org/D13769





More information about the llvm-commits mailing list