RFR(XS) 8025657: compiler/intrinsics/mathexact/ConstantTest.java fails on assert in lcm.cpp on solaris x64

[Christian Thalinger]

Would it be possible to put all this logic into FlagsProjNode::pinned()?

On Oct 4, 2013, at 4:35 AM, Rickard Bäckman <rickard.backman at oracle.com> wrote:

> Hi all,
> 
> I would appreciate some reviews on this change.
> 
> On some machines when running with -XX:+DeoptimizeALot a test (test/compiler/intrinsics/mathexact/ConstantTest.java) failed with an assert.
> assert(idx >= 0) failed: index should be set
> 
> It seems that gcm would put the addExact in a different block then the jump.
> This change forces the producer of FlagsProjNode to be in the same block as
> the consumer.
> 
> Bug: https://bugs.openjdk.java.net/browse/JDK-8025657
> Webrev: http://cr.openjdk.java.net/~rbackman/8025657.1/
> 
> Thanks
> /R