This is an archive of the discontinued LLVM Phabricator instance.

[NFC] [Object] Create library to fetch debug info by build ID.
ClosedPublic

Authored by mysterymath on Aug 23 2022, 2:52 PM.

Details

Summary

This creates a library for fetching debug info by build ID, whether
locally or remotely via debuginfod. The functionality was refactored
out of existing code in the Symboliize library. Existing utilities
were refactored to use this library.

Diff Detail

Event Timeline

mysterymath created this revision.Aug 23 2022, 2:52 PM
Herald added a project: Restricted Project. · View Herald TranscriptAug 23 2022, 2:52 PM
mysterymath requested review of this revision.Aug 23 2022, 2:52 PM
Herald added a project: Restricted Project. · View Herald TranscriptAug 23 2022, 2:52 PM
dblaikie added inline comments.
llvm/include/llvm/Debuginfod/BuildIDFetcher.h
25
llvm/include/llvm/Object/BuildID.h
42
mysterymath marked 2 inline comments as done.

Remove unneeded semicolons.

[NFC] [Object] Refactor out library to fetch debug info by build ID.

typo? [Object] Refactor Object to fetch debug info by build ID?

Update commit message.

[NFC] [Object] Refactor out library to fetch debug info by build ID.

typo? [Object] Refactor Object to fetch debug info by build ID?

The intent was "refactor-out", as in "extract by refactoring." Changed wording to "Create".

mysterymath retitled this revision from [NFC] [Object] Refactor out library to fetch debug info by build ID. to [NFC] [Object] Create library to fetch debug info by build ID..Sep 14 2022, 3:59 PM
mysterymath edited the summary of this revision. (Show Details)
mysterymath edited the summary of this revision. (Show Details)

Correct clang-format error.

mysterymath edited the summary of this revision. (Show Details)Sep 21 2022, 2:30 PM

Trim includes.

phosek accepted this revision.Sep 28 2022, 12:13 PM

This could be potentially split into multiple smaller patches to make this easier to review, land and revert (if needed). For example introducing llvm/include/llvm/Object/BuildID.h with just BuildID and BuildIDRef types and their usage could be its own patch. With that being said, I'd be also fine with this landing as is.

This revision is now accepted and ready to land.Sep 28 2022, 12:13 PM
This revision was landed with ongoing or failed builds.Sep 28 2022, 1:35 PM
This revision was automatically updated to reflect the committed changes.
llvm/include/llvm/Object/ObjectFile.h