Re: [PATCH] D15350: [clang-cl] Let /W4 map to -Wall -Wextra instead of just -Wall.

2015-12-11 Thread Nico Weber via cfe-commits
thakis closed this revision. thakis added a comment. r255382, thanks! http://reviews.llvm.org/D15350 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: [PATCH] D15350: [clang-cl] Let /W4 map to -Wall -Wextra instead of just -Wall.

2015-12-08 Thread Hans Wennborg via cfe-commits
hans accepted this revision. hans added a comment. This revision is now accepted and ready to land. > There's no way to make a flag alias to two flags Well, at least not with the alias mechanism in the option parser, but David pointed out we could expand it manually in MSVCToolChain::TranslateA

[PATCH] D15350: [clang-cl] Let /W4 map to -Wall -Wextra instead of just -Wall.

2015-12-08 Thread Nico Weber via cfe-commits
thakis created this revision. thakis added a reviewer: hans. thakis added a subscriber: cfe-commits. There's no way to make a flag alias to two flags, so add a /WCL4 flag that maps to the All, Extra diag groups. Fixes PR25563. http://reviews.llvm.org/D15350 Files: docs/UsersManual.rst incl