[PATCH] D67839: [FPEnv] Document requirement of function attributes with constrained floating point
Ulrich Weigand via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Sep 24 11:20:03 PDT 2019
uweigand added a comment.
I agree with marking function call sites as "strictfp".
But I'm wondering how you came up with this particular list for function definitions? In particular, I'm not sure whether noimplicitfloat is really needed. I understand this flag is supposed to prevent unintended use of floating-point registers (e.g. for Linux kernel code). You don't really need to prevent that for constrained floating point semantics, I think. (You do need to prevent introduction of floating-point instruction that might cause spurious exception, but e.g. introducing moves through FP registers should still be fine.)
As to "strictfp" at the function definition, does this actually do anything currently? I cannot find any LLVM code that checks for this. The documentation also doesn't really say.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D67839/new/
https://reviews.llvm.org/D67839
More information about the llvm-commits
mailing list