[PATCH] D88164: [clang][Sema] Use enumerator instead of hard-coded constant

2020-09-24 Thread Mikhail Maltsev via Phabricator via cfe-commits
This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG8cc842a95072: [clang][Sema] Use enumerator instead of hard-coded constant (authored by miyuki). Repository: rG LLVM Github Monorepo CHANGES SINCE

[PATCH] D88164: [clang][Sema] Use enumerator instead of hard-coded constant

2020-09-23 Thread Aaron Ballman via Phabricator via cfe-commits
aaron.ballman accepted this revision. aaron.ballman added a comment. This revision is now accepted and ready to land. Good catch, LGTM! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D88164/new/ https://reviews.llvm.org/D88164 __

[PATCH] D88164: [clang][Sema] Use enumerator instead of hard-coded constant

2020-09-23 Thread Mikhail Maltsev via Phabricator via cfe-commits
miyuki created this revision. miyuki added reviewers: compnerd, aaron.ballman. Herald added a project: clang. Herald added a subscriber: cfe-commits. miyuki requested review of this revision. Sema::DiagnoseSwiftName uses the constant 12 instead of the corresponding enumerator ExpectedFunctionWithP