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

[Matt Arsenault via All-commits]

  Branch: refs/heads/users/arsenm/min-max-intrinsic-cost-model
  Home:   https://github.com/llvm/llvm-project
  Commit: 810200d58e7d9ebaa282aa050afe1b4e9cec9595
      https://github.com/llvm/llvm-project/commit/810200d58e7d9ebaa282aa050afe1b4e9cec9595
  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: fabb2a99a36cb8418ab4c32d7face42fe5611578
      https://github.com/llvm/llvm-project/commit/fabb2a99a36cb8418ab4c32d7face42fe5611578
  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/810200d58e7d%5E...fabb2a99a36c

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