[flang-commits] [flang] [flang] Consider bind(c) when lowering calls to intrinsic module procedures (PR #70386)

Kiran Chandramohan via flang-commits flang-commits at lists.llvm.org
Fri Oct 27 07:15:26 PDT 2023


kiranchandramohan wrote:

> Second, I do see that the module there is missing bind(c) also. I have looked at that module and I see several differences - not just this case. For example, in the F18 module, `omp_logical_kind = 1` and in that one `omp_logical_kind = 4`. My point is that there are several differences that likely need consolidated at some point - and I am not the right person to start doing that.

Yes, at some point we have to copy that in (instead of the local omp_lib.f90), atleast for the upstream version installation since the upstream version will use the llvm openmp library. I agree that we have to consolidate the changes and that need not be part of this patch.

https://github.com/llvm/llvm-project/pull/70386


More information about the flang-commits mailing list