<div dir="ltr">Ahh, I meant to remind you, because I noticed this when I was looking through the SymbolFilePDB code recently.  I think the LLVM guideline is not to use so much auto.  The generally accepted rule is that we can use for the result of make_shared, make_unique, and result of iterator types, but otherwise we prefer to explicitly spell the types out.  You don't have to go and submit another patch on top of this one to fix it again, it's just something to keep in mind.<div><br></div><div>That said, thanks for fixing this for me.  Sorry for the break.</div></div><br><div class="gmail_quote"><div dir="ltr">On Fri, Oct 26, 2018 at 2:11 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 created this revision.<br>
aleksandr.urakov added reviewers: zturner, stella.stamenova.<br>
aleksandr.urakov added a project: LLDB.<br>
Herald added a subscriber: lldb-commits.<br>
<br>
This one fixes tests compilation preserving the type of the `scope` parameter.<br>
<br>
<br>
Repository:<br>
  rLLDB LLDB<br>
<br>
<a href="https://reviews.llvm.org/D53749" rel="noreferrer" target="_blank">https://reviews.llvm.org/D53749</a><br>
<br>
Files:<br>
  unittests/SymbolFile/PDB/SymbolFilePDBTests.cpp<br>
<br>
<br>
Index: unittests/SymbolFile/PDB/SymbolFilePDBTests.cpp<br>
===================================================================<br>
--- unittests/SymbolFile/PDB/SymbolFilePDBTests.cpp<br>
+++ unittests/SymbolFile/PDB/SymbolFilePDBTests.cpp<br>
@@ -270,7 +270,7 @@<br>
   EXPECT_EQ(2u, cus);<br>
<br>
   SymbolContextList sc_list;<br>
-  uint32_t scope = lldb::eSymbolContextCompUnit | lldb::eSymbolContextLineEntry;<br>
+  auto scope = lldb::eSymbolContextCompUnit | lldb::eSymbolContextLineEntry;<br>
<br>
   uint32_t count =<br>
       symfile->ResolveSymbolContext(source_file, 0, true, scope, sc_list);<br>
@@ -319,7 +319,7 @@<br>
   EXPECT_EQ(2u, cus);<br>
<br>
   SymbolContextList sc_list;<br>
-  uint32_t scope = lldb::eSymbolContextCompUnit | lldb::eSymbolContextLineEntry;<br>
+  auto scope = lldb::eSymbolContextCompUnit | lldb::eSymbolContextLineEntry;<br>
<br>
   // First test with line 7, and verify that only line 7 entries are added.<br>
   uint32_t count =<br>
<br>
<br>
</blockquote></div>