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



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to