[all-commits] [llvm/llvm-project] c90819: [CostModel] Return Invalid cost in getArithmeticCo...

[sdesmalen-arm via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: c908196e100a5ac1b25cba859094bfc28cffb5f3
      https://github.com/llvm/llvm-project/commit/c908196e100a5ac1b25cba859094bfc28cffb5f3
  Author: Sander de Smalen <sander.desmalen at arm.com>
  Date:   2021-06-07 (Mon, 07 Jun 2021)

  Changed paths:
    M llvm/include/llvm/CodeGen/BasicTTIImpl.h
    A llvm/test/Analysis/CostModel/AArch64/sve-remainder.ll

  Log Message:
  -----------
  [CostModel] Return Invalid cost in getArithmeticCost instead of crashing for scalable vectors.

This fixes an issue in BasicTTIImpl.h where it tries to do a
cast<FixedVectorType> on a scalable vector type in order to get the
scalarization cost. Because scalarization of scalable vectors is not
supported, we return Invalid instead.

Reviewed By: RKSimon

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


  Commit: be663c4337c6e3d81a9a539abc11477fdb74cc6e
      https://github.com/llvm/llvm-project/commit/be663c4337c6e3d81a9a539abc11477fdb74cc6e
  Author: Sander de Smalen <sander.desmalen at arm.com>
  Date:   2021-06-07 (Mon, 07 Jun 2021)

  Changed paths:
    R llvm/test/Analysis/CostModel/AArch64/scalable-mem-op-cost-model.ll
    M llvm/test/Analysis/CostModel/AArch64/sve-gather.ll
    A llvm/test/Analysis/CostModel/AArch64/sve-ldst.ll
    M llvm/test/Analysis/CostModel/AArch64/sve-scatter.ll

  Log Message:
  -----------
  [CostModel][AArch64] NFC: Simplify some cost model tests for SVE.

* Merged some functions into a single function, to make the costs more obvious.
* Moved scalable-mem-op-cost-model.ll -> sve-ldst.ll to be more consistent with other filenames.


Compare: https://github.com/llvm/llvm-project/compare/619a65e5e4ed...be663c4337c6