[Lldb-commits] Patch for Review: Fix "source list -n printf" on Linux (printf is symbol alias for __printf)

Michael Sartain mikesart at valvesoftware.com
Fri Jul 5 18:23:12 PDT 2013


On Fri, Jul 5, 2013 at 6:00 PM, Michael Sartain
<mikesart at valvesoftware.com>wrote:

> "source list -n printf" wasn't working on Linux because printf is a symbol
> and the code was only searching for functions.
>
> The patch now searches for symbols if function searching fails and tries
> to find a symbol that lines up exactly with a function.
>
> The Linux test suite passes with this patch. Please let me know if it's ok
> to submit.
>

arc diff added every open file I had to that patch. I need to figure out
how to use this thing before I hurt someone.

In any case, the patch is here now: http://llvm-reviews.chandlerc.com/D1109

Thanks.
 -Mike
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/lldb-commits/attachments/20130705/3c185c77/attachment.html>


More information about the lldb-commits mailing list