This is an archive of the discontinued LLVM Phabricator instance.

[ELF] Use llvm::bsearch. NFC
ClosedPublic

Authored by MaskRay on Apr 16 2019, 10:32 PM.

Diff Detail

Repository
rL LLVM

Event Timeline

MaskRay created this revision.Apr 16 2019, 10:32 PM
ruiu accepted this revision.Apr 17 2019, 12:35 AM

LGTM

Nice

ELF/DWARF.cpp
85 ↗(On Diff #195511)

We usually just use = or & to capture variables.

ELF/InputSection.cpp
1228 ↗(On Diff #195511)

Ditto

This revision is now accepted and ready to land.Apr 17 2019, 12:35 AM
MaskRay updated this revision to Diff 195516.Apr 17 2019, 12:54 AM

Use = to capture variables.

MaskRay marked 2 inline comments as done.Apr 17 2019, 12:57 AM
This revision was automatically updated to reflect the committed changes.