[all-commits] [llvm/llvm-project] 1c4f11: [X86][SSE] getShuffleScalarElt - minor NFC cleanup.

Simon Pilgrim via All-commits all-commits at lists.llvm.org
Wed May 6 06:06:42 PDT 2020


  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: 1c4f118d893fd8042b0ce781d68242e88337be62
      https://github.com/llvm/llvm-project/commit/1c4f118d893fd8042b0ce781d68242e88337be62
  Author: Simon Pilgrim <llvm-dev at redking.me.uk>
  Date:   2020-05-06 (Wed, 06 May 2020)

  Changed paths:
    M llvm/lib/Target/X86/X86ISelLowering.cpp

  Log Message:
  -----------
  [X86][SSE] getShuffleScalarElt - minor NFC cleanup.

Use SelectionDAG::MaxRecursionDepth instead of (equal) hard coded constant.

clang-format


  Commit: 8650b36935ae8e9d584550539161e5851a17a912
      https://github.com/llvm/llvm-project/commit/8650b36935ae8e9d584550539161e5851a17a912
  Author: Simon Pilgrim <llvm-dev at redking.me.uk>
  Date:   2020-05-06 (Wed, 06 May 2020)

  Changed paths:
    M llvm/lib/Target/X86/X86ISelLowering.cpp

  Log Message:
  -----------
  [X86][SSE] Move VZEXT_MOVL removal into SimplifyDemandedVectorEltsForTargetNode

This patch replaces the VZEXT_MOVL removal from combineShuffle with a more general version based in SimplifyDemandedVectorEltsForTargetNode.

By using computeKnownBits we can always remove the VZEXT_MOVL if the upper elements of the source operand are known to be zero.

This requires us to add the conversion ops to computeKnownBitsForTargetNode as well.

Reviewed By: @craig.topper

Differential Revision: https://reviews.llvm.org/D79335


Compare: https://github.com/llvm/llvm-project/compare/055ea585c77c...8650b36935ae


More information about the All-commits mailing list