[PATCH] D69989: Assume ieee behavior without denormal-fp-math attribute

Cameron McInally via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Nov 20 09:08:09 PST 2019


cameron.mcinally added subscribers: craig.topper, eli.friedman.
cameron.mcinally added a comment.

Attn: @craig.topper @eli.friedman. The FMF behavior looks like it will take a performance hit with the switch to default IEEE conformance. Is this something that needs to be publicized on llvm-dev before merging? Users who don't care will want to add an explicit `denormal-fp-math` attribute to their code.

Nit: the IR tests may be overly brittle -- checking for specific registers. It might be good to use `update_test_checks.py` so that we get the regex'd register expressions.


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

https://reviews.llvm.org/D69989





More information about the llvm-commits mailing list