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

[Matt Arsenault via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 47d831f2c90225a7d267ec5e3149d5584287dbbc
      https://github.com/llvm/llvm-project/commit/47d831f2c90225a7d267ec5e3149d5584287dbbc
  Author: Matt Arsenault <Matthew.Arsenault at amd.com>
  Date:   2024-08-08 (Thu, 08 Aug 2024)

  Changed paths:
    M llvm/include/llvm/CodeGen/BasicTTIImpl.h
    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 (#100514)

Instead of counting the cost of the assumed expansion.

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

Preserve default expansion logic



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