Re: [PATCH] D24069: Document option '-rtlib' in clang's man page and help info

2016-09-14 Thread Jonas Hahnfeld via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL281440: Document option '-rtlib' in clang's man page and help info (authored by Hahnfeld). Changed prior to commit: https://reviews.llvm.org/D24069?vs=69947=71298#toc Repository: rL LLVM

Re: [PATCH] D24069: Document option '-rtlib' in clang's man page and help info

2016-09-13 Thread Lei Zhang via cfe-commits
zlei added a comment. In https://reviews.llvm.org/D24069#540804, @Hahnfeld wrote: > @zlei will you commit the patch or do you want me to do that on your behalf? I don't have commit access, so please do me a favor :) https://reviews.llvm.org/D24069

Re: [PATCH] D24069: Document option '-rtlib' in clang's man page and help info

2016-09-13 Thread Jonas Hahnfeld via cfe-commits
Hahnfeld added a comment. @zlei will you commit the patch or do you want me to do that on your behalf? https://reviews.llvm.org/D24069 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

Re: [PATCH] D24069: Document option '-rtlib' in clang's man page and help info

2016-09-12 Thread Jonas Hahnfeld via cfe-commits
Hahnfeld accepted this revision. Hahnfeld added a comment. This revision is now accepted and ready to land. LGTM, thanks for the improvement! https://reviews.llvm.org/D24069 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

Re: [PATCH] D24069: Document option '-rtlib' in clang's man page and help info

2016-09-01 Thread Lei Zhang via cfe-commits
zlei edited reviewers, added: bruno; removed: cfe-commits. zlei edited subscribers, added: cfe-commits; removed: bruno. zlei updated this revision to Diff 69947. zlei added a comment. Update the man page entry to state default behavior of this option. https://reviews.llvm.org/D24069 Files:

Re: [PATCH] D24069: Document option '-rtlib' in clang's man page and help info

2016-08-31 Thread Bruno Cardoso Lopes via cfe-commits
bruno added inline comments. Comment at: docs/CommandGuide/clang.rst:114 @@ -110,1 +113,3 @@ + compiler-rt. + .. option:: -ansi zlei wrote: > bruno wrote: > > It might be worth mentioning what's the default behaviour in case the flag > > isn't specified? > This

Re: [PATCH] D24069: Document option '-rtlib' in clang's man page and help info

2016-08-31 Thread Bruno Cardoso Lopes via cfe-commits
bruno added a subscriber: bruno. bruno added a comment. Hi Lei, Thanks for the doc improvement! Comment at: docs/CommandGuide/clang.rst:114 @@ -110,1 +113,3 @@ + compiler-rt. + .. option:: -ansi It might be worth mentioning what's the default behaviour in

[PATCH] D24069: Document option '-rtlib' in clang's man page and help info

2016-08-31 Thread Lei Zhang via cfe-commits
zlei created this revision. zlei added reviewers: cfe-commits, Hahnfeld. This patch adds an entry for "-rtlib" in the output of `man clang` and `clang -help`. https://reviews.llvm.org/D24069 Files: docs/CommandGuide/clang.rst include/clang/Driver/Options.td Index: