dependencies mpi: add -show-compile-info and -show-link-info for mpich#15586
dependencies mpi: add -show-compile-info and -show-link-info for mpich#15586lgarrison wants to merge 1 commit intomesonbuild:masterfrom
Conversation
Per pmodels/mpich#1045 (specifically pmodels/mpich@f8e9f16), -show-compile-info and -show-link-info are the preferred ways to get the compile and link flags in recent MPICH/Intel MPI.
27c3ead to
b53d377
Compare
|
Did you test this with Intel? I've had #15260 opened for a while because we don't actually query the right tools for the oneAPI based compilers. |
If I understand it correctly, this one is orthogonal to yours (even though they conflict)? Anyway, since you have #15260 I was going to let you merge both in the order that you see fit! |
|
I was just using the MPICH wrappers (around the Intel LLVM compilers). A recent-ish Intel OneAPI compiler wrapper (on a different system) doesn't seem to know about But I think that's fine; it just means Meson will continue to discover flags through |
Using
dependency('mpi')with the MPICH compiler wrappers currently results in compiler warnings about unused command-line arguments. This is because Meson current discovers the compile and link arguments with-compile_infoand-link_info:meson/mesonbuild/dependencies/mpi.py
Lines 112 to 123 in 0ac8ebb
The
-compile_infoand-link_infoflags actually return complete (and identical) compile + link commands, causing link flags to get mixed into Meson's compile command:The newer
-show-compile-infoinstead just shows the compile flags (likewise for-show-link-info):Per pmodels/mpich#1045 (specifically pmodels/mpich@f8e9f16), it appears that
-show-compile-infoand-show-link-infoare the preferred ways to get the compile and link flags in recent MPICH/Intel MPI, and that-compile_infoand-link_infowere kept for legacy reasons.Summary of command output for completeness
With this patch, I confirm that the compile and link arguments are no longer mixed in the Meson build.