-
Notifications
You must be signed in to change notification settings - Fork 858
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
NEWS: update for 2.0.2 rc4 #2746
NEWS: update for 2.0.2 rc4 #2746
Conversation
------------------------- | ||
|
||
Bug fixes/minor improvements: | ||
|
||
|
||
- Fix a deadlock issue arising from OpenMPI's approach to catching calls to |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"Open MPI", not "OpenMPI". 😲
munmap. Thanks to Paul Hargrove for reporting and helping to analyze this | ||
problem. | ||
- Fix a problem with PPC atomics which causes make check to fail unless builtin | ||
atomics configure option is enabled. Thanks to Orion Poplawski for reporting. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think you need s/causes/caused/ to make the sentence make grammatical sense.
- Fix a problem with PPC atomics which causes make check to fail unless builtin | ||
atomics configure option is enabled. Thanks to Orion Poplawski for reporting. | ||
- Fix a problem with use of x86_64 cpuid instruction which led to segmentation | ||
faults when Open MPI is configured with -O3 optimization. Thanks to Mark |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
s/is/was/
- Fix a problem with use of x86_64 cpuid instruction which led to segmentation | ||
faults when Open MPI is configured with -O3 optimization. Thanks to Mark | ||
Santcroos for reporting this problem. | ||
- Fix a problem when using built in atomics configure options on PPC platforms |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You use "built in" here, but "builtin" above. Pick one. 😄
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Heh. Still didn't fix this one. 😄
c950ddb
to
7db8fe4
Compare
@jsquyres grammar fixes incorporated. |
bot:notest Signed-off-by: Howard Pritchard <howardp@lanl.gov>
7db8fe4
to
addbead
Compare
@jsquyres NEWS should be up to date now. |
[ci skip] [skip ci]
Signed-off-by: Howard Pritchard howardp@lanl.gov