diff options
| author | Carl Shapiro <cshapiro@google.com> | 2010-03-08 14:38:42 -0800 |
|---|---|---|
| committer | Carl Shapiro <cshapiro@google.com> | 2010-03-08 14:55:43 -0800 |
| commit | 147dd3f9f0905dfc6687843a6ca3276031067505 (patch) | |
| tree | 9a73736fc198419cd0cc358cf9d7efc7c6c525e2 /vm/compiler/codegen/arm/CodegenDriver.c | |
| parent | d682adface0ef5679e0f272ad7f50deb713b6a5b (diff) | |
Hoist shape discrimination above thin lock owner test in the lock
procedure. It is unsafe to reckon a thin lock owner without having
first determined that the bit pattern of the lock word corresponds to
that of a thin lock. Without proper ordering, a monitor lock can and
will be created which, excluding the shape bit, corresponds to a lock
owned by the calling thread.
In addition, move compiler barriers so they immediately preceed base
address publication. Also, kill cargo-cult volatiles that confounded
my bug hunt.
Diffstat (limited to 'vm/compiler/codegen/arm/CodegenDriver.c')
0 files changed, 0 insertions, 0 deletions
