njames93 marked 2 inline comments as done.
njames93 added inline comments.

================
Comment at: clang/include/clang/AST/Type.h:1319
 
+llvm::raw_ostream &operator<<(llvm::raw_ostream &OS, const clang::QualType 
&QT);
+
----------------
aaron.ballman wrote:
> Cleaning up the declaration a bit, but the important change is not making 
> `QT` a const ref (the type is small and meant to be passed by value cheaply).
I knew it needed cleaning, Just had issues with the declaration not matching 
the definition so added all qualifiers.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D123926

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

Reply via email to