Skip to content
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

Help on some downstream feedstocks -fallow-argument-mismatch #76

Closed
2 tasks
hmaarrfk opened this issue Dec 4, 2022 · 5 comments
Closed
2 tasks

Help on some downstream feedstocks -fallow-argument-mismatch #76

hmaarrfk opened this issue Dec 4, 2022 · 5 comments
Labels

Comments

@hmaarrfk
Copy link

hmaarrfk commented Dec 4, 2022

Comment:

Seem to be suffering from a bug you all resolved wrt -fallow-argument-mismatch

Your help is a ppreciated

@dalcinl
Copy link
Contributor

dalcinl commented Dec 6, 2022

Maybe it is a problem within cmake's FindMPI? I see this line in the logs:

-- BLT MPI Compile Flags:  -fallow-argument-mismatch

That flag is only valid for Fortran, but maybe cmake is somehow picking it and passing it down to the C compiler?

clang-14: error: unknown argument: '-fallow-argument-mismatch'

@hmaarrfk
Copy link
Author

hmaarrfk commented Dec 6, 2022

Looking at the PC file, it does appear as a fflag.

@dalcinl
Copy link
Contributor

dalcinl commented Dec 6, 2022

Looking at the PC file, it does appear as a fflag.

What about running mpicc -show and mpifort -show? Any chance cmake's FindMPI is picking the flag from the later, but then wrongly adding it to the flags for the C compiler?

@hmaarrfk
Copy link
Author

hmaarrfk commented Dec 6, 2022

Thank you for the tips. I likely have limited time. But I'll let the original feedstock maintainers take over with your input here and there!

@cyrush
Copy link

cyrush commented Jan 9, 2023

Hi All,

I found a resolution.

FindMPI is doing the correct thing, the problem is in conduit's build system.

We have an mpi smoke test that is combining all c + fortran flags reported and passing the combined flags to the c compiler -- which clang is (rightly) unhappy with.

I have a fix to filter the flags in conduit's build system and avoid the problem that will be in our next release.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants