[PATCH] D79335: [X86][SSE] Move VZEXT_MOVL removal into SimplifyDemandedVectorEltsForTargetNode

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon May 4 14:00:15 PDT 2020


RKSimon marked an inline comment as done.
RKSimon added inline comments.


================
Comment at: llvm/lib/Target/X86/X86ISelLowering.cpp:36574
+  }
+  case X86ISD::CVTP2SI:
+  case X86ISD::CVTP2UI:
----------------
craig.topper wrote:
> Should this be in computeKnownBitsForTargetNode instead since we're not doing the simplification?
Sure, I'll check what it looks like using computeKnownBits instead.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D79335





More information about the llvm-commits mailing list