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

Andrzej Warzynski via Phabricator via flang-commits flang-commits at lists.llvm.org
Fri May 13 01:36:50 PDT 2022


awarzynski added a comment.

Thank you @dpalermo , you are a ⭐ ! This way we can have a much less intrusive solution, which is fantastic :)

> Note that the other libs would still use add_flang_library with either of the 2 patches, so we wouldn't be diverging that much.

Yes, but I was referring to other sub-projects and the semantics of similar macros there. By creating `install_flang_library`, the semantics start to diverge. In particular, how would people know whether to use `install_flang_library` or not?

In any case, sounds like this is sorted now - great team effort! 🎉 !


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

https://reviews.llvm.org/D124759



More information about the flang-commits mailing list