harmony-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "George Timoshenko (JIRA)" <j...@apache.org>
Subject [jira] Commented: (HARMONY-5500) [drlvm][jit][opt] IRBuilder improvement to make it usable in all HLO optimizations
Date Mon, 07 Apr 2008 05:50:24 GMT

    [ https://issues.apache.org/jira/browse/HARMONY-5500?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12586246#action_12586246

George Timoshenko commented on HARMONY-5500:


Egor was wrong in that post.
The issue 5500 has nothing similar with removing simplifier and CSE funtionality from IRBuilder!

IMO these IRBuilder features MUST be kept available even when IRBuilder becomes independent
from translator.
By "independence" from translator I mean IRBuilder will live during all HLO stages.
Each HLO pass can configure it (set flags, turn ON or OFF simplification and CSE hashing etc.)
and use for HIR instruction creation.
CSE hash must be cleaned when a pass finishes its usage.

> [drlvm][jit][opt] IRBuilder improvement to make it usable in all HLO optimizations
> ----------------------------------------------------------------------------------
>                 Key: HARMONY-5500
>                 URL: https://issues.apache.org/jira/browse/HARMONY-5500
>             Project: Harmony
>          Issue Type: Improvement
>          Components: DRLVM
>            Reporter: George Timoshenko
>            Priority: Minor
> This time IRBuilder lives only during translation.
> All HLO optimizations creates necessary instructions "manually". And sometimes it is
not very convenient.
> HLOAPIMagic pass is the best example of such inconvenience. A number of methods for generating
instructions into HIR are just copy/pasted there from IRBuilder.
> It would be better to make IRBuilder independant from translator and keep it alive during
all the HLO so HLOAPIMagic all other passes can use it for instruction creation.

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

View raw message