Mercurial > hg > graal-jvmci-8
comparison src/share/vm/opto/idealKit.cpp @ 6615:09aad8452938
7190310: Inlining WeakReference.get(), and hoisting $referent may lead to non-terminating loops
Summary: In C2 add software membar after load from Reference.referent field to prevent commoning of loads across safepoint since GC can change its value. In C1 always generate Reference.get() intrinsic.
Reviewed-by: roland, twisti, dholmes, johnc
author | kvn |
---|---|
date | Mon, 20 Aug 2012 09:58:58 -0700 |
parents | 6c97c830fb6f |
children | e626685e9f6c |
comparison
equal
deleted
inserted
replaced
6614:006050192a5a | 6615:09aad8452938 |
---|---|
293 //-----------------------------transform----------------------------------- | 293 //-----------------------------transform----------------------------------- |
294 Node* IdealKit::transform(Node* n) { | 294 Node* IdealKit::transform(Node* n) { |
295 if (_delay_all_transforms) { | 295 if (_delay_all_transforms) { |
296 return delay_transform(n); | 296 return delay_transform(n); |
297 } else { | 297 } else { |
298 return gvn().transform(n); | 298 n = gvn().transform(n); |
299 if (!gvn().is_IterGVN()) { | |
300 C->record_for_igvn(n); | |
301 } | |
302 return n; | |
299 } | 303 } |
300 } | 304 } |
301 | 305 |
302 //-----------------------------delay_transform----------------------------------- | 306 //-----------------------------delay_transform----------------------------------- |
303 Node* IdealKit::delay_transform(Node* n) { | 307 Node* IdealKit::delay_transform(Node* n) { |