[PATCH] D80751: [clang][diagnostics] Add '-Wundef-prefix' warning option

2020-06-30 Thread Cyndy Ishida via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rG89a0c4066b0e: [clang][diagnostics] Add '-Wundef-prefix' warning option (authored by zixuw, committed by cishida). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.o

[PATCH] D80751: [clang][diagnostics] Add '-Wundef-prefix' warning option

2020-06-30 Thread Juergen Ributzka via Phabricator via cfe-commits
ributzka accepted this revision. ributzka added a comment. LGTM Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D80751/new/ https://reviews.llvm.org/D80751 ___ cfe-commits mailing list cfe-commits@lists.l

[PATCH] D80751: [clang][diagnostics] Add '-Wundef-prefix' warning option

2020-06-29 Thread Alex Lorenz via Phabricator via cfe-commits
arphaman accepted this revision. arphaman added a comment. This revision is now accepted and ready to land. LGTM. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D80751/new/ https://reviews.llvm.org/D80751 __

[PATCH] D80751: [clang][diagnostics] Add '-Wundef-prefix' warning option

2020-06-29 Thread Zixu Wang via Phabricator via cfe-commits
zixuw updated this revision to Diff 274276. zixuw added a comment. Abort the design of making 'Wundef' an alias to 'Wundef-prefix' because it depends on the alias expansion to work, which adds an empty string to 'UndefPrefixes' to do the trick. However, any other way to enable 'Wundef', for examp

[PATCH] D80751: [clang][diagnostics] Add '-Wundef-prefix' warning option

2020-06-29 Thread Zixu Wang via Phabricator via cfe-commits
zixuw updated this revision to Diff 274219. zixuw added a comment. - Remove implementation details from the help text of 'Wundef'; - Hide help text for 'Wundef-prefix' and 'Wundef'. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D80751/new/ https://r

[PATCH] D80751: [clang][diagnostics] Add '-Wundef-prefix' warning option

2020-06-29 Thread Zixu Wang via Phabricator via cfe-commits
zixuw updated this revision to Diff 274212. zixuw added a comment. - Refine test cases to check combinations of 'Wundef' and 'Wundef-prefix', and with/without 'Werror'; - Fix issues with '-Werror=undef' by explicitly looking for the option. Repository: rG LLVM Github Monorepo CHANGES SINCE L

[PATCH] D80751: [clang][diagnostics] Add '-Wundef-prefix' warning option

2020-06-12 Thread Zixu Wang via Phabricator via cfe-commits
zixuw added inline comments. Comment at: clang/lib/Lex/PPExpressions.cpp:262 + // string to UndefPrefixes as an explicit "-Wundef" does. + if (UndefPrefixes.empty() || + llvm::any_of(UndefPrefixes, arphaman wrote: > zixuw wrote: > >

[PATCH] D80751: [clang][diagnostics] Add '-Wundef-prefix' warning option

2020-06-12 Thread Alex Lorenz via Phabricator via cfe-commits
arphaman added inline comments. Comment at: clang/include/clang/Basic/DiagnosticGroups.td:108 +def Undefined : DiagGroup<"undef">; +def UndefinedPrefix : DiagGroup<"undef-prefix", [Undefined]>; def UnsupportedNan : DiagGroup<"unsupported-nan">; zixuw wrote: > ar

[PATCH] D80751: [clang][diagnostics] Add '-Wundef-prefix' warning option

2020-06-09 Thread Zixu Wang via Phabricator via cfe-commits
zixuw added inline comments. Comment at: clang/lib/Lex/PPExpressions.cpp:262 + // string to UndefPrefixes as an explicit "-Wundef" does. + if (UndefPrefixes.empty() || + llvm::any_of(UndefPrefixes, ributzka wrote: > What happens whe

[PATCH] D80751: [clang][diagnostics] Add '-Wundef-prefix' warning option

2020-06-09 Thread Zixu Wang via Phabricator via cfe-commits
zixuw updated this revision to Diff 269628. zixuw added a comment. Update test case for `#pragma clang diagnostic` Update test case to verify that the state of `-Wundef` can still be modified using `#pragma clang diagnostic` Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION htt

[PATCH] D80751: [clang][diagnostics] Add '-Wundef-prefix' warning option

2020-06-08 Thread Zixu Wang via Phabricator via cfe-commits
zixuw added inline comments. Comment at: clang/include/clang/Basic/DiagnosticGroups.td:108 +def Undefined : DiagGroup<"undef">; +def UndefinedPrefix : DiagGroup<"undef-prefix", [Undefined]>; def UnsupportedNan : DiagGroup<"unsupported-nan">; arphaman wrote: > It

[PATCH] D80751: [clang][diagnostics] Add '-Wundef-prefix' warning option

2020-06-08 Thread Alex Lorenz via Phabricator via cfe-commits
arphaman added a comment. Can you still modify the state of `-Wundef` with `#pragma clang diagnostic`? Could you add a test to cover it. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D80751/new/ https://reviews.llvm.org/D80751 __

[PATCH] D80751: [clang][diagnostics] Add '-Wundef-prefix' warning option

2020-06-08 Thread Alex Lorenz via Phabricator via cfe-commits
arphaman added inline comments. Comment at: clang/include/clang/Basic/DiagnosticGroups.td:108 +def Undefined : DiagGroup<"undef">; +def UndefinedPrefix : DiagGroup<"undef-prefix", [Undefined]>; def UnsupportedNan : DiagGroup<"unsupported-nan">; It seems like you

[PATCH] D80751: [clang][diagnostics] Add '-Wundef-prefix' warning option

2020-06-08 Thread Juergen Ributzka via Phabricator via cfe-commits
ributzka added inline comments. Comment at: clang/include/clang/Driver/Options.td:475 + Alias, + HelpText<"Aliased to '-Wundef-prefix=\"\"', enable warnings for undefined macros">; def Wwrite_strings : Flag<["-"], "Wwrite-strings">, Group, Flags<[CC1Option, HelpHidden]>; ---

[PATCH] D80751: [clang][diagnostics] Add '-Wundef-prefix' warning option

2020-05-28 Thread Zixu Wang via Phabricator via cfe-commits
zixuw created this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. zixuw added reviewers: ributzka, steven_wu, cishida, bruno, arphaman. Herald added a subscriber: dexonsmith. zixuw edited the summary of this revision. - Add an `-Wundef-prefix=,...` option, which i