X10

Optimize atomic regions that are simple instance field updates

Details

  • Number of attachments :
    0

Description

Given a simple x10 atomic:

atomic o.f = v;

we might be able to implement this with an AtomicFieldUpdater (Java) or an atomic memory operation (C++) instead of a full blown X10 atomic.

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
David Grove added a comment -

Unassign issues assigned to Nate.

Show
David Grove added a comment - Unassign issues assigned to Nate.
David Grove made changes -
Field Original Value New Value
Assignee Nate Nystrom [ nystrom ]
Hide
Permalink
Josh Milthorpe added a comment -

Given that in the general case atomic is currently implemented as "stop the world" – which is antithetical to multithreaded performance – the compiler should generate a warning for any non-optimized section.

I would also vote +1 on priority and -998 on fix version

Show
Josh Milthorpe added a comment - Given that in the general case atomic is currently implemented as "stop the world" – which is antithetical to multithreaded performance – the compiler should generate a warning for any non-optimized section. I would also vote +1 on priority and -998 on fix version
Hide
Permalink
Igor Peshansky added a comment -

It would have to also notify the runtime to process any conditional atomics waiting on the value (see x10.lang.Runtime.Latch.release()).

Show
Igor Peshansky added a comment - It would have to also notify the runtime to process any conditional atomics waiting on the value (see x10.lang.Runtime.Latch.release() ).
Yoav Zibin made changes -
Component/s X10 Compiler: Front-end [ 13471 ]
Hide
Permalink
Vijay Saraswat added a comment -

+1

Show
Vijay Saraswat added a comment - +1
Vijay Saraswat made changes -
Assignee David Grove [ dgrove ]
Fix Version/s X10 2.2 [ 16002 ]
Fix Version/s X10 1000 [ 14901 ]
Priority Minor [ 4 ] Major [ 3 ]
Component/s X10 Compiler: Common Optimizer [ 13920 ]
Component/s X10 Compiler: Java Backend [ 13490 ]
Component/s Native X10: Compiler Codegen [ 13491 ]
David Grove made changes -
Assignee David Grove [ dgrove ]
Fix Version/s X10 1000 [ 14901 ]
Fix Version/s X10 2.2 [ 16002 ]

People

  • Assignee:
    Unassigned
    Reporter:
    David Grove
Vote (2)
Watch (1)

Dates

  • Created:
    Updated: