[PATCH] D69878: Consoldiate internal denormal flushing controls

[Anastasia Stulova via Phabricator via cfe-commits]

Anastasia added inline comments.


================
Comment at: clang/lib/CodeGen/CGCall.cpp:1775
-    if (getLangOpts().OpenCL)
-      FuncAttrs.addAttribute("denorms-are-zero",
-                             llvm::toStringRef(CodeGenOpts.FlushDenorm));
----------------
arsenm wrote:
> Anastasia wrote:
> > so where would `denorms-are-zero` be emitted now (in case some out of tree implementations rely on this)?
> Rely on in what sense? Do you have a concrete use of this?
Since it has been emitted before in the module potentially some LLVM implementations could be using that attribute?


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

https://reviews.llvm.org/D69878