[Lldb-commits] [PATCH] D59708: [ExpressionParser] Add swift-lldb case for finding clang resource dir

Alex Langford via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Tue Mar 26 11:45:59 PDT 2019


xiaobai added inline comments.


================
Comment at: source/Plugins/ExpressionParser/Clang/ClangHost.cpp:42
 /// This will compute the clang resource directory assuming that clang was
-/// installed with the same prefix as lldb.
+/// installed with the same prefix as lldb. Note: the verify paramter is
+/// currently only used for testing.
----------------
jingham wrote:
> spelling: parameter
> 
> But also, you should say something more informative, like:
> 
> If verify is true, the first candidate resource directory will be returned.  This mode is only used for testing
I'll adjust this before I commit. Thanks for pointing that out.


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

https://reviews.llvm.org/D59708





More information about the lldb-commits mailing list