[all-commits] [llvm/llvm-project] ab05ab: [CostModel][AMDGPU] Fix instructions costs estimat...

Daniil Fukalov via All-commits all-commits at lists.llvm.org
Thu Dec 2 16:08:30 PST 2021


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: ab05ab59a7bd22889fe19389431676b3414c6b7d
      https://github.com/llvm/llvm-project/commit/ab05ab59a7bd22889fe19389431676b3414c6b7d
  Author: Daniil Fukalov <daniil.fukalov at amd.com>
  Date:   2021-12-03 (Fri, 03 Dec 2021)

  Changed paths:
    M llvm/lib/Target/AMDGPU/AMDGPUTargetTransformInfo.cpp
    M llvm/lib/Target/AMDGPU/SIISelLowering.cpp
    M llvm/test/Analysis/CostModel/AMDGPU/add-sub.ll
    M llvm/test/Analysis/CostModel/AMDGPU/fabs.ll
    M llvm/test/Analysis/CostModel/AMDGPU/fadd.ll
    M llvm/test/Analysis/CostModel/AMDGPU/fdiv.ll
    M llvm/test/Analysis/CostModel/AMDGPU/fmul.ll
    M llvm/test/Analysis/CostModel/AMDGPU/fneg.ll
    M llvm/test/Analysis/CostModel/AMDGPU/fsub.ll
    M llvm/test/Analysis/CostModel/AMDGPU/mul.ll
    M llvm/test/Analysis/CostModel/AMDGPU/shifts.ll

  Log Message:
  -----------
  [CostModel][AMDGPU] Fix instructions costs estimation for vector types.

1. Fixed vector instructions costs estimations incosistency - removed different
   logic for "not simple types" since it biases costs for these types.
2. Fixed legalization penalty for vectors too big for the target: changed from
   overwrite default legalization cost value estimation to added penalty.
3. Fixed few typos in tests.

Reviewed By: rampitec

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




More information about the All-commits mailing list