rengolin added reviewers: rsmith, t.p.northover.
rengolin added a comment.

Adding Richard Smith, as he was the one choosing C99/!freestanding for runtime 
rounding.

================
Comment at: lib/Basic/Targets.cpp:4790
@@ -4779,1 +4789,3 @@
+    if (!Opts.C11)
+      Builder.defineMacro("__SUPPORT_SNAN__", "1");
   }
----------------
labrinea wrote:
> rengolin wrote:
> > Er, well, the ACLE doc says it wasn't adopted in C11, but that doesn't mean 
> > that it was present in all others, just that there is a proposal for C11 
> > which wasn't accepted.
> > 
> > I think this macro needs some other mechanism to be set, and unless 
> > Clang/LLVM supports signaling NANs unconditionally, we shouldn't add it.
> Maybe we could skip this macro in this patch? @richard.barton.arm
If this item sparks controversy, we should keep it out. For now, I'm interested 
in what Tim and Richard(s) have to say.


http://reviews.llvm.org/D12633



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

Reply via email to