Skip to content

Conversation

@nysal
Copy link
Member

@nysal nysal commented Jan 11, 2017

Add a missing constraint to the input operand list.
This fixes a regression caused by d4be138.
Thanks to Orion Poplawski for reporting the issue.

Refs #2610

Signed-off-by: Nysal Jan K.A [email protected]

Add a missing constraint to the input operand list.
This fixes a regression caused by d4be138.
Thanks to Orion Poplawski for reporting the issue.

Refs #2610

Signed-off-by: Nysal Jan K.A <[email protected]>
@nysal nysal added the bug label Jan 11, 2017
@nysal nysal self-assigned this Jan 11, 2017
@nysal nysal requested a review from hjelmn January 11, 2017 16:05
@hjelmn
Copy link
Member

hjelmn commented Jan 11, 2017

Same comment as the other. Use "+m" to reflect the read constraint.

@nysal
Copy link
Member Author

nysal commented Jan 12, 2017

@hjelmn I didn't change it to "+m" because of issue #2086.

@jsquyres
Copy link
Member

@nysal @gpaulsen @jjhursey Still need a review for this PR.

@nysal
Copy link
Member Author

nysal commented Jan 12, 2017

@jsquyres I'll merge this. Nathan, just reviewed PR #2708

@nysal nysal merged commit 16ca8c1 into open-mpi:master Jan 12, 2017
@jsquyres jsquyres changed the title asm/ppc: Fix a regression in powerpc atomics master: asm/ppc: Fix a regression in powerpc atomics Jan 12, 2017
@nysal nysal deleted the ppc_atomic_master branch February 26, 2019 06:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants