You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@harmony.apache.org by "Evgueni Brevnov (JIRA)" <ji...@apache.org> on 2007/10/22 09:30:50 UTC

[jira] Created: (HARMONY-4984) [drlvm][jit] FlowGraph::duplicateRegion doesn't support SSA form

[drlvm][jit] FlowGraph::duplicateRegion doesn't support SSA form
----------------------------------------------------------------

                 Key: HARMONY-4984
                 URL: https://issues.apache.org/jira/browse/HARMONY-4984
             Project: Harmony
          Issue Type: Improvement
          Components: DRLVM
         Environment: ALL
            Reporter: Evgueni Brevnov


With attached patch FlowGraph::duplicateRegion() works fine on SSA form. Moreover it produces correct SSA graph with out additional necessity to fix up ssa form.

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


[jira] Commented: (HARMONY-4984) [drlvm][jit] FlowGraph::duplicateRegion doesn't support SSA form

Posted by "Mikhail Fursov (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/HARMONY-4984?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12537313 ] 

Mikhail Fursov commented on HARMONY-4984:
-----------------------------------------

Evgueni,
unfortunately I can't start eclipse 3.2 with this patch on Windows IA32

C:\tools\eclipse_3_2>C:\users\mfursov\harmony\drlvm_test\build\win_ia32_msvc_release\deploy\jdk\jre\bin\java.exe -Xem:op
t   -Dosgi.install.area=.  -cp ./startup.jar org.eclipse.core.launcher.Main
Windows reported exception: 0xc0000005
Registers:
    EAX: 0x00000000, EBX: 0x08ed6834, ECX: 0x00000000, EDX: 0x162a0017
    ESI: 0x00000000, EDI: 0x08ed68a4, ESP: 0x0013dc68, EBP: 0x0013dd10
    EIP: 0x01279c1d
Stack trace:
  0: Jitrino::FlowGraph::duplicateNode (c:\users\mfursov\harmony\drlvm_test\vm\jitrino\src\optimizer\flowgraph.cpp:239)
  1: Jitrino::FlowGraph::_duplicateRegion (c:\users\mfursov\harmony\drlvm_test\vm\jitrino\src\optimizer\flowgraph.cpp:38
7)
  2: ?? (??:-1)
  3: org/eclipse/jdt/internal/compiler/parser/Parser.parse(Lorg/eclipse/jdt/internal/compiler/env/ICompilationUnit;Lorg/
eclipse/jdt/internal/compiler/CompilationResult;)Lorg/eclipse/jdt/internal/compiler/ast/CompilationUnitDeclaration; (Par
ser.java:9019)
  4: org/eclipse/jdt/internal/compiler/SourceElementParser.parseCompilationUnit(Lorg/eclipse/jdt/internal/compiler/env/I
CompilationUnit;Z)Lorg/eclipse/jdt/internal/compiler/ast/CompilationUnitDeclaration; (SourceElementParser.java:1518)
  5: org/eclipse/jdt/internal/core/CompilationUnit.buildStructure(Lorg/eclipse/jdt/internal/core/OpenableElementInfo;Lor
g/eclipse/core/runtime/IProgressMonitor;Ljava/util/Map;Lorg/eclipse/core/resources/IResource;)Z (CompilationUnit.java:13
4)
  6: org/eclipse/jdt/internal/core/Openable.generateInfos(Ljava/lang/Object;Ljava/util/HashMap;Lorg/eclipse/core/runtime
/IProgressMonitor;)V (Openable.java:-1)
  7: org/eclipse/jdt/internal/core/JavaElement.openWhenClosed(Ljava/lang/Object;Lorg/eclipse/core/runtime/IProgressMonit
or;)Ljava/lang/Object; (JavaElement.java:-1)
  8: org/eclipse/jdt/internal/core/BecomeWorkingCopyOperation.executeOperation()V (BecomeWorkingCopyOperation.java:38)
  9: org/eclipse/jdt/internal/core/JavaModelOperation.run(Lorg/eclipse/core/runtime/IProgressMonitor;)V (JavaModelOperat
ion.java:-1)
 10: org/eclipse/jdt/internal/core/JavaModelOperation.runOperation(Lorg/eclipse/core/runtime/IProgressMonitor;)V (JavaMo
delOperation.java:-1)
 11: org/eclipse/jdt/internal/core/CompilationUnit.becomeWorkingCopy(Lorg/eclipse/jdt/core/IProblemRequestor;Lorg/eclips
e/core/runtime/IProgressMonitor;)V (CompilationUnit.java:-1)
 12: org/eclipse/jdt/internal/ui/javaeditor/CompilationUnitDocumentProvider.createFileInfo(Ljava/lang/Object;)Lorg/eclip
