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

Florian Hahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Mar 26 10:18:09 PDT 2020


fhahn added a comment.

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

> The attribute seems like overkill as a performance optimization.  You can get 99% of the way there by just checking if if the module declares any matrix intrinsics.


Hmm, is there an easy way to check that without checking all the declarations in a module? Or would we have to check all declarations for all functions in a module?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D76327





More information about the llvm-commits mailing list