RFR(L) 8026844: Various Math functions needs intrinsification

Vladimir Kozlov vladimir.kozlov at oracle.com
Fri Oct 18 08:50:42 PDT 2013


Rickard,

is_MathExactOpcode() could be replaced by adding the node to DEFINE_CLASS_ID structure as you did with FlagsProj. 
is_MathExact()  will be true for all its subclasses.

Put MathExactNode() constructors together.

In library_call.cpp can you name Integer version of method with I: inline_math_addExactI() to be symmetrical with Long 
version. And I don't think you need separate methods for increment and decrement - path flag to add and sub method.

Why you did not implement mulexact(reg, mem) mach nodes?

Otherwise it is good.

Thanks,
Vladimir

On 10/18/13 2:34 AM, Rickard Bäckman wrote:
> Hi, can I get reviews for the following change:
>
> This change creates intrinsics for addExact(long, long), subtractExact,
> negateExact, incrementExact, decrementExact and multiplyExact.
>
> The intrinsics are only added on x86 and only 64-bit has the intrinsics
> for the long-versions. (32-bit only has int-versions).
>
> Bug: https://bugs.openjdk.java.net/browse/JDK-8026844
> Webrev: http://cr.openjdk.java.net/~rbackman/8026844/
>
> Thanks
> /R
>


More information about the hotspot-compiler-dev mailing list