[all-commits] [llvm/llvm-project] 68aeaa: [X86] Don't count the chain uses when forming broa...

[topperc via All-commits]

  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: 68aeaab888094fa192bf9257ffd5929b93778a70
      https://github.com/llvm/llvm-project/commit/68aeaab888094fa192bf9257ffd5929b93778a70
  Author: Craig Topper <craig.topper at gmail.com>
  Date:   2020-03-03 (Tue, 03 Mar 2020)

  Changed paths:
    M llvm/lib/Target/X86/X86ISelLowering.cpp
    M llvm/test/CodeGen/X86/pr35443.ll

  Log Message:
  -----------
  [X86] Don't count the chain uses when forming broadcast loads in lowerBuildVectorAsBroadcast.

The build_vector needs to be the only user of the data, but the
chain will likely have another use. So we can't make sure the
build_vector is the only user of the node.


  Commit: d8ad7cc0885f32104a7cd83c77191aec15fd684f
      https://github.com/llvm/llvm-project/commit/d8ad7cc0885f32104a7cd83c77191aec15fd684f
  Author: Craig Topper <craig.topper at gmail.com>
  Date:   2020-03-03 (Tue, 03 Mar 2020)

  Changed paths:
    M llvm/lib/CodeGen/SelectionDAG/DAGCombiner.cpp
    M llvm/test/CodeGen/X86/avx512-extract-subvector-load-store.ll
    M llvm/test/CodeGen/X86/avx512-mask-op.ll
    M llvm/test/CodeGen/X86/avx512bw-intrinsics-upgrade.ll

  Log Message:
  -----------
  [DAGCombiner][X86] Improve narrowExtractedVectorLoad to handle cases where the element size isn't byte sized by the subvector is.

Summary:
Follow up from D75377. If the subvector is byte sized and the
index is aligned to the subvector size, we can shrink the load.

Reviewers: spatel, RKSimon

Reviewed By: RKSimon

Subscribers: dbabokin, hiraditya, llvm-commits

Tags: #llvm

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


Compare: https://github.com/llvm/llvm-project/compare/383e3ec1b2ad...d8ad7cc0885f