You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@harmony.apache.org by "Xiaoming Gu (JIRA)" <ji...@apache.org> on 2008/11/27 08:45:44 UTC
[jira] Commented: (HARMONY-6027) [drlvm][jit] An incorrect register
assignment in graph coloring register allocation
[ https://issues.apache.org/jira/browse/HARMONY-6027?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12651276#action_12651276 ]
Xiaoming Gu commented on HARMONY-6027:
--------------------------------------
With some debugging on graph-coloring RA, I find the bug from register assignment. Following are the details:
Opnd#165 interferes with Opnd#20 which uses EDX. When assigning register to Opnd#165, it doesn't think DH is not available because it forgets to check the size of operands (Opnd#165 is 8-bit and Opnd#20 is 32-bit). Then DH is assigned to Opnd#165 incorrectly.
Maybe the same problem exists in other RA.
> [drlvm][jit] An incorrect register assignment 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.