[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