Casey Marshall wrote:
Attached is a patch I'd like to get some comments on, before I check it in, especially if anyone can spot holes in my C. This patch

+  /* FIXME: I'd bet dollars for donuts the compiler optimizes this
+     away. How can we load memory into core??? */
+  for (i = 0; i < size; i += pagesize)
+    {
+      foo += ((int *) address)[i];
+    }
 }

You can force the memory to be accessed e.g. like this:

  for (i = 0; ... )
     *((volatile int *)address + i;

Otherwise looks good after a quick scan.

-Archie

__________________________________________________________________________
Archie Cobbs      *        CTO, Awarix        *      http://www.awarix.com


_______________________________________________
Classpath-patches mailing list
Classpath-patches@gnu.org
http://lists.gnu.org/mailman/listinfo/classpath-patches

Reply via email to