[PATCH] D69989: Assume ieee behavior without denormal-fp-math attribute
Sanjay Patel via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Feb 14 11:35:18 PST 2020
spatel added inline comments.
================
Comment at: llvm/test/CodeGen/X86/sqrt-fastmath-tune.ll:101
attributes #1 = { nounwind readnone }
+attributes #2 = { "denormal-fp-math"="ieee,preseve-sign" "unsafe-fp-math"="true" }
----------------
arsenm wrote:
> spatel wrote:
> > Does the "preseve" typo affect anything?
> These aren't hitting the expansion, so I guess it doesn't. I assume this has something to do with not using reciprocal-estimates?
Yes, this file isn't well-specified about what it's trying to test for. I updated it here:
rG6071fc57a45f
That should remove it from this patch?
Independent of that - should we have an assert or warning that would catch typos in the attribute string?
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D69989/new/
https://reviews.llvm.org/D69989
More information about the llvm-commits
mailing list