-
Notifications
You must be signed in to change notification settings - Fork 12.4k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Flang not respecting -module
flag
#66969
Comments
@llvm/issue-subscribers-flang-driver
While [trying](https://github.com/conda-forge/scipy-feedstock/pull/252) to get flang working on a project that uses the meson build system, we ran into:
This is because It seems this assumption matches parts of flang's documentation (which do indicate llvm-project/flang/docs/ModFiles.md Lines 106 to 121 in 2147e9e
... but not others, which say: I'm not sure if this was an oversight in the recent option-refactor. It's certainly conceivable to teach meson that (llvm-)flang needs CC @banach-space |
Hi @h-vetinari , sorry for the delay. You will find the relevant discussion here: https://lists.llvm.org/pipermail/flang-dev/2020-November/000597.html
|
Ok, thanks for the context and confirming that this is intentional. Closing |
While trying to get flang working on a project that uses the meson build system, we ran into:
This is because
-module
is apparently so common in Fortran compilers, that it's the baseline assumption for how meson constructs its commandline invocations for most Fortran compilers (it's worth noting that this method gets overridden for several compilers, though notably, meson for classic flang defaults to-module
).It seems this assumption matches parts of flang's documentation (which do indicate
-module
support)...llvm-project/flang/docs/ModFiles.md
Lines 106 to 121 in 2147e9e
... but not others, which say:
I'm not sure if this was an oversight in the recent option-refactor. It's certainly conceivable to teach meson that (llvm-)flang needs
-module-dir
instead of-module
. However, it's at least surprising (given the documentation divergence), so I thought I'd raise this.CC @banach-space
The text was updated successfully, but these errors were encountered: