Page MenuHomePhabricator

[Docs] Increase Doxygen cache size
ClosedPublic

Authored by jryans on May 20 2019, 5:59 AM.

Details

Summary

When building Doxygen docs for llvm and clang, it helpfully prints a warning at
the end noting that the LOOKUP_CACHE_SIZE value was too small to keep all
symbols in memory.

By increasing to the size it recommends, Doxygen builds have greatly improved
performance. On my machine, time to run doxygen-llvm changes from 34 minutes
to 22 minutes, which is a decent amount of time saved by changing a single
number.

Diff Detail

Repository
rC Clang

Event Timeline

jryans created this revision.May 20 2019, 5:59 AM
Herald added projects: Restricted Project, Restricted Project. ยท View Herald TranscriptMay 20 2019, 5:59 AM

I don't currently have commit access, so I would need someone to land this for me assuming it's approved. If it would be easier for me to submit separate patches for the llvm and clang changes, please let me know! ๐Ÿ˜„

hintonda accepted this revision.May 20 2019, 7:32 AM

Thanks, LGTM!

I'll give it a day for further comments, and barring any, commit it for you tomorrow.

This revision is now accepted and ready to land.May 20 2019, 7:32 AM
Closed by commit rC361343: [Docs] Increase Doxygen cache size (authored by dhinton, committed by ). ยท Explain WhyMay 21 2019, 5:54 PM
This revision was automatically updated to reflect the committed changes.