[polly] r303549 - [Docs] Fix Sphinx documentation in CMake check.

Siddharth Bhat via llvm-commits llvm-commits at lists.llvm.org
Mon May 22 06:16:02 PDT 2017


Author: bollu
Date: Mon May 22 08:16:02 2017
New Revision: 303549

URL: http://llvm.org/viewvc/llvm-project?rev=303549&view=rev
Log:
[Docs] Fix Sphinx documentation in CMake check.

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?

Differential Revision: https://reviews.llvm.org/D33386

Modified:
    polly/trunk/docs/CMakeLists.txt

Modified: polly/trunk/docs/CMakeLists.txt
URL: http://llvm.org/viewvc/llvm-project/polly/trunk/docs/CMakeLists.txt?rev=303549&r1=303548&r2=303549&view=diff
==============================================================================
--- polly/trunk/docs/CMakeLists.txt (original)
+++ polly/trunk/docs/CMakeLists.txt Mon May 22 08:16:02 2017
@@ -90,8 +90,8 @@ endif()
 endif()
 
 if (LLVM_ENABLE_SPHINX)
+  include(AddSphinxTarget)
   if (SPHINX_FOUND)
-    include(AddSphinxTarget)
     if (${SPHINX_OUTPUT_HTML})
       add_sphinx_target(html polly)
     endif()




More information about the llvm-commits mailing list