<p dir="ltr">No commit access, could you please commit? Thanks.</p>
<div class="gmail_quote">On Jun 18, 2013 7:31 AM, "Greg Clayton" <<a href="mailto:gclayton@apple.com">gclayton@apple.com</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
These patches look good. Do you have commit access? If so, please commit, else let my know and I will commit these.<br>
<br>
On Jun 17, 2013, at 9:11 AM, Virgile Bello <<a href="mailto:virgile.bello@gmail.com">virgile.bello@gmail.com</a>> wrote:<br>
<br>
> 3 patches, aiming to improve PE/COFF support:<br>
> - First patch fix symbol reading (invalid header size from sizeof() == 20 != 18, and various bugfixes such as invalid skipping of auxiliary symbols, 4 bytes shift from beginning, etc...).<br>
> - Second patch add image_base to section vmaddr offset so that VM addr is in image_base space.<br>
> - Third patch add support for DWARF section in PECOFF (taken from ELF counterpart), since they are generated by gcc/clang under windows.<br>
><br>
> This is all part of _much_ bigger patches to come, in order to bring MinGW32 support (patch 2 was necessary to make it backtrace/evaluate variables properly during the port, in case you're wondering).<br>
><br>
> Please let me know if anything seems wrong.<br>
> Thanks!<br>
> <0001-Fix-for-PECOFF-symbol-reading.patch><0002-Fix-for-PECOFF-section-address-should-use-image_base.patch><0003-PECOFF-Added-support-for-DWARF-sections.patch>_______________________________________________<br>

> lldb-commits mailing list<br>
> <a href="mailto:lldb-commits@cs.uiuc.edu">lldb-commits@cs.uiuc.edu</a><br>
> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits</a><br>
<br>
</blockquote></div>