[all-commits] [llvm/llvm-project] a5f2fd: [X86][Costmodel] `trunc v16i32 to v32i16` can appe...

[Roman Lebedev via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: a5f2fdca993eca0ae85e90f987fbbfb3b4aee626
      https://github.com/llvm/llvm-project/commit/a5f2fdca993eca0ae85e90f987fbbfb3b4aee626
  Author: Roman Lebedev <lebedev.ri at gmail.com>
  Date:   2021-11-14 (Sun, 14 Nov 2021)

  Changed paths:
    M llvm/lib/Target/X86/X86TargetTransformInfo.cpp
    M llvm/test/Analysis/CostModel/X86/arith-fix.ll
    M llvm/test/Analysis/CostModel/X86/arith-overflow.ll
    M llvm/test/Analysis/CostModel/X86/trunc.ll

  Log Message:
  -----------
  [X86][Costmodel] `trunc v16i32 to v32i16` can appear after legalization, cost is same as for `trunc v16i32 to v16i16`

This was noticed in D113609, hopefully it unblocks that patch.
There are likely other similar problems.

Reviewed By: RKSimon

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


  Commit: b2839610124bd8a3a881a65ab14005c9405479ef
      https://github.com/llvm/llvm-project/commit/b2839610124bd8a3a881a65ab14005c9405479ef
  Author: Roman Lebedev <lebedev.ri at gmail.com>
  Date:   2021-11-14 (Sun, 14 Nov 2021)

  Changed paths:
    M llvm/lib/Target/X86/X86TargetTransformInfo.cpp
    M llvm/test/Analysis/CostModel/X86/trunc.ll

  Log Message:
  -----------
  [X86][Costmodel] `trunc v8i64 to v16i16/v32i16` can appear after legalization, cost is same as for `trunc v8i64 to v8i16`

Same as D113842, but for i64

Reviewed By: RKSimon

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


Compare: https://github.com/llvm/llvm-project/compare/0662a3612cea...b2839610124b