[PATCH] D27962: Get function start line number from DWARF info

[David Blaikie via llvm-commits]

On Tue, Jan 17, 2017 at 2:10 PM Simon Que via Phabricator <
reviews at reviews.llvm.org> wrote:

> sque added a comment.
>
> In https://reviews.llvm.org/D27962#647456, @dblaikie wrote:
>
> > Also - what's the use case for this? (& test coverage would be
> good/necessary)
>
>
> Use case: Internal use case, will email you about it.
>

Would it be possible to motivate these changes with improvements to
llvm-symbolizer (since we use that internally, seems like maybe
improvements to internal symbolication things might be good to have in
llvm-symbolizer too & that'd make it easier to carry these changes upstream
(since then regressions would be more obvious and other developers would
have a stronger motivation/demonstration of the value/need for these
features))?

Don't want to sign you up for lots of work, but can be a bit tricky
carrying unused code upstream like this.

(perhaps Eric can weigh in on what he's got/had in mind here)


>
> Unit test: No unit test exists for DWARFContext, but I added one here,
> awaiting review: https://reviews.llvm.org/D28428
>
> (Both are already mentioned earlier in this thread)
>
>
> https://reviews.llvm.org/D27962
>
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170117/1d9288c7/attachment.html>