[PATCH] D71353: Fix denormal-fp-math flag and attribute interaction
Matt Arsenault via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Mar 9 19:58:10 PDT 2020
arsenm added a comment.
In D71353#1913550 <https://reviews.llvm.org/D71353#1913550>, @echristo wrote:
> This is wanting a command line flag rather than setting it on the function, or rather in addition to being able to set it on the function? Also "this" in the commit message isn't quite clear. Having this be a separate set of both global function attributes and different than the other fp-math attributes means it really should be called out fairly explicitly why they need to be different, what the semantics are, etc.
This makes the denormal handling consistent with the fp-math attributes. We previously had inconsistent behaviors, but now the flags should never override an existing attribute
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D71353/new/
https://reviews.llvm.org/D71353
More information about the llvm-commits
mailing list