[Lldb-commits] [PATCH] D12115: [LLDB-MI] Fix -data-info-line and -symbol-list-lines when Windows filenames are used.
Dawn Perchik via lldb-commits
lldb-commits at lists.llvm.org
Tue Sep 8 14:12:12 PDT 2015
dawn retitled this revision from "[LLDB-MI] Fix -data-info-line when Windows filenames are used." to "[LLDB-MI] Fix -data-info-line and -symbol-list-lines when Windows filenames are used.".
dawn updated the summary for this revision.
dawn updated this revision to Diff 34255.
dawn added a comment.
This changes the code to use regex to handle the parsing. It is cleaner, more accurate, and faster than the previous code, while not requiring special support to check for Windows paths.
In reworking this patch, I discovered a major bug in -symbol-list-lines where it didn't check the filename so would report lines in header files as belonging to the compilation unit. That is fixed in this patch and a test added, but to fix -symbol-list-lines for header files requires a rewrite of -symbol-list-lines which I didn't have time for. I've added a FIXME for now.
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 21583 bytes
Desc: not available
More information about the lldb-commits