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