[PATCH] D76327: [Matrix] Also run LowerMatrixIntrinsics for -O0.

[Florian Hahn via Phabricator via llvm-commits]

fhahn added a comment.

In D76327#1944354 <https://reviews.llvm.org/D76327#1944354>, @efriedma wrote:

> I assume the matrix intrinsics overloaded?  In that case, you'd have to iterate over the module, yes.


Unfortunately yes :(

> I'm tempted to say we should add a new mechanism to the backend for passes that only care about calls to a specific set of intrinsics, so we don't have to reinvent this repeatedly.

We would also need it in the middle-end for the matrix lowering (and potentially others as well).

Do you have anything in particular in mind? For intrinsics that get solely emitted by the frontend, I guess it would be possible check if the module contains any such instructions and store it and have something like Module::containsMatrixIntrinsics(). Not sure if that's acceptable though.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D76327