<div dir="auto">Ah, it looks like my fail. I had an old piece of code, which retrieves an FPO program for the old PDB plugin, and have shared it with Leonid (we are colleagues). Sure, we will move this to the new plugin (and it looks like it's not so difficult).</div><br><div class="gmail_quote"><div dir="ltr">Am Fr., 30. Nov. 2018, 18:39 hat Zachary Turner <<a href="mailto:zturner@google.com">zturner@google.com</a>> geschrieben:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">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.<br><br>So I think it’s very close.<br><br>Would it be possible to start making this transition now and reimplement this patch on the other plugin?<br><div class="gmail_quote"><div dir="ltr">On Fri, Nov 30, 2018 at 6:44 AM Leonid Mashinskiy via Phabricator <<a href="mailto:reviews@reviews.llvm.org" target="_blank" rel="noreferrer">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">leonid.mashinskiy created this revision.<br>
leonid.mashinskiy added reviewers: zturner, asmith, stella.stamenova, aleksandr.urakov.<br>
leonid.mashinskiy added a project: LLDB.<br>
Herald added subscribers: lldb-commits, JDevlieghere, aprantl, mgorny.<br>
<br>
This patch makes LLDB able to retrieve proper values for function arguments and local variables stored in PDB relative to VFRAME register.<br>
<br>
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.<br>
<br>
Patch also improves variables-locations.test and makes this test passable on x86.<br>
<br>
Related to D53086 <<a href="https://reviews.llvm.org/D53086" rel="noreferrer noreferrer" target="_blank">https://reviews.llvm.org/D53086</a>><br>
<br>
<br>
Repository:<br>
  rLLDB LLDB<br>
<br>
<a href="https://reviews.llvm.org/D55122" rel="noreferrer noreferrer" target="_blank">https://reviews.llvm.org/D55122</a><br>
<br>
Files:<br>
  lit/SymbolFile/PDB/Inputs/VariablesLocationsTest.cpp<br>
  lit/SymbolFile/PDB/Inputs/VariablesLocationsTest.script<br>
  lit/SymbolFile/PDB/variables-locations.test<br>
  source/Plugins/SymbolFile/PDB/CMakeLists.txt<br>
  source/Plugins/SymbolFile/PDB/CodeViewRegisterMapping.cpp<br>
  source/Plugins/SymbolFile/PDB/CodeViewRegisterMapping.h<br>
  source/Plugins/SymbolFile/PDB/PDBFPOProgramToDWARFExpression.cpp<br>
  source/Plugins/SymbolFile/PDB/PDBFPOProgramToDWARFExpression.h<br>
  source/Plugins/SymbolFile/PDB/PDBLocationToDWARFExpression.cpp<br>
  source/Plugins/SymbolFile/PDB/PDBLocationToDWARFExpression.h<br>
  source/Plugins/SymbolFile/PDB/SymbolFilePDB.cpp<br>
<br>
</blockquote></div>
</blockquote></div>