klimek added a comment.

In D68914#1709002 <https://reviews.llvm.org/D68914#1709002>, @owenpan wrote:

> Here are some `const StringRef &` examples 
> <https://github.com/llvm/llvm-project/blob/master/clang/lib/StaticAnalyzer/Checkers/IteratorChecker.cpp#L229>.


StringRef already is a const pointer type, so the idiomatic way is to pass it 
by value.


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

https://reviews.llvm.org/D68914



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

Reply via email to