[Bug java/1305] [JSR133] GCJ ignores volatile modifier

[Bug java/1305] [JSR133] GCJ ignores volatile modifier

[pinskia at gcc dot gnu dot org]

------- Additional Comments From pinskia at gcc dot gnu dot org  2005-04-25 20:28 -------
http://gee.cs.oswego.edu/dl/jmm/cookbook.html
Changing to New as JSR133 was voted on and approved in September.

-- 
           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|SUSPENDED                   |NEW
   Last reconfirmed|2005-01-20 02:13:44         |2005-04-25 20:28:13
               date|                            |


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=1305

[Bug java/1305] [JSR133] GCJ ignores volatile modifier

[aph at gcc dot gnu dot org]

-- 

aph at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |SUSPENDED


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=1305

[Bug java/1305] [JSR133] GCJ ignores volatile modifier

[aph at gcc dot gnu dot org]

------- Comment #9 from aph at gcc dot gnu dot org  2006-06-19 17:38 -------
Subject: Bug 1305

Author: aph
Date: Mon Jun 19 17:38:08 2006
New Revision: 114778

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=114778
Log:
2006-06-19  Andrew Haley  <aph@redhat.com>

        PR java/1305
        PR java/27908
        * expr.c (java_modify_addr_for_volatile): New function.
        (expand_java_field_op): Handle volatile fields.
        * java-gimplify.c (java_gimplify_component_ref): Call
        java_modify_addr_for_volatile to give the field_ref the correct
        volatile type.
        (java_gimplify_modify_expr): Likewise.
        * java-tree.h (java_modify_addr_for_volatile): New decl.


Modified:
    trunk/gcc/java/ChangeLog
    trunk/gcc/java/expr.c
    trunk/gcc/java/java-gimplify.c
    trunk/gcc/java/java-tree.h


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=1305

[Bug java/1305] [JSR133] GCJ ignores volatile modifier

[aph at gcc dot gnu dot org]

------- Comment #8 from aph at gcc dot gnu dot org  2006-06-13 12:52 -------
Note that this patch has some problems.  In particular, it doesn't work with
BC-compiled code.  Also, on x86 it doesn't insert memory barrier instructions,
but this is arguably a bug in __sync_synchronize() rather than a problem with
this patch.


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=1305

[Bug java/1305] [JSR133] GCJ ignores volatile modifier

[aph at gcc dot gnu dot org]

------- Comment #7 from aph at gcc dot gnu dot org  2006-06-13 12:44 -------
Subject: Bug 1305

Author: aph
Date: Tue Jun 13 12:43:56 2006
New Revision: 114609

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=114609
Log:
2006-06-09  Andrew Haley  <aph@redhat.com>

        PR java/1305
        PR java/27908
        * builtins.c (initialize_builtins): Add __sync_synchronize().
        * class.c (add_field): Mark volatile fields.
        * java-gimplify.c (java_gimplify_expr): Call new functions to
        handle self-modifying exprs and COMPONENT_REFs.
        (java_gimplify_component_ref): New.
        (java_gimplify_modify_expr): Add handling for volatiles.


Modified:
    trunk/gcc/java/ChangeLog
    trunk/gcc/java/builtins.c
    trunk/gcc/java/class.c
    trunk/gcc/java/java-gimplify.c


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=1305

[Bug java/1305] [JSR133] GCJ ignores volatile modifier

[csm at gnu dot org]

------- Comment #6 from csm at gnu dot org  2006-06-07 02:52 -------
*** Bug 27908 has been marked as a duplicate of this bug. ***


-- 

csm at gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |r_ovidius at eml dot cc


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=1305