[aarch64-port-dev ] 8211064: [AArch64] Interpreter and c1 don't correctly handle jboolean results in native calls

[Andrew Haley]

On 10/01/2018 06:27 PM, Aleksey Shipilev wrote:
> On 10/01/2018 07:24 PM, Andrew Haley wrote:
>> On 10/01/2018 06:13 PM, Andrew Haley wrote:
>>> The jdk8u patch for 8211064 includes a new test case. It is, like all such
>>> cases, shared code.
>>
>> Never mind, I've figured out what to do.
>>
>> http://cr.openjdk.java.net/~aph/8211064-2/
> 
> Wait a minute, why T_CHAR here?
> 
> templateInterpreter_aarch64.cpp:
> 
>  288   case T_BOOLEAN: __ uxtb(r0, r0);        break;
>  289   case T_CHAR   : __ c2bool(r0);          break;

Great catch, thanks. I misapplied the patch.

-- 
Andrew Haley
Java Platform Lead Engineer
Red Hat UK Ltd. <https://www.redhat.com>
EAC8 43EB D3EF DB98 CC77 2FAD A5CD 6035 332F A671