aaron.ballman added a comment.

In terms of testing, I'd recommend adding `-test-print-qualified-type` and see 
if you can thread a qualified vs unqualified parameter through `PrintType()` 
and friends to try to share as much code as possible. You'll probably need new 
wrapper functions that calls `PrintType` with the correct argument value 
depending on which test is being run. WDYT?



================
Comment at: clang/tools/libclang/CXType.cpp:318-325
+  CXTranslationUnit TU = GetTU(CT);
+  SmallString<64> Str;
+  llvm::raw_svector_ostream OS(Str);
+  PrintingPolicy PP(cxtu::getASTUnit(TU)->getASTContext().getLangOpts());
+
+  std::string qname = clang::TypeName::getFullyQualifiedName(
+      T, cxtu::getASTUnit(TU)->getASTContext(), PP);
----------------
Simplifying a bit and fixing up a naming convention nit.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D138377/new/

https://reviews.llvm.org/D138377

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

Reply via email to