This is an archive of the discontinued LLVM Phabricator instance.

[MIR] Speedup parsing (up to 1000x+)
ClosedPublic

Authored by evgeny777 on Jul 8 2020, 5:38 AM.

Details

Summary

Patch eliminates string length calculation when lexing a token. It lowers parsing time from 15 minutes to few seconds for 4MB MIR file on Ryzen7 4800H

Diff Detail

Event Timeline

evgeny777 created this revision.Jul 8 2020, 5:38 AM
Herald added a project: Restricted Project. · View Herald TranscriptJul 8 2020, 5:38 AM
Eugene.Zelenko set the repository for this revision to rG LLVM Github Monorepo.
arsenm accepted this revision.Jul 8 2020, 6:37 AM
This revision is now accepted and ready to land.Jul 8 2020, 6:37 AM
This revision was automatically updated to reflect the committed changes.
dblaikie added inline comments.
llvm/lib/CodeGen/MIRParser/MIParser.cpp
566

drop_front(SkipChar) might be simpler/clearer?