harmony-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Dmitry Pronichkin (JIRA)" <j...@apache.org>
Subject [jira] Updated: (HARMONY-5580) [drlvm][jitrino][opt][perf] branch translator enhancement
Date Mon, 17 Mar 2008 11:57:24 GMT

     [ https://issues.apache.org/jira/browse/HARMONY-5580?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]

Dmitry Pronichkin updated HARMONY-5580:
---------------------------------------

    Attachment: btr3.patch

I agree with all proposed refactoring. I've noticed just one small mistake in patch:

if(brachInst == NULL && !brachInst->hasKind(Inst::Kind_BranchInst))

should be replaced with:

if(brachInst == NULL || !brachInst->hasKind(Inst::Kind_BranchInst))


After this everything works correctly.

> [drlvm][jitrino][opt][perf] branch translator enhancement
> ---------------------------------------------------------
>
>                 Key: HARMONY-5580
>                 URL: https://issues.apache.org/jira/browse/HARMONY-5580
>             Project: Harmony
>          Issue Type: Improvement
>          Components: DRLVM
>         Environment: IA32
>            Reporter: Dmitry Pronichkin
>            Assignee: Mikhail Fursov
>         Attachments: btr.patch, btr2.diff, btr3.patch
>
>
> The patch introduces new way of branch elimination. It eliminates branches like this:
> if (x<0) {
>     x+=y;
> }
> transforming them in following assembly:
> mov tmp, x
> sar tmp, 31  (spreading sign for the whole value)
> and tmp, y    (moving "y" in tmp if x<0)
> add x, tmp

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


Mime
View raw message