[PATCH] D100834: Bug 49739 - [Matrix] Support #pragma clang fp

Florian Hahn via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Apr 21 01:21:56 PDT 2021


fhahn added a comment.

In D100834#2702550 <https://reviews.llvm.org/D100834#2702550>, @kpn wrote:

> I don't know the matrix implementation so I can't swear this hits every place needed, but the uses of CGFPOptionsRAII in this patch look correct at least.

Other parts of the extension include `__builtin_matrix_transpose`, indexing into a matrix and casting, but I don't think the FMFs are needed there. One thing that would be good to also test would be the compound operators, (`-=`, `+=`, `*=`). @effective-light it would be great if you could add a test for those, then LGTM from my side. If you need someone to commit the change on your behalf, please let us know and share the name + email to use for the commit authorship.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D100834



More information about the cfe-commits mailing list