RFR(S): 8248901: Signed immediate support in .../share/assembler.hpp is broken.

[Patric Hedlin]

Thanks for reviewing Tobias.

/Patric

On 2020-07-20 09:46, Tobias Hartmann wrote:
> Hi Patric,
>
> looks good to me.
>
> Best regards,
> Tobias
>
> On 07.07.20 13:00, Patric Hedlin wrote:
>> Dear all,
>>
>> I would like to ask for help to review the following change/update:
>>
>> Issue:  https://bugs.openjdk.java.net/browse/JDK-8248901
>> Webrev: http://cr.openjdk.java.net/~phedlin/tr8248901/
>>
>>
>> Current definition(s) of is_simm() and friends are not robust over inputs. Both min and max values
>> are undefined for width > 32 (and width < 0).
>> No is_uimm() is currently provided (added). Several definitions are not used (cleanup).
>>
>> NOTE: Adding currently unused is_simm9() and is_uimm12(), required by JDK-8247766.
>>
>>
>> Testing: hs-tier1-3
>>
>>
>> Best regards,
>> Patric