[PATCH] D101759: [PowerPC] Scalar IBM MASS library conversion pass
Sanjay Patel via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Fri Aug 27 07:18:10 PDT 2021
spatel added a reviewer: efriedma.
spatel added a comment.
I'm not familiar with this library, and I haven't looked at current state of how we enable/map optional libs in a while...
We definitely want to avoid adding another target option/debug flag, and if we can avoid relying on a function parameter too, that would be even better.
Ie, the "afn" fast-math-flag (possibly in combination with some other IR- or node-level flags) seems like it should be enough to allow this transform/lowering.
Scanning the earlier review comments, there was some concern about the semantics wrt errno. If we need to adjust the "afn" definition, it's probably fine. There haven't been many uses of that flag AFAIK.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D101759/new/
https://reviews.llvm.org/D101759
More information about the cfe-commits
mailing list