It would be great if future work could go towards the native pdb plugin. To be honest, I’m trying very hard to get it to the point that the DIA plugin can be deleted, so large pieces of new functionality that only go to the DIA plugin are, in a way, wasted effort. I’m currently working on getting all of the tests in SymbolFile/PDB to pass with the other plugin, and there are only 2 remaining issues, neither of which is related to the plugin but rather clang.
So I think it’s very close. Would it be possible to start making this transition now and reimplement this patch on the other plugin? On Fri, Nov 30, 2018 at 6:44 AM Leonid Mashinskiy via Phabricator < revi...@reviews.llvm.org> wrote: > leonid.mashinskiy created this revision. > leonid.mashinskiy added reviewers: zturner, asmith, stella.stamenova, > aleksandr.urakov. > leonid.mashinskiy added a project: LLDB. > Herald added subscribers: lldb-commits, JDevlieghere, aprantl, mgorny. > > This patch makes LLDB able to retrieve proper values for function > arguments and local variables stored in PDB relative to VFRAME register. > > Patch contains retrieval of corresponding FPO table entries from PDB and a > generic translator from FPO programs to DWARF expressions to get correct > VFRAME value. > > Patch also improves variables-locations.test and makes this test passable > on x86. > > Related to D53086 <https://reviews.llvm.org/D53086> > > > Repository: > rLLDB LLDB > > https://reviews.llvm.org/D55122 > > Files: > lit/SymbolFile/PDB/Inputs/VariablesLocationsTest.cpp > lit/SymbolFile/PDB/Inputs/VariablesLocationsTest.script > lit/SymbolFile/PDB/variables-locations.test > source/Plugins/SymbolFile/PDB/CMakeLists.txt > source/Plugins/SymbolFile/PDB/CodeViewRegisterMapping.cpp > source/Plugins/SymbolFile/PDB/CodeViewRegisterMapping.h > source/Plugins/SymbolFile/PDB/PDBFPOProgramToDWARFExpression.cpp > source/Plugins/SymbolFile/PDB/PDBFPOProgramToDWARFExpression.h > source/Plugins/SymbolFile/PDB/PDBLocationToDWARFExpression.cpp > source/Plugins/SymbolFile/PDB/PDBLocationToDWARFExpression.h > source/Plugins/SymbolFile/PDB/SymbolFilePDB.cpp > >
_______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
