This is an archive of the discontinued LLVM Phabricator instance.

[llvm-readobj] Change variable names to match LLVM-style. NFC.
ClosedPublic

Authored by Higuoxing on Mar 28 2019, 6:49 AM.

Diff Detail

Repository
rL LLVM

Event Timeline

Higuoxing created this revision.Mar 28 2019, 6:49 AM
Herald added a project: Restricted Project. · View Herald TranscriptMar 28 2019, 6:49 AM

LGTM, with the change in name of one of the variables as suggested inline.

llvm/tools/llvm-readobj/ELFDumper.cpp
232 ↗(On Diff #192625)

Ref -> Need, to match the type.

Higuoxing updated this revision to Diff 192658.Mar 28 2019, 9:37 AM

Change SymbolVersionRefSection to SymbolVersionNeedSection

Higuoxing retitled this revision from [llvm-readobj] Change variable names to match LLVM-style to [llvm-readobj] Change variable names to match LLVM-style. NFC..Mar 28 2019, 6:18 PM
Higuoxing accepted this revision.Mar 28 2019, 6:23 PM
This revision is now accepted and ready to land.Mar 28 2019, 6:23 PM
This revision was automatically updated to reflect the committed changes.