[PATCH] D52529: [Frontend] Simplify -print-decl-contexts with DeclNodes.inc

2018-10-01 Thread Fangrui Song via Phabricator via cfe-commits
MaskRay updated this revision to Diff 167836. MaskRay added a comment. Delete -print-decl-contexts Repository: rC Clang https://reviews.llvm.org/D52529 Files: include/clang/Driver/CC1Options.td include/clang/Frontend/ASTConsumers.h include/clang/Frontend/FrontendOptions.h lib/Fronten

[PATCH] D52529: [Frontend] Simplify -print-decl-contexts with DeclNodes.inc

2018-10-01 Thread Richard Smith - zygoloid via Phabricator via cfe-commits
rsmith added a comment. Do we need `-print-decl-contexts` at all? It's not exposed by the driver, not tested, not documented, has a very strange output format (the bracketing character supplies some information about the kind of declaration, but it's not clear *what* information), and it doesn'

[PATCH] D52529: [Frontend] Simplify -print-decl-contexts with DeclNodes.inc

2018-10-01 Thread Fangrui Song via Phabricator via cfe-commits
MaskRay added a comment. A gentle ping :) Repository: rC Clang https://reviews.llvm.org/D52529 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D52529: [Frontend] Simplify -print-decl-contexts with DeclNodes.inc

2018-09-26 Thread Fangrui Song via Phabricator via cfe-commits
MaskRay created this revision. MaskRay added reviewers: rsmith, arphaman. Herald added subscribers: cfe-commits, jfb. Currently the switch conditions display Decl nodes with space-separated lowercase words. Some conditions are missing and will cause llvm_unreachable() failure. It is simpler to j