[flang-commits] [PATCH] D78136: [flang][docs]Doxygen support in flang

sameeran joshi via Phabricator via flang-commits flang-commits at lists.llvm.org
Thu Apr 16 11:41:43 PDT 2020

sameeranjoshi requested review of this revision.
sameeranjoshi added a reviewer: mehdi_amini.
sameeranjoshi added a comment.

Above doxygen patch seems to fail for me for below flags

cmake ../llvm

ninja install

As it tries to copy <path to build>/docs/html to <path to install prefix>/docs the same error for out-of-tree reported (here <https://github.com/flang-compiler/f18/pull/1065#issuecomment-606709553>) which I tried to fix for out-of-tree builds (here <https://github.com/flang-compiler/f18/pull/1065/commits/ea4f9612b8acac681ee64993663c964142b92304>)
Now that we are in tree, things seem to have dependency on mlir and llvm as well.

llvm-project/mlir as well shows the same error.
When adding the dependency (here <https://github.com/flang-compiler/f18/pull/1065/commits/ea4f9612b8acac681ee64993663c964142b92304>)  to llvm and mlir docs/CMakeLists.txt seems to fix error.

@sscalpone is that the expected way how the doxygen is tested in other llvm umbrella projects or am I testing it in wrong way?

@jdoerfert does clang and mlir also have the similar issue, which needs to be fixed, as adding a fix in these projects seems to solve issue in flang ?



More information about the flang-commits mailing list