[all-commits] [llvm/llvm-project] fc7e74: [CostModel][X86] getCastInstrCost - improve CostKi...

Simon Pilgrim via All-commits all-commits at lists.llvm.org
Fri May 3 04:17:55 PDT 2024


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

  Changed paths:
    M llvm/lib/Target/X86/X86TargetTransformInfo.cpp
    M llvm/test/Analysis/CostModel/X86/intrinsic-cost-kinds.ll
    M llvm/test/Analysis/CostModel/X86/masked-intrinsic-codesize.ll
    M llvm/test/Analysis/CostModel/X86/masked-intrinsic-latency.ll
    M llvm/test/Analysis/CostModel/X86/masked-intrinsic-sizelatency.ll
    M llvm/test/Analysis/CostModel/X86/shuffle-replication-i1-codesize.ll
    M llvm/test/Analysis/CostModel/X86/shuffle-replication-i1-latency.ll
    M llvm/test/Analysis/CostModel/X86/shuffle-replication-i1-sizelatency.ll
    M llvm/test/Analysis/CostModel/X86/shuffle-replication-i16-codesize.ll
    M llvm/test/Analysis/CostModel/X86/shuffle-replication-i16-latency.ll
    M llvm/test/Analysis/CostModel/X86/shuffle-replication-i16-sizelatency.ll
    M llvm/test/Analysis/CostModel/X86/shuffle-replication-i8-codesize.ll
    M llvm/test/Analysis/CostModel/X86/shuffle-replication-i8-latency.ll
    M llvm/test/Analysis/CostModel/X86/shuffle-replication-i8-sizelatency.ll
    M llvm/test/Analysis/CostModel/X86/trunc-codesize.ll
    M llvm/test/Analysis/CostModel/X86/trunc-latency.ll
    M llvm/test/Analysis/CostModel/X86/trunc-sizelatency.ll

  Log Message:
  -----------
  [CostModel][X86] getCastInstrCost - improve CostKind adjustment when splitting src/dst types

Noticed in #90883 review - for non-Throughput costs, we weren't applying the split count to '0 or 1' cost value.

This still doesn't work well as many of the type legalizations are hidden so we don't have the split count, really we need to move a CostKindCosts based costs table, but that's going to be a lot of work :/



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