Re: [PATCH] D20457: Update -ffast-math documentation to match reality.

2016-05-20 Thread Justin Lebar via cfe-commits
jlebar marked 2 inline comments as done. jlebar added a comment. Repository: rL LLVM http://reviews.llvm.org/D20457 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: [PATCH] D20457: Update -ffast-math documentation to match reality.

2016-05-20 Thread Justin Lebar via cfe-commits
jlebar added a comment. Thank you for the review! http://reviews.llvm.org/D20457 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: [PATCH] D20457: Update -ffast-math documentation to match reality.

2016-05-20 Thread Justin Lebar via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL270279: Update -ffast-math documentation to match reality. (authored by jlebar). Changed prior to commit: http://reviews.llvm.org/D20457?vs=57958=57995#toc Repository: rL LLVM

Re: [PATCH] D20457: Update -ffast-math documentation to match reality.

2016-05-20 Thread Richard Smith via cfe-commits
rsmith accepted this revision. This revision is now accepted and ready to land. Comment at: docs/UsersManual.rst:1067 @@ +1066,3 @@ + ``(a + b) * c == a * c + b * c``), + * operands to fp operations are not equal to ``NaN`` and ``Inf``, and + * ``+0`` and ``-0`` are

Re: [PATCH] D20457: Update -ffast-math documentation to match reality.

2016-05-20 Thread Justin Lebar via cfe-commits
jlebar updated this revision to Diff 57958. jlebar added a comment. Update per Richard's review. http://reviews.llvm.org/D20457 Files: docs/UsersManual.rst include/clang/Basic/LangOptions.def include/clang/Driver/Options.td Index: include/clang/Driver/Options.td

Re: [PATCH] D20457: Update -ffast-math documentation to match reality.

2016-05-19 Thread Richard Smith via cfe-commits
On Thu, May 19, 2016 at 6:11 PM, Justin Lebar via cfe-commits < cfe-commits@lists.llvm.org> wrote: > jlebar created this revision. > jlebar added a reviewer: rsmith. > jlebar added a subscriber: cfe-commits. > > http://reviews.llvm.org/D20457 > > Files: > include/clang/Basic/LangOptions.def >

[PATCH] D20457: Update -ffast-math documentation to match reality.

2016-05-19 Thread Justin Lebar via cfe-commits
jlebar created this revision. jlebar added a reviewer: rsmith. jlebar added a subscriber: cfe-commits. http://reviews.llvm.org/D20457 Files: include/clang/Basic/LangOptions.def include/clang/Driver/Options.td Index: include/clang/Driver/Options.td