[all-commits] [llvm/llvm-project] 0a32e6: [DAGCombiner][NFC] add testcases for D138899

Chen Zheng via All-commits all-commits at lists.llvm.org
Wed Feb 1 18:06:33 PST 2023


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 0a32e693e31f1a7d6de2701655a4555555f8d4f9
      https://github.com/llvm/llvm-project/commit/0a32e693e31f1a7d6de2701655a4555555f8d4f9
  Author: Chen Zheng <czhengsz at cn.ibm.com>
  Date:   2023-02-01 (Wed, 01 Feb 2023)

  Changed paths:
    A llvm/test/CodeGen/PowerPC/byval-lhs.ll

  Log Message:
  -----------
  [DAGCombiner][NFC] add testcases for D138899


  Commit: f35a09daebd0a90daa536432e62a2476f708150d
      https://github.com/llvm/llvm-project/commit/f35a09daebd0a90daa536432e62a2476f708150d
  Author: Chen Zheng <czhengsz at cn.ibm.com>
  Date:   2023-02-01 (Wed, 01 Feb 2023)

  Changed paths:
    M llvm/lib/CodeGen/SelectionDAG/DAGCombiner.cpp
    M llvm/test/CodeGen/AMDGPU/ctpop16.ll
    M llvm/test/CodeGen/AMDGPU/load-constant-i16.ll
    M llvm/test/CodeGen/AMDGPU/load-global-i16.ll
    M llvm/test/CodeGen/AMDGPU/shl.ll
    M llvm/test/CodeGen/AMDGPU/sra.ll
    M llvm/test/CodeGen/Mips/o32_cc_byval.ll
    M llvm/test/CodeGen/PowerPC/aix-cc-byval.ll
    M llvm/test/CodeGen/PowerPC/byval-lhs.ll
    M llvm/test/CodeGen/PowerPC/ppc64-byval-larger-struct.ll
    M llvm/test/CodeGen/X86/fastcc-byval.ll

  Log Message:
  -----------
  [DAGCombiner] handle more store value forwarding

When lowering calls on target like PPC, some stack loads
will be generated for by value parameters. Node CALLSEQ_START
prevents such loads from being combined.

Suggested by @RolandF, this patch removes the unnecessary
loads for the byval parameter by extending ForwardStoreValueToDirectLoad

Reviewed By: nemanjai, RolandF

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


Compare: https://github.com/llvm/llvm-project/compare/d42c2352aa3a...f35a09daebd0


More information about the All-commits mailing list