[PATCH] D129298: Add denormal-fp-math attribute for f16

2022-11-02 Thread David Candler via Phabricator via cfe-commits
dcandler abandoned this revision. dcandler added a comment. Sorry for the quiet on this. I'm going to abandon this for the moment, as what I eventually found was that there was some ambiguity in the ARM ABI regarding half-floats which would be better to address first, so that the attributes can

[PATCH] D129298: Add denormal-fp-math attribute for f16

2022-09-28 Thread Matt Arsenault via Phabricator via cfe-commits
arsenm added a comment. Reverse ping Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D129298/new/ https://reviews.llvm.org/D129298 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.

[PATCH] D129298: Add denormal-fp-math attribute for f16

2022-07-11 Thread Matt Arsenault via Phabricator via cfe-commits
arsenm added a comment. In D129298#3638759 , @dcandler wrote: > There are currently no Arm specific changes, this is just being able to more > accurately describe the floating point environment via attributes in the case > where singles and doubles shou

[PATCH] D129298: Add denormal-fp-math attribute for f16

2022-07-08 Thread David Candler via Phabricator via cfe-commits
dcandler added a comment. There are currently no Arm specific changes, this is just being able to more accurately describe the floating point environment via attributes in the case where singles and doubles should be flushed, but not halves. With three precisions to control, an alternative may

[PATCH] D129298: Add denormal-fp-math attribute for f16

2022-07-07 Thread Matt Arsenault via Phabricator via cfe-commits
arsenm added a comment. Missing ARM changes that demonstrate the use of the control? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D129298/new/ https://reviews.llvm.org/D129298 ___ cfe-commits mailing lis

[PATCH] D129298: Add denormal-fp-math attribute for f16

2022-07-07 Thread David Candler via Phabricator via cfe-commits
dcandler created this revision. dcandler added reviewers: arsenm, spatel, echristo, andrew.w.kaylor, cameron.mcinally. Herald added subscribers: jsji, kosarev, jdoerfert, pengfei, hiraditya, kristof.beyls, tpr. Herald added a project: All. dcandler requested review of this revision. Herald added