[Lldb-commits] [PATCH] D53749: [PDB] Fix `SymbolFilePDBTests` after r345313

Zachary Turner via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Fri Oct 26 02:21:38 PDT 2018


zturner added a subscriber: aleksandr.urakov.
zturner added a comment.

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.

That said, thanks for fixing this for me.  Sorry for the break.


Repository:
  rLLDB LLDB

https://reviews.llvm.org/D53749





More information about the lldb-commits mailing list