se/ui/editors/text/TextFileDocumentProvider$FileInfo; (CompilationUnitDocumentProvider.java:-1)
 13: org/eclipse/ui/editors/text/TextFileDocumentProvider.connect(Ljava/lang/Object;)V (TextFileDocumentProvider.java:-1
)
 14: org/eclipse/jdt/internal/ui/javaeditor/CompilationUnitDocumentProvider.connect(Ljava/lang/Object;)V (CompilationUni
tDocumentProvider.java:1069)
 15: org/eclipse/ui/texteditor/AbstractTextEditor.doSetInput(Lorg/eclipse/ui/IEditorInput;)V (AbstractTextEditor.java:-1
)
 16: org/eclipse/ui/texteditor/StatusTextEditor.doSetInput(Lorg/eclipse/ui/IEditorInput;)V (StatusTextEditor.java:173)
 17: org/eclipse/ui/texteditor/AbstractDecoratedTextEditor.doSetInput(Lorg/eclipse/ui/IEditorInput;)V (AbstractDecorated
TextEditor.java:1512)
 18: org/eclipse/jdt/internal/ui/javaeditor/JavaEditor.internalDoSetInput(Lorg/eclipse/ui/IEditorInput;)V (JavaEditor.ja
va:2370)
 19: org/eclipse/jdt/internal/ui/javaeditor/JavaEditor.doSetInput(Lorg/eclipse/ui/IEditorInput;)V (JavaEditor.java:2343)

 20: org/eclipse/jdt/internal/ui/javaeditor/CompilationUnitEditor.doSetInput(Lorg/eclipse/ui/IEditorInput;)V (Compilatio
nUnitEditor.java:1428)
 21: org/eclipse/ui/texteditor/AbstractTextEditor$17.run(Lorg/eclipse/core/runtime/IProgressMonitor;)V (AbstractTextEdit
or.java:-1)
 22: org/eclipse/jface/operation/ModalContext.runInCurrentThread(Lorg/eclipse/jface/operation/IRunnableWithProgress;Lorg
/eclipse/core/runtime/IProgressMonitor;)V (ModalContext.java:-1)
 23: org/eclipse/jface/operation/ModalContext.run(Lorg/eclipse/jface/operation/IRunnableWithProgress;ZLorg/eclipse/core/
runtime/IProgressMonitor;Lorg/eclipse/swt/widgets/Display;)V (ModalContext.java:313)
 24: org/eclipse/jface/window/ApplicationWindow$1.run()V (ApplicationWindow.java:763)
 25: org/eclipse/swt/custom/BusyIndicator.showWhile(Lorg/eclipse/swt/widgets/Display;Ljava/lang/Runnable;)V (BusyIndicat
or.java:-1)
 26: org/eclipse/jface/window/ApplicationWindow.run(ZZLorg/eclipse/jface/operation/IRunnableWithProgress;)V (Application
Window.java:-1)
 27: org/eclipse/ui/internal/WorkbenchWindow.run(ZZLorg/eclipse/jface/operation/IRunnableWithProgress;)V (WorkbenchWindo
w.java:2283)
 28: org/eclipse/ui/texteditor/AbstractTextEditor.internalInit(Lorg/eclipse/ui/IWorkbenchWindow;Lorg/eclipse/ui/IEditorS
ite;Lorg/eclipse/ui/IEditorInput;)V (AbstractTextEditor.java:-1)
...

> [drlvm][jit] FlowGraph::duplicateRegion doesn't support SSA form
> ----------------------------------------------------------------
>
>                 Key: HARMONY-4984
>                 URL: https://issues.apache.org/jira/browse/HARMONY-4984
>             Project: Harmony
>          Issue Type: Improvement
>          Components: DRLVM
>         Environment: ALL
>            Reporter: Evgueni Brevnov
>            Assignee: Mikhail Fursov
>         Attachments: duplicate_region.patch
>
>
> With attached patch FlowGraph::duplicateRegion() works fine on SSA form. Moreover it produces correct SSA graph with out additional necessity to fix up ssa form.

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


