[PATCH] D30946: [ScopePrinting] Added support to print full scopes of types and declarations.

[Simon Schroeder via Phabricator via cfe-commits]

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