[PATCH] D69978: Separately track input and output denormal mode

2020-02-04 Thread Matt Arsenault via Phabricator via cfe-commits
arsenm closed this revision. arsenm added a comment. a3c814d23497bc71b8ed53c35f773366aff02922 CHANGES SINCE LAST ACTION https://reviews.llvm.org/D69978/new/ https://reviews.llvm.org/D69978 __

[PATCH] D69978: Separately track input and output denormal mode

2020-01-31 Thread Andy Kaylor via Phabricator via cfe-commits
andrew.w.kaylor accepted this revision. andrew.w.kaylor added a comment. This revision is now accepted and ready to land. lgtm CHANGES SINCE LAST ACTION https://reviews.llvm.org/D69978/new/ https://reviews.llvm.org/D69978 ___ cfe-commits mailing

[PATCH] D69978: Separately track input and output denormal mode

2020-01-31 Thread Matt Arsenault via Phabricator via cfe-commits
arsenm updated this revision to Diff 241714. arsenm marked an inline comment as done. arsenm added a comment. Tweak langref CHANGES SINCE LAST ACTION https://reviews.llvm.org/D69978/new/ https://reviews.llvm.org/D69978 Files: clang/include/clang/Basic/CodeGenOptions.h clang/include/clang

[PATCH] D69978: Separately track input and output denormal mode

2020-01-31 Thread Matt Arsenault via Phabricator via cfe-commits
arsenm marked 3 inline comments as done. arsenm added inline comments. Comment at: llvm/docs/LangRef.rst:1829 + operations. The second indicates the handling of denormal inputs to + floating point instructions. + andrew.w.kaylor wrote: > Based on the changes

[PATCH] D69978: Separately track input and output denormal mode

2020-01-29 Thread Andy Kaylor via Phabricator via cfe-commits
andrew.w.kaylor added inline comments. Comment at: llvm/docs/LangRef.rst:1829 + operations. The second indicates the handling of denormal inputs to + floating point instructions. + Based on the changes below, if the second value is omitted the input mode will

[PATCH] D69978: Separately track input and output denormal mode

2020-01-20 Thread Matt Arsenault via Phabricator via cfe-commits
arsenm updated this revision to Diff 239196. arsenm added a comment. Herald added a subscriber: kerbowa. Rebase CHANGES SINCE LAST ACTION https://reviews.llvm.org/D69978/new/ https://reviews.llvm.org/D69978 Files: clang/include/clang/Basic/CodeGenOptions.h clang/include/clang/Driver/Tool

[PATCH] D69978: Separately track input and output denormal mode

2019-11-18 Thread Matt Arsenault via Phabricator via cfe-commits
arsenm marked an inline comment as done. arsenm added inline comments. Comment at: llvm/docs/LangRef.rst:1822 ``"denorm-fp-mode"`` + This indicates the subnormal handling that may be assumed for the andrew.w.kaylor wrote: > I don't like the definition of thi

[PATCH] D69978: Separately track input and output denormal mode

2019-11-12 Thread Andy Kaylor via Phabricator via cfe-commits
andrew.w.kaylor added inline comments. Comment at: llvm/docs/LangRef.rst:1822 ``"denorm-fp-mode"`` + This indicates the subnormal handling that may be assumed for the I don't like the definition of this attribute. It's not reader-friendly. The comma-separat

[PATCH] D69978: Separately track input and output denormal mode

2019-11-07 Thread Matt Arsenault via Phabricator via cfe-commits
arsenm created this revision. arsenm added reviewers: scanon, cameron.mcinally, spatel, andrew.w.kaylor, mibintc, SjoerdMeijer. Herald added subscribers: dexonsmith, hiraditya, tpr, nhaehnle, wdng, jvesely, jholewinski. Herald added a project: LLVM. arsenm added parent revisions: D69878: Consoldi