Harmony
  1. Harmony
  2. HARMONY-1781

[drlvm][jit][unit] BigDecimal.toString() works incorrectly on OPT.

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Critical Critical
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: None
    • Component/s: DRLVM
    • Labels:
      None

      Description

      This cause the following classlib test failures
      org.apache.harmony.tests.java.math.BigDecimalArithmeticTest and tests.api.java.math.BigDecimalTest.

      A standalone test to reproduce the issue is attached.

      1. TestBigDecimal.java
        0.5 kB
        Elena Semukhina
      2. fix-in-simplifier-reassoc.patch.txt
        0.5 kB
        Egor Pasko

        Activity

        Hide
        Elena Semukhina added a comment -
        Show
        Elena Semukhina added a comment - [drlvm] [unit] Blocked http://wiki.apache.org/harmony/Unit_Tests_Pass_on_DRLVM
        Hide
        Egor Pasko added a comment -

        test reduction showed that a simple test fails on OPT:

        public class ArithmeticTest {

        public static void main(String argv[]) {
        if ( test(6) == -4 )

        { System.out.println("PASS"); }

        else

        { System.out.println("FAIL"); }

        }

        static int test(int s)

        { return 1 - (s - 1); }

        }

        I have a patch to fix it. I'll attach it as soon as JIRA allows

        Show
        Egor Pasko added a comment - test reduction showed that a simple test fails on OPT: public class ArithmeticTest { public static void main(String argv[]) { if ( test(6) == -4 ) { System.out.println("PASS"); } else { System.out.println("FAIL"); } } static int test(int s) { return 1 - (s - 1); } } I have a patch to fix it. I'll attach it as soon as JIRA allows
        Hide
        Egor Pasko added a comment -

        the fix attached

        Show
        Egor Pasko added a comment - the fix attached
        Hide
        Geir Magnusson Jr added a comment -

        r463928

        Ubuntu 6 - testcase, smoke, c-unit, ~kernel

        Show
        Geir Magnusson Jr added a comment - r463928 Ubuntu 6 - testcase, smoke, c-unit, ~kernel

          People

          • Assignee:
            Geir Magnusson Jr
            Reporter:
            Elena Semukhina
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development