[jira] Updated: (HARMONY-4984) [drlvm][jit] FlowGraph::duplicateRegion doesn't support SSA form

Posted by "Evgueni Brevnov (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/HARMONY-4984?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Evgueni Brevnov updated HARMONY-4984:
-------------------------------------

    Attachment: duplicate_region.patch

The patch updated.

> [drlvm][jit] FlowGraph::duplicateRegion doesn't support SSA form
> ----------------------------------------------------------------
>
>                 Key: HARMONY-4984
>                 URL: https://issues.apache.org/jira/browse/HARMONY-4984
>             Project: Harmony
>          Issue Type: Improvement
>          Components: DRLVM
>         Environment: ALL
>            Reporter: Evgueni Brevnov
>            Assignee: Mikhail Fursov
>         Attachments: duplicate_region.patch, duplicate_region.patch
>
>
> With attached patch FlowGraph::duplicateRegion() works fine on SSA form. Moreover it produces correct SSA graph with out additional necessity to fix up ssa form.

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


[jira] Assigned: (HARMONY-4984) [drlvm][jit] FlowGraph::duplicateRegion doesn't support SSA form

Posted by "Mikhail Fursov (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/HARMONY-4984?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Mikhail Fursov reassigned HARMONY-4984:
---------------------------------------

    Assignee: Mikhail Fursov

> [drlvm][jit] FlowGraph::duplicateRegion doesn't support SSA form
> ----------------------------------------------------------------
>
>                 Key: HARMONY-4984
>                 URL: https://issues.apache.org/jira/browse/HARMONY-4984
>             Project: Harmony
>          Issue Type: Improvement
>          Components: DRLVM
>         Environment: ALL
>            Reporter: Evgueni Brevnov
>            Assignee: Mikhail Fursov
>         Attachments: duplicate_region.patch
>
>
> With attached patch FlowGraph::duplicateRegion() works fine on SSA form. Moreover it produces correct SSA graph with out additional necessity to fix up ssa form.

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


[jira] Resolved: (HARMONY-4984) [drlvm][jit] FlowGraph::duplicateRegion doesn't support SSA form

Posted by "Mikhail Fursov (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/HARMONY-4984?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Mikhail Fursov resolved HARMONY-4984.
-------------------------------------

    Resolution: Fixed

committed revision r594094

> [drlvm][jit] FlowGraph::duplicateRegion doesn't support SSA form
> ----------------------------------------------------------------
>
>                 Key: HARMONY-4984
>                 URL: https://issues.apache.org/jira/browse/HARMONY-4984
>             Project: Harmony
>          Issue Type: Improvement
>          Components: DRLVM
>         Environment: ALL
>            Reporter: Evgueni Brevnov
>            Assignee: Mikhail Fursov
>         Attachments: duplicate_region.patch, duplicate_region.patch
>
>
> With attached patch FlowGraph::duplicateRegion() works fine on SSA form. Moreover it produces correct SSA graph with out additional necessity to fix up ssa form.

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


[jira] Updated: (HARMONY-4984) [drlvm][jit] FlowGraph::duplicateRegion doesn't support SSA form

Posted by "Evgueni Brevnov (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/HARMONY-4984?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Evgueni Brevnov updated HARMONY-4984:
-------------------------------------

    Attachment: duplicate_region.patch

The patch was checked on Windows & Linux IA32 with 'build test'

> [drlvm][jit] FlowGraph::duplicateRegion doesn't support SSA form
> ----------------------------------------------------------------
>
>                 Key: HARMONY-4984
>                 URL: https://issues.apache.org/jira/browse/HARMONY-4984
>             Project: Harmony
>          Issue Type: Improvement
>          Components: DRLVM
>         Environment: ALL
>            Reporter: Evgueni Brevnov
>         Attachments: duplicate_region.patch
>
>
> With attached patch FlowGraph::duplicateRegion() works fine on SSA form. Moreover it produces correct SSA graph with out additional necessity to fix up ssa form.

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