review (S) for 6849984: Value methods for platform dependent math functions constant fold incorrectly

[Vladimir Kozlov]

>> src/share/vm/opto/subnode.cpp
>> Did you tested on sparc where all StubRoutines::intrinsic_* are NULL?
>> I don't see their initialization on sparc.
> 
> I ran the whole thing through JPRT.  We don't use those nodes on sparc so that's why they don't have an implementation.  If you use those nodes you are required to provide an implementation.  There's an explicit comment about that in stubGenerator_sparc.cpp.

I forgot that these nodes generation is guarded by
Matcher::has_match_rule() and on sparc it is false.

Final changes look good.

Thanks,
Vladimir


> 
>> cpu/x86/vm/stubGenerator_x86_64.cpp
>> Use movdbl() instead of movsd().
> 
> Fixed.
> 
>> +       StubCodeMark mark(this, "StubRoutines", "cos");
>> ...
>> +       __ movsd(xmm0, Address(rsp, 0));
>> +       __ addq(rsp, 8);
>> +       __ fld_d(Address(rsp, 4)); <<<<<<  What is that???
> 
> I don't know.  It looks like a piece left behind during editing.  I've removed it.
> 
> tom
> 
>> +       __ ret(0);
>> +     }
>>
>> Vladimir
>>
>> Tom Rodriguez wrote:
>>> http://cr.openjdk.java.net/~never/6849984
>