[PATCH] D95291: [CostModel] Remove VF from IntrinsicCostAttributes

[Jinsong Ji via Phabricator via llvm-commits]

jsji added a comment.

Looks like this is causing an assert in PowerPC test-suite if built with CMake. 
The current PowerPC buildbots are using Make, so did not trigger this (no `-fenable-matrix` tested), we will update that soon.

@dmgreen  Can you have a quick look? Thanks.

Reduced source and command line to reproduce.

`.../build/bin/clang -cc1 -fenable-matrix -triple=powerpc64le-unknown-linux-gnu -O3 -emit-llvm matrix-types-spec-f4acb3.reduced.cpp`

  $ cat matrix-types-spec-f4acb3.reduced.cpp
  template <typename a, int b, int c, int d> void e() {
    a f, g, h;
    using i = a __attribute__((matrix_type(b, d)));
    auto j = __builtin_matrix_column_major_load(&g, b, c, b);
    auto k = __builtin_matrix_column_major_load(&f, c, d, c);
    i l = j * k;
    __builtin_matrix_column_major_store(l, &h, b);
  }
  template <typename, int b, int c, int> void m() { e<int, b, c, 1>(); }
  int main() { m<double, 10, 1, 3>(); }


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D95291/new/

https://reviews.llvm.org/D95291