Opened 3 years ago

Closed 3 years ago

#63608 closed enhancement (fixed)

openmpi/mpich: add support for clang12, and clang13 too if possible

Reported by: mascguy (Christopher Nielsen) Owned by: mascguy (Christopher Nielsen)
Priority: Normal Milestone:
Component: ports Version:
Keywords: Cc: eborisch (Eric A. Borisch), Dave-Allured (Dave Allured)
Port: openmpi mpich

Description (last modified by mascguy (Christopher Nielsen))

Since MacPorts Clang 12 has had time to mature, it's worth testing both openmpi and mpich to see if they'll build and function with it.

It might also be worth testing with Clang 13 as well.

Change History (9)

comment:1 Changed 3 years ago by mascguy (Christopher Nielsen)

Description: modified (diff)
Summary: openmpi/mpich: add support for clang12 → openmpi/mpich: add support for clang12; evaluate clang13

comment:2 Changed 3 years ago by Christopher Nielsen <mascguy@…>

In ff34b6db8b3260f664c67213cf15c188c5aae02a/macports-ports (master):

mpi pg: disable clang 13 for openmpi/mpich

  • Not yet supported by our MPI-related ports

See: #63608

comment:3 Changed 3 years ago by mascguy (Christopher Nielsen)

Update: Still waiting to see whether openmpi and mpich build successfully, with Clang 12 and 13. But one thing that was immediately apparent, is the need to cleanup implicit function declarations.

Hopefully it shouldn't be a big deal to fix, at least in principle. But definitely needs to be done before release, particularly for ARM.

Last edited 3 years ago by mascguy (Christopher Nielsen) (previous) (diff)

comment:4 Changed 3 years ago by mascguy (Christopher Nielsen)

Summary: openmpi/mpich: add support for clang12; evaluate clang13 → openmpi/mpich: add support for clang12, and clang13 too if possible

comment:5 Changed 3 years ago by mascguy (Christopher Nielsen)

Good News: openmpi and mpich both build successfully, against Clang 12 and 13. No surprise, but still great to see.

Next step is to fix the implicit function declarations.

Last edited 3 years ago by mascguy (Christopher Nielsen) (previous) (diff)

comment:6 Changed 3 years ago by eborisch (Eric A. Borisch)

See ​https://github.com/macports/macports-ports/pull/13416 for WIP port. Any reason not to push the green button?

comment:7 Changed 3 years ago by kencu (Ken)

see also #64436

comment:8 Changed 3 years ago by Dave-Allured (Dave Allured)

Cc: Dave-Allured added

comment:9 Changed 3 years ago by mascguy (Christopher Nielsen)

Resolution: → fixed
Status: assigned → closed

Fixed, thanks to work from Eric and others.

Note: See TracTickets for help on using tickets.