[all-commits] [llvm/llvm-project] 1b4c37: [TTI][X86] getGSVectorCost/getGSScalarCost - add C...

[Simon Pilgrim via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 1b4c37fec236f39a1ec530ecf07b6aa162518a6b
      https://github.com/llvm/llvm-project/commit/1b4c37fec236f39a1ec530ecf07b6aa162518a6b
  Author: Simon Pilgrim <llvm-dev at redking.me.uk>
  Date:   2024-04-05 (Fri, 05 Apr 2024)

  Changed paths:
    M llvm/lib/Target/X86/X86TargetTransformInfo.cpp
    M llvm/lib/Target/X86/X86TargetTransformInfo.h

  Log Message:
  -----------
  [TTI][X86] getGSVectorCost/getGSScalarCost - add CostKind to the function arguments.

Initial refactor - only getGSScalarCost can actually use CostKind so far, and currently both are only ever set to TCK_RecipThroughput.


  Commit: 58187fad93bca76d1cce43a5caa70728282d2ff8
      https://github.com/llvm/llvm-project/commit/58187fad93bca76d1cce43a5caa70728282d2ff8
  Author: Simon Pilgrim <llvm-dev at redking.me.uk>
  Date:   2024-04-05 (Fri, 05 Apr 2024)

  Changed paths:
    M llvm/test/Analysis/CostModel/X86/masked-intrinsic-codesize.ll
    M llvm/test/Analysis/CostModel/X86/masked-intrinsic-cost-inseltpoison.ll
    M llvm/test/Analysis/CostModel/X86/masked-intrinsic-cost.ll
    M llvm/test/Analysis/CostModel/X86/masked-intrinsic-latency.ll
    M llvm/test/Analysis/CostModel/X86/masked-intrinsic-sizelatency.ll

  Log Message:
  -----------
  [CostModel][X86] Update masked load/store/gather/scatter tests to explicitly use variable masks

Using <X x i1> undef masks means they are treated as constants, which underestimates the scalar costs as it assumes that the masks/branches will fold away.


Compare: https://github.com/llvm/llvm-project/compare/9a16c12abe13...58187fad93bc

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