[jira] Commented: (HARMONY-6027) [drlvm][jit] An undetected interference in graph coloring register allocation

["Xiaoming Gu (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/HARMONY-6027?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12650152#action_12650152 ] 

Xiaoming Gu commented on HARMONY-6027:
--------------------------------------

There is no bug if bin-packing register allocation used for runMultiEncryptDecrypt(). Following are the code:

    with early_prop
	01FF67E1 I140: MOVSX t166(ECX):I_32,s164(ECX):I_32 

    without early_prop
	01FF6E6B I696: MOV t165(CL):I_8,s164(ECX):I_32 
	01FF6E6D I140: MOVSX t166(ECX):I_32,t165(CL):I_8 

No matter whether with early_prop or not, only ECX is changed. So the bug is gone using bin-packing.

> [drlvm][jit] An undetected interference in graph coloring register allocation
> -----------------------------------------------------------------------------
>
>                 Key: HARMONY-6027
>                 URL: https://issues.apache.org/jira/browse/HARMONY-6027
>             Project: Harmony
>          Issue Type: Bug
>          Components: DRLVM
>         Environment: IA32 Windows
>            Reporter: Xiaoming Gu
>
> When running crypto.rsa with early_prop bypassed, a run-time bug happens because of the following code for spec/benchmarks/crypto/rsa/Main.runMultiEncryptDecrypt().
>     with early_prop
> 	0230D094 I140: MOVSX t166(ESI):I_32,s164(EBX):I_32 
>     without early_prop
> 	0230D0B4 I663: MOV t165(DH):I_8,s164(EBX):I_32 
> 	0230D0B6 I140: MOVSX t166(ESI):I_32,t165(DH):I_8 
> After checking all the code in runMultiEncryptDecrypt(), I find EDX is defined and used several time in this method. With the control flow graph, I think the bug comes from that EDX is changed uncarefully for the above modification on DH.
> This bug is similar to H6016 from the view of early_prop. (https://issues.apache.org/jira/browse/HARMONY-6016) But they trig different bugs.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.