On Oct 7, 2010, at 9:28 AM, Supalov, Alexander wrote:
Our definition: volatile int MPIR_debug_state
Proposal definition: int MPIR_debug_state
This may be material, as the MPIR_debug_state is set by the program and inspected by the tool. We should probably add VOLATILE to the proposal.
I think you're right, the document needs to be updated. I looked back at the original text document that described the interface, and it has:
extern VOLATILE int MPIR_debug_state;
I just updated the document to make MPIR_debug_state be VOLATILE. I'm up in a few minutes for the 1st vote here at the Forum in Chicago. Martin and I talked about this in person: technically, this is a change in the document that was formally read in Stuttgart. I will ask the Forum if they're comfortable voting on this, or if we reset the clock and call this meeting the really-first-reading (and we have a first vote in December). Technically, we should do the latter, but it may be a small enough change to eek through. I'll let you know how it goes. -- Jeff Squyres [email protected] For corporate legal information go to: http://www.cisco.com/web/about/doing_business/legal/cri/