This is an archive of the discontinued LLVM Phabricator instance.

Add function to DWARFContext to get all line addresses
Needs ReviewPublic

Authored by sque-google on Nov 28 2016, 6:11 PM.

Details

Reviewers
echristo
Summary

Get all addresses that appear in the line tables of all compile units within the DWARF context.

Diff Detail

Event Timeline

sque-google retitled this revision from to Add function to DWARFContext to get all line addresses.
sque-google updated this object.
sque-google added a reviewer: echristo.
sque-google added a subscriber: llvm-commits.

We generally don't want to introduce unused code. What is the use-case for this API?

Seems like there should be an in-tree unit test, if it's not super complicated. Up to Eric though.

echristo edited edge metadata.Dec 8 2016, 3:07 PM

I think this is going to turn out to be not necessary yes Simon?

Yeah it seems that way. I'll close this once I make the internal changes to verify that this is no longer needed.