This revision was automatically updated to reflect the committed changes.
Closed by commit rL352535: Fix the behavior of clangs -w flag. (authored
by jyknight, committed by ).
Herald added a subscriber: llvm-commits.
Changed prior to commit:
https://reviews.llvm.org/D53199?vs=183966=184142#toc
jyknight added inline comments.
Comment at: clang/lib/Basic/DiagnosticIDs.cpp:460-463
+ // Honor -w: this disables all messages mapped to Warning severity, and
*also*
+ // any diagnostics which are not Error/Fatal by default (that is, they were
+ // upgraded by any of the
jyknight updated this revision to Diff 183966.
jyknight marked 2 inline comments as done.
jyknight added a comment.
Fix to not disable remarks, reword comment, adjust implementation-of-module.m
test-case.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D53199/new/
rsmith accepted this revision.
rsmith marked an inline comment as done.
rsmith added inline comments.
This revision is now accepted and ready to land.
Comment at: clang/lib/Basic/DiagnosticIDs.cpp:460-463
+ // Honor -w: this disables all messages mapped to Warning severity, and
jyknight added a comment.
Ping.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D53199/new/
https://reviews.llvm.org/D53199
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
jyknight added a comment.
Ping.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D53199/new/
https://reviews.llvm.org/D53199
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
jyknight created this revision.
jyknight added a reviewer: rsmith.
It is intended to disable _all_ warnings, even those upgraded to
errors via `-Werror=warningname` or `#pragma clang diagnostic error'
https://reviews.llvm.org/D53199
Files:
clang/lib/Basic/DiagnosticIDs.cpp
