This is an archive of the discontinued LLVM Phabricator instance.

Some minor improvements on the symtab parsing code
ClosedPublic

Authored by tberghammer on Oct 26 2015, 11:35 AM.

Details

Summary

Some minor improvements on the symtab parsing code

  • Remove an unnecessary re-computation on arch spec from the ELF file
  • Use a local cache to optimize name based section lookup in symtab parsing
  • Optimize C++ method basename validation with replacing a regex with hand written code

These modifications reduce the time required to parse the symtab from
large applications by ~25% (tested with LLDB as inferior)

Diff Detail

Repository
rL LLVM

Event Timeline

tberghammer retitled this revision from to Some minor improvements on the symtab parsing code.
tberghammer updated this object.
tberghammer added a reviewer: clayborg.
tberghammer added a subscriber: lldb-commits.
clayborg accepted this revision.Oct 26 2015, 11:50 AM
clayborg edited edge metadata.
This revision is now accepted and ready to land.Oct 26 2015, 11:50 AM
This revision was automatically updated to reflect the committed changes.