<div dir="auto">Ok, thank you! I'll contact you when I will be ready to work on the new plugin.</div><br><div class="gmail_quote"><div dir="ltr">Am Mo., 3. Dez. 2018, 20:07 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">Ok that makes sense.  I’ll work on this soon, I’m trying to get all existing dia PDB tests passing with the native plugin so i can delete old plugin, so I’ll keep this in mind and probably do this next.<br><br>I’m trying to fix some bugs in clang so that we emit compatible debug info to msvc, which currently blocks conversion of a few tests.  So after that I’ll work on this <br><div class="gmail_quote"><div dir="ltr">On Mon, Dec 3, 2018 at 9:02 AM Aleksandr Urakov <<a href="mailto:aleksandr.urakov@jetbrains.com" target="_blank" rel="noreferrer">aleksandr.urakov@jetbrains.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="auto"><div>If I understand correctly, the native plugin processes for now globals and constants, but it can't create locals, right? But this FPO conversion is applicable to locals only, so we need to implement locals creation in the new plugin first, before moving the conversion there. That's why we need these functions.<br><br><div class="gmail_quote"><div dir="ltr">Am Mo., 3. Dez. 2018, 19:41 hat Zachary Turner <<a href="mailto:zturner@google.com" target="_blank" rel="noreferrer">zturner@google.com</a>> geschrieben:<br></div></div></div></div><div dir="auto"><div><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Is the only reason we need these methods so that we can use lldb-test?  If so, perhaps the tests can be written in terms of lldb itself, similar to the existing NativePDB tests<br><div class="gmail_quote"><div dir="ltr">On Mon, Dec 3, 2018 at 8:39 AM Aleksandr Urakov via Phabricator <<a href="mailto:reviews@reviews.llvm.org" rel="noreferrer noreferrer" target="_blank">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 added a comment.<br>
<br>
I can do it, but unfortunately not this week... I want to join the native plugin development some later, at the end of this month, after some current work.<br>
<br>
<br>
Repository:<br>
  rLLDB LLDB<br>
<br>
CHANGES SINCE LAST ACTION<br>
  <a href="https://reviews.llvm.org/D55122/new/" rel="noreferrer noreferrer noreferrer" target="_blank">https://reviews.llvm.org/D55122/new/</a><br>
<br>
<a href="https://reviews.llvm.org/D55122" rel="noreferrer noreferrer noreferrer" target="_blank">https://reviews.llvm.org/D55122</a><br>
<br>
<br>
<br>
</blockquote></div>
</blockquote></div></div></div>
</blockquote></div>
</blockquote></div>