arsenm added a comment.

In D69979#1738723 <https://reviews.llvm.org/D69979#1738723>, @spatel wrote:

> Also, I may have missed some discussions. Does this patch series replace the 
> proposal to add instruction-level FMF for denorms?
>  http://lists.llvm.org/pipermail/llvm-dev/2019-September/135183.html
>
> Ie, did we decide that a function-level attribute is good enough?


I think this is an orthogonal question. I would still find a ftz flag useful 
even in the presence of this attribute indicating flushing. For AMDGPU it would 
be useful with a specific instruction context to allow flushing even when the 
default mode is set to not flush. For example llvm.fmuladd could be emitted 
with an ftz flag which would select to an instruction that would ordinarily be 
illegal if denormals are enabled


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

https://reviews.llvm.org/D69979



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to