Skip to content

Conversation

@ggouaillardet
Copy link
Contributor

-march=... currently works at configure time, but is replaced
with -march=nehalem at build time under brew's superenv.

This causes Open MPI build to fail under brew.

This commit does pass the -march=... option that worked at configure time
via the HOMEBREW_ARCHFLAGS environment variable.

This is a bit hacky, but less suboptimal than simply not building
the op/avx component (e.g. configure --enable-mca-no-build=op-avx)

The suspicious superenv behavior has been reported at Homebrew/brew#10078.

Refs. #8306

Signed-off-by: Gilles Gouaillardet [email protected]

-march=... currently works at configure time, but is replaced
with -march=nehalem at build time under brew's superenv.

This causes Open MPI build to fail under brew.

This commit does pass the -march=... option that worked at configure time
via the HOMEBREW_ARCHFLAGS environment variable.

This is a bit hacky, but less suboptimal than simply not building
the op/avx component (e.g. configure --enable-mca-no-build=op-avx)

The suspicious superenv behavior has been reported at Homebrew/brew#10078.

Refs. open-mpi#8306

Signed-off-by: Gilles Gouaillardet <[email protected]>
@jsquyres
Copy link
Member

jsquyres commented Jan 1, 2021

Given all the discussion that occurred (on #8306 and related) since this was filed, I think this PR is made moot by #8322, right?

@ggouaillardet
Copy link
Contributor Author

@jsquyres yep, this PR is now moot, so I am closing it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants