[PATCH] D30946: [ScopePrinting] Added support to print full scopes of types and declarations.
Simon Schroeder via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Wed Jun 14 03:53:50 PDT 2017
schroedersi added a comment.
Would it help the acceptance of the patch if I add a fourth option `LegacySuppressScope` (or similar) that emulates the current/pre-patch `SuppressScope==true`? This way the patched type printing would be a real superset of the current/non-patched type printing (except for some small changes to `SuppressScope==false`/`DefaultScope` as seen in the diff of tests `p6.cpp` and `comment-cplus-decls.cpp`).
https://reviews.llvm.org/D30946
More information about the cfe-commits
mailing list