This is an archive of the discontinued LLVM Phabricator instance.

[LLD][NFC] Refactor: BuildID hash size calculation is now done in a single place.
ClosedPublic

Authored by bd1976llvm on Apr 24 2019, 10:14 AM.

Diff Detail

Event Timeline

bd1976llvm created this revision.Apr 24 2019, 10:14 AM
MaskRay added inline comments.Apr 24 2019, 7:08 PM
lld/ELF/SyntheticSections.h
151

const size_t HashSize;

ruiu added inline comments.Apr 24 2019, 7:26 PM
lld/ELF/Writer.cpp
2551

nit: instead of assigning to a new variable, I'd use In.BuildId->HashSize directly everywhere.

2554

nit: add a blank line before switch.

Addressed review comments

bd1976llvm marked 3 inline comments as done.Apr 25 2019, 5:30 PM
ruiu accepted this revision.Apr 25 2019, 8:43 PM

LGTM

This revision is now accepted and ready to land.Apr 25 2019, 8:43 PM
This revision was automatically updated to reflect the committed changes.