This is an archive of the discontinued LLVM Phabricator instance.

[Polly] [Docs] Fix Sphinx documentation CMake check.
ClosedPublic

Authored by bollu on May 21 2017, 7:38 AM.

Details

Summary
  • include(AddSphinxTarget) needs to occur before checking SPHINX_FOUND.
  • docs-polly-html and docs-polly-man are now usable again.
  • Perhaps we should build docs in the CI as well?

Diff Detail

Repository
rL LLVM

Event Timeline

bollu created this revision.May 21 2017, 7:38 AM
bollu added a comment.May 21 2017, 7:39 AM

@grosser, @Meinersbur: would it be reasonable to enable building docs in the CI as well? Or is this too much overhead?

I'm no CMake expert, so I put this up for review instead of committing directly.

grosser accepted this revision.May 22 2017, 12:59 AM

LGTM.

This revision is now accepted and ready to land.May 22 2017, 12:59 AM
This revision was automatically updated to reflect the committed changes.