You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@harmony.apache.org by Naveen Neelakantam <ne...@uiuc.edu> on 2007/02/27 01:48:16 UTC
Fwd: [jira] Updated: (HARMONY-3222) assertion fires with DaCapo eclipse (jitrino built in debug)
Fix provided.
Should I be emailing the dev list when I fix something? Or should
the issue just be marked as resolved?
I'm not really clear on the protocol (and yes, I read http://
harmony.apache.org/issue_resolution_guideline.html). :-)
Thanks,
Naveen
Begin forwarded message:
> From: "Naveen Neelakantam (JIRA)" <ji...@apache.org>
> Date: February 26, 2007 6:41:05 PM CST
> To: neelakan@uiuc.edu
> Subject: [jira] Updated: (HARMONY-3222) assertion fires with DaCapo
> eclipse (jitrino built in debug)
>
>
> [ https://issues.apache.org/jira/browse/HARMONY-3222?
> page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
>
> Naveen Neelakantam updated HARMONY-3222:
> ----------------------------------------
>
> Attachment: imm_cast_bug.patch
>
> This patch (imm_cast_bug) fixes the issue. The problem is with
> casting. Basically the following was occuring:
>
> in32 cast to uint32 cast to int64
>
> As a result, negative constants would become large positive constants:
>
> -1 cast to 4294967295 cast to 4294967295
>
> If this casted value was used to dereference an array it would be
> multipled by the array element size (let's say 4), which would
> exceed the maximum value that could fit in a 32-bit integer:
>
> 4294967295* 4 = 17179869180
>
>> assertion fires with DaCapo eclipse (jitrino built in debug)
>> ------------------------------------------------------------
>>
>> Key: HARMONY-3222
>> URL: https://issues.apache.org/jira/browse/
>> HARMONY-3222
>> Project: Harmony
>> Issue Type: Bug
>> Components: DRLVM
>> Environment: RHEL4 update 4, core 2 duo, gcc 4.1.0
>> Reporter: Naveen Neelakantam
>> Attachments: imm_cast_bug.patch
>>
>>
>> To see the assert, jitrino must be built in debug mode and the -
>> Xem:server option must be used
>>> java -showversion -Xem:server -jar dacapo-2006-10-MR1.jar eclipse
>> Apache Harmony Launcher : (c) Copyright 1991, 2006 The Apache
>> Software Foundation or its licensors, as applicable.
>> java version "1.5.0"
>> pre-alpha : not complete or compatible
>> svn = r510120, (Feb 21 2007), Linux/ia32/gcc 4.1.0, debug build
>> http://incubator.apache.org/harmony
>> ===== DaCapo eclipse starting =====
>> <setting up workspace...>
>> <creating
>> projects.............................................................
>> .>
>> <running tests at level 0...>
>> <performing build tests...>
>> org.apache.ant (not open) opening cleaning building
>> org.junit (not open) opening cleaning building
>> java: /home/zilles/neelakan/Sandbox/Harmony/test/working_vm/vm/
>> jitrino/src/codegenerator/ia32/Ia32Inst.cpp:92: void
>> Jitrino::Ia32::Opnd::setMemOpndSubOpnd
>> (Jitrino::Ia32::MemOpndSubOpndKind, Jitrino::Ia32::Opnd*):
>> Assertion `(so != MemOpndSubOpndKind_Displacement) || (2147483647
>> >= opnd->getImmValue() && (-2147483647 - 1) <= opnd->getImmValue
>> ())' failed.
>
> --
> This message is automatically generated by JIRA.
> -
> You can reply to this email to add a comment to the issue online.
>
Re: [jira] Updated: (HARMONY-3222) assertion fires with DaCapo eclipse (jitrino built in debug)
Posted by "Geir Magnusson Jr." <ge...@pobox.com>.
I think that if it's interesting (non trivial), it's not a bad idea
to mention it on the dev list. More eyes...
geir
On Feb 26, 2007, at 11:50 PM, Alexey Varlamov wrote:
> I believe it depends on issue priority/severity: normally it is enough
> to set "Patch available flag", but if you need the patch reviewed &
> integrated urgently, feel free to alert on the dev-list.
> --
> Alexey
>
> 2007/2/27, Naveen Neelakantam <ne...@uiuc.edu>:
>> Fix provided.
>>
>> Should I be emailing the dev list when I fix something? Or should
>> the issue just be marked as resolved?
>>
>> I'm not really clear on the protocol (and yes, I read http://
>> harmony.apache.org/issue_resolution_guideline.html). :-)
>>
>> Thanks,
>> Naveen
>>
>> Begin forwarded message:
>>
>> > From: "Naveen Neelakantam (JIRA)" <ji...@apache.org>
>> > Date: February 26, 2007 6:41:05 PM CST
>> > To: neelakan@uiuc.edu
>> > Subject: [jira] Updated: (HARMONY-3222) assertion fires with DaCapo
>> > eclipse (jitrino built in debug)
>> >
>> >
>> > [ https://issues.apache.org/jira/browse/HARMONY-3222?
>> > page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
>> >
>> > Naveen Neelakantam updated HARMONY-3222:
>> > ----------------------------------------
>> >
>> > Attachment: imm_cast_bug.patch
>> >
>> > This patch (imm_cast_bug) fixes the issue. The problem is with
>> > casting. Basically the following was occuring:
>> >
>> > in32 cast to uint32 cast to int64
>> >
>> > As a result, negative constants would become large positive
>> constants:
>> >
>> > -1 cast to 4294967295 cast to 4294967295
>> >
>> > If this casted value was used to dereference an array it would be
>> > multipled by the array element size (let's say 4), which would
>> > exceed the maximum value that could fit in a 32-bit integer:
>> >
>> > 4294967295* 4 = 17179869180
>> >
>> >> assertion fires with DaCapo eclipse (jitrino built in debug)
>> >> ------------------------------------------------------------
>> >>
>> >> Key: HARMONY-3222
>> >> URL: https://issues.apache.org/jira/browse/
>> >> HARMONY-3222
>> >> Project: Harmony
>> >> Issue Type: Bug
>> >> Components: DRLVM
>> >> Environment: RHEL4 update 4, core 2 duo, gcc 4.1.0
>> >> Reporter: Naveen Neelakantam
>> >> Attachments: imm_cast_bug.patch
>> >>
>> >>
>> >> To see the assert, jitrino must be built in debug mode and the -
>> >> Xem:server option must be used
>> >>> java -showversion -Xem:server -jar dacapo-2006-10-MR1.jar eclipse
>> >> Apache Harmony Launcher : (c) Copyright 1991, 2006 The Apache
>> >> Software Foundation or its licensors, as applicable.
>> >> java version "1.5.0"
>> >> pre-alpha : not complete or compatible
>> >> svn = r510120, (Feb 21 2007), Linux/ia32/gcc 4.1.0, debug build
>> >> http://incubator.apache.org/harmony
>> >> ===== DaCapo eclipse starting =====
>> >> <setting up workspace...>
>> >> <creating
>> >>
>> projects.............................................................
>> >> .>
>> >> <running tests at level 0...>
>> >> <performing build tests...>
>> >> org.apache.ant (not open) opening cleaning building
>> >> org.junit (not open) opening cleaning building
>> >> java: /home/zilles/neelakan/Sandbox/Harmony/test/working_vm/vm/
>> >> jitrino/src/codegenerator/ia32/Ia32Inst.cpp:92: void
>> >> Jitrino::Ia32::Opnd::setMemOpndSubOpnd
>> >> (Jitrino::Ia32::MemOpndSubOpndKind, Jitrino::Ia32::Opnd*):
>> >> Assertion `(so != MemOpndSubOpndKind_Displacement) || (2147483647
>> >> >= opnd->getImmValue() && (-2147483647 - 1) <= opnd->getImmValue
>> >> ())' failed.
>> >
>> > --
>> > This message is automatically generated by JIRA.
>> > -
>> > You can reply to this email to add a comment to the issue online.
>> >
>>
>>
Re: [jira] Updated: (HARMONY-3222) assertion fires with DaCapo eclipse (jitrino built in debug)
Posted by Alexey Varlamov <al...@gmail.com>.
I believe it depends on issue priority/severity: normally it is enough
to set "Patch available flag", but if you need the patch reviewed &
integrated urgently, feel free to alert on the dev-list.
--
Alexey
2007/2/27, Naveen Neelakantam <ne...@uiuc.edu>:
> Fix provided.
>
> Should I be emailing the dev list when I fix something? Or should
> the issue just be marked as resolved?
>
> I'm not really clear on the protocol (and yes, I read http://
> harmony.apache.org/issue_resolution_guideline.html). :-)
>
> Thanks,
> Naveen
>
> Begin forwarded message:
>
> > From: "Naveen Neelakantam (JIRA)" <ji...@apache.org>
> > Date: February 26, 2007 6:41:05 PM CST
> > To: neelakan@uiuc.edu
> > Subject: [jira] Updated: (HARMONY-3222) assertion fires with DaCapo
> > eclipse (jitrino built in debug)
> >
> >
> > [ https://issues.apache.org/jira/browse/HARMONY-3222?
> > page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
> >
> > Naveen Neelakantam updated HARMONY-3222:
> > ----------------------------------------
> >
> > Attachment: imm_cast_bug.patch
> >
> > This patch (imm_cast_bug) fixes the issue. The problem is with
> > casting. Basically the following was occuring:
> >
> > in32 cast to uint32 cast to int64
> >
> > As a result, negative constants would become large positive constants:
> >
> > -1 cast to 4294967295 cast to 4294967295
> >
> > If this casted value was used to dereference an array it would be
> > multipled by the array element size (let's say 4), which would
> > exceed the maximum value that could fit in a 32-bit integer:
> >
> > 4294967295* 4 = 17179869180
> >
> >> assertion fires with DaCapo eclipse (jitrino built in debug)
> >> ------------------------------------------------------------
> >>
> >> Key: HARMONY-3222
> >> URL: https://issues.apache.org/jira/browse/
> >> HARMONY-3222
> >> Project: Harmony
> >> Issue Type: Bug
> >> Components: DRLVM
> >> Environment: RHEL4 update 4, core 2 duo, gcc 4.1.0
> >> Reporter: Naveen Neelakantam
> >> Attachments: imm_cast_bug.patch
> >>
> >>
> >> To see the assert, jitrino must be built in debug mode and the -
> >> Xem:server option must be used
> >>> java -showversion -Xem:server -jar dacapo-2006-10-MR1.jar eclipse
> >> Apache Harmony Launcher : (c) Copyright 1991, 2006 The Apache
> >> Software Foundation or its licensors, as applicable.
> >> java version "1.5.0"
> >> pre-alpha : not complete or compatible
> >> svn = r510120, (Feb 21 2007), Linux/ia32/gcc 4.1.0, debug build
> >> http://incubator.apache.org/harmony
> >> ===== DaCapo eclipse starting =====
> >> <setting up workspace...>
> >> <creating
> >> projects.............................................................
> >> .>
> >> <running tests at level 0...>
> >> <performing build tests...>
> >> org.apache.ant (not open) opening cleaning building
> >> org.junit (not open) opening cleaning building
> >> java: /home/zilles/neelakan/Sandbox/Harmony/test/working_vm/vm/
> >> jitrino/src/codegenerator/ia32/Ia32Inst.cpp:92: void
> >> Jitrino::Ia32::Opnd::setMemOpndSubOpnd
> >> (Jitrino::Ia32::MemOpndSubOpndKind, Jitrino::Ia32::Opnd*):
> >> Assertion `(so != MemOpndSubOpndKind_Displacement) || (2147483647
> >> >= opnd->getImmValue() && (-2147483647 - 1) <= opnd->getImmValue
> >> ())' failed.
> >
> > --
> > This message is automatically generated by JIRA.
> > -
> > You can reply to this email to add a comment to the issue online.
> >
>
>