[flang-commits] [PATCH] D124759: [flang] Install Fortran_main library

[Andrzej Warzynski via Phabricator via flang-commits]

awarzynski added a comment.

Hi @rovka & @dpalermo, thanks for working on this!

I am in favor of this change, but would much prefer the original approach from this patch, which is in line with how e.g. LLVM <https://github.com/llvm/llvm-project/blob/be656df18721dc55a1de2eea64a3f73b6afa29a2/llvm/cmake/modules/AddLLVM.cmake#L801-L858> or Clang <https://github.com/llvm/llvm-project/blob/be656df18721dc55a1de2eea64a3f73b6afa29a2/clang/cmake/modules/AddClang.cmake#L49-L143> implement `add_llvm_library` and `add_clang_library`, respectively.

I think that it would be good for the overall health of project to keep CMake modules semi-consistent across sub-projects. If we do diverge, that I'd like to make sure that that's unavoidable :) In particular, the error message from one of the earlier comments suggests //build issue// rather than an installation issue. I would hope that that can be solved //without// creating a dedicated macro for //installing// a library.

I'm traveling today, and for testing Flang I need stable SSH. Once I have access to a better connection, I can poke around - that's in the next few days. Would you mind waiting or, alternatively, having another go yourself ?

Thanks again for this work!


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D124759/new/

https://reviews.llvm.org/D124759