[flang-commits] [flang] [NFC][flang][OpenMP] Create `FortranUtils` lib and move `createMapInfoOp` to it (PR #154483)
Abid Qadeer via flang-commits
flang-commits at lists.llvm.org
Tue Aug 26 10:00:53 PDT 2025
abidh wrote:
Hi @ergawy
Thanks for working on my comments and thanks to @Meinersbur for explaining issues with `Support` library. The new lib route looks better to me.
Another way to understand my earlier comment is to look at the include headers in `llvm/lib/Support` and `llvm/include/llvm/Support`. You will see that all the files include llvm headers only from `llvm/ADT`, `llvm/Config` and `llvm/Support`. Now if one adds some new functionality in that library which require inclusion of header say from llvm/CodeGen (and linking with CodeGen library), then that is breaking the library layering. Adding a higher level library dependency on a lower level library.
https://github.com/llvm/llvm-project/pull/154483
More information about the flang-commits
mailing list