I don’t think I’m really a good person to look at AST stuff.  I can look for general style comments, obvious flaws, and test coverage.  but you may be the best person regarding on the content of the patch.  Does that sound ok?<br><div class="gmail_quote"><div dir="ltr">On Fri, Aug 31, 2018 at 7:20 AM Aleksandr Urakov via Phabricator <<a href="mailto:reviews@reviews.llvm.org">reviews@reviews.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">aleksandr.urakov updated this revision to Diff 163528.<br>
aleksandr.urakov added a comment.<br>
<br>
Drop a scope part of a class member function name.<br>
<br>
<br>
<a href="https://reviews.llvm.org/D51162" rel="noreferrer" target="_blank">https://reviews.llvm.org/D51162</a><br>
<br>
Files:<br>
  include/lldb/Symbol/ClangASTContext.h<br>
  lit/SymbolFile/PDB/class-layout.test<br>
  lit/SymbolFile/PDB/func-symbols.test<br>
  lit/SymbolFile/PDB/pointers.test<br>
  source/Plugins/SymbolFile/PDB/PDBASTParser.cpp<br>
  source/Plugins/SymbolFile/PDB/PDBASTParser.h<br>
  source/Plugins/SymbolFile/PDB/SymbolFilePDB.cpp<br>
  source/Plugins/SymbolFile/PDB/SymbolFilePDB.h<br>
  source/Symbol/ClangASTContext.cpp<br>
<br>
</blockquote></div>