Bugsplat bug #365031: moved the definition of PRStack from pratom.h

to primpl.h so that it is not exported.
Bugsplat bug #364813: declare 'prstk_head' volatile, to prevent compiler
from optimizing away references in a loop.
r=larryh,srinivas@netscape.com

nsprpub/pr/include/pratom.h

@@ -87,15 +87,6 @@ struct PRStackElemStr {
 
 typedef struct PRStackStr PRStack;
 
-struct PRStackStr {
-    PRStackElem	prstk_head;			/* head MUST be at offset 0; assembly
-										language code relies on this
-									*/
-    PRLock		*prstk_lock;
-    char		*prstk_name;
-};
-
-
 /*
 ** FUNCTION: PR_CreateStack
 ** DESCRIPTION:

nsprpub/pr/include/private/primpl.h

@@ -1341,6 +1341,20 @@ struct PRSem {
 #endif
 };
 
+/*************************************************************************/
+
+struct PRStackStr {
+    /* head MUST be at offset 0; assembly language code relies on this */
+#if defined(AIX) || defined(OSF1)
+    volatile PRStackElem prstk_head;
+#else
+    PRStackElem prstk_head;
+#endif
+
+    PRLock *prstk_lock;
+    char *prstk_name;
+};
+
 /************************************************************************/
 
 /* XXX this needs to be exported (sigh) */