[all-commits] [llvm/llvm-project] b688c9: TTI: Check legalization cost of min/max ISD nodes

Matt Arsenault via All-commits all-commits at lists.llvm.org
Thu Jul 25 13:54:12 PDT 2024


  Branch: refs/heads/users/arsenm/min-max-intrinsic-cost-model
  Home:   https://github.com/llvm/llvm-project
  Commit: b688c91479416578d95b99a3c41cf776a4f76c68
      https://github.com/llvm/llvm-project/commit/b688c91479416578d95b99a3c41cf776a4f76c68
  Author: Matt Arsenault <Matthew.Arsenault at amd.com>
  Date:   2024-07-25 (Thu, 25 Jul 2024)

  Changed paths:
    M llvm/include/llvm/CodeGen/BasicTTIImpl.h
    M llvm/test/Analysis/CostModel/AArch64/min-max.ll
    M llvm/test/Analysis/CostModel/AMDGPU/arith-sminmax.ll
    M llvm/test/Analysis/CostModel/AMDGPU/arith-uminmax.ll
    M llvm/test/Analysis/CostModel/ARM/cmps.ll
    M llvm/test/Analysis/CostModel/ARM/reduce-smax.ll
    M llvm/test/Analysis/CostModel/ARM/reduce-smin.ll
    M llvm/test/Analysis/CostModel/ARM/reduce-umax.ll
    M llvm/test/Analysis/CostModel/ARM/reduce-umin.ll
    M llvm/test/Transforms/SLPVectorizer/AMDGPU/min_max.ll

  Log Message:
  -----------
  TTI: Check legalization cost of min/max ISD nodes

Instead of counting the cost of the assumed expansion.

The AMDGPU costs for the i64 case look too high to me.


  Commit: 564757e1ad0b813cb0716c1563de08ff6218cb71
      https://github.com/llvm/llvm-project/commit/564757e1ad0b813cb0716c1563de08ff6218cb71
  Author: Matt Arsenault <Matthew.Arsenault at amd.com>
  Date:   2024-07-25 (Thu, 25 Jul 2024)

  Changed paths:
    M llvm/include/llvm/CodeGen/BasicTTIImpl.h
    M llvm/test/Analysis/CostModel/AArch64/min-max.ll
    M llvm/test/Analysis/CostModel/AMDGPU/arith-sminmax.ll
    M llvm/test/Analysis/CostModel/AMDGPU/arith-uminmax.ll
    M llvm/test/Analysis/CostModel/ARM/cmps.ll
    M llvm/test/Analysis/CostModel/ARM/reduce-smax.ll
    M llvm/test/Analysis/CostModel/ARM/reduce-smin.ll
    M llvm/test/Analysis/CostModel/ARM/reduce-umax.ll
    M llvm/test/Analysis/CostModel/ARM/reduce-umin.ll

  Log Message:
  -----------
  Preserve default expansion logic


Compare: https://github.com/llvm/llvm-project/compare/5fa2f2d528bc...564757e1ad0b

To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications


More information about the All-commits mailing list