This is an archive of the discontinued LLVM Phabricator instance.

docs: Add documentation for the ThinLTO cache pruning policy string.
ClosedPublic

Authored by pcc on Jun 22 2017, 6:56 PM.

Diff Detail

Repository
rL LLVM

Event Timeline

pcc created this revision.Jun 22 2017, 6:56 PM
mehdi_amini added inline comments.Jun 22 2017, 10:08 PM
clang/docs/ThinLTO.rst
160 ↗(On Diff #103682)

Why do we document linker-specific option in clang docs? Is it usual?

pcc added inline comments.Jun 22 2017, 10:21 PM
clang/docs/ThinLTO.rst
160 ↗(On Diff #103682)

It does seem a little odd, but it's in the same place as the other linker-specific options above. I was also imagining that we would eventually make this into a clang driver option (similar to the cache location as you mentioned in D30522). At that point the clang docs would indeed be the right place to document it.

mehdi_amini added inline comments.Jun 22 2017, 10:25 PM
clang/docs/ThinLTO.rst
160 ↗(On Diff #103682)

Fair!
I'll let Teresa look and approve.

This revision is now accepted and ready to land.Jun 23 2017, 6:10 AM
This revision was automatically updated to reflect the committed changes.