[Mono-bugs] [Bug 41989][Wis] Changed - Volatile emit/use missing

[bugzilla-daemon@rocky.ximian.com]

Please do not reply to this email- if you want to comment on the bug, go to the
URL shown below and enter your comments there.

Changed by miguel@ximian.com.

http://bugzilla.ximian.com/show_bug.cgi?id=41989

--- shadow/41989	Mon Apr 28 08:07:33 2003
+++ shadow/41989.tmp.9283	Sun May  4 21:42:36 2003
@@ -39,6 +39,22 @@
 There is also one other thing to consider: it is very rare to access a
 field in some referenced assembly, so most of the time, mcs has all
 the info it needs in ModFlags and it doesn't need to do a dynamic call
 at all (it seems to do that already).
 So, since the operation turns out to be rare anyway, I'd go with the
 dynamic call. Let me know if my assumptions are wrong.
+
+------- Additional Comments From miguel@ximian.com  2003-05-04 21:42 -------
+you are correct that MCS already has this information around.  It
+does, and it uses this information to generate the volatile prefix
+when we are compiling the same group of files.
+
+So we work correctly as long as you do not access a volatile field
+exposed by an assembly.  This is low priority, as I have yet to see
+one of these beasts in the field, but it is worth keeping this in mind.
+
+In the MCS case 682 references happen to an external field, so that
+would be roughly 5%
+
+We can go with your solution, and later revisit the issue if it
+becomes an issue.
+