Skip to content

Conversation

@carlocab
Copy link
Member

@carlocab carlocab commented May 4, 2021

  • Have you followed the guidelines for contributing?
  • Have you checked that there aren't other open pull requests for the same formula update/change?
  • Have you built your formula locally with brew install --build-from-source <formula>, where <formula> is the name of the formula you're submitting?
  • Is your test running fine brew test <formula>, where <formula> is the name of the formula you're submitting?
  • Does your build pass brew audit --strict <formula> (after doing brew install <formula>)?

The op-avx flag disables building features that take advantage of AVX
instructions.

Since open-mpi is able to detect CPU features at runtime, disabling
this is not necessary, and simply leads to lower performance for users
with newer machines.

See also open-mpi/ompi#8306, open-mpi/ompi#8361.

The `op-avx` flag disables building features that take advantage of AVX
instructions.

Since `open-mpi` is able to detect CPU features at runtime, disabling
this is not necessary, and simply leads to lower performance for users
with newer machines.

See also:

    open-mpi/ompi#8306
    open-mpi/ompi#8361
@BrewTestBot
Copy link
Contributor

🤖 A scheduled task has triggered a merge.

@BrewTestBot
Copy link
Contributor

:shipit: @carlocab has triggered a merge.

@BrewTestBot
Copy link
Contributor

⚠️ @carlocab bottle publish failed.

@carlocab carlocab deleted the open-mpi-avx branch May 4, 2021 18:08
@carlocab
Copy link
Member Author

carlocab commented May 4, 2021

oops.

@github-actions github-actions bot added the outdated PR was locked due to age label Jun 4, 2021
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jun 4, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

outdated PR was locked due to age

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants