Uploaded image for project: 'JDK'
  1. JDK
  2. JDK-8213440

Lingering INCLUDE_ALL_GCS in test_oopStorage_parperf.cpp

    XMLWordPrintable

Details

    • gc
    • b20
    • generic
    • generic

    Backports

      Description

        The code in test_oopStorage_parperf.cpp is conditionalized on INCLUDE_ALL_GCS. The test only makes sense, and at one time only compiled, when a non-Serial GC was included. However, that macro is no longer the proper way to say that, as that macro is no longer ever defined (per JDK-8200729). As a result, the this test is always conditionalized away.

        JDK-8200557, which introduced this new test, was pushed 2018-05-03. JDK-8200729, which removed INCLUDE_ALL_GCS, was pushed 2018-05-04. The effect of the latter change on the test was missed because the test just quietly ceased to exist.

        Attachments

          Issue Links

            backported by

            Backport - A issue that is required to port a Bug or Feature into another product release. This issue type is generally associated with the main Bug/Feature to represent each individual release of the port. JDK-8284941 Lingering INCLUDE_ALL_GCS in test_oopStorage_parperf.cpp

            • P4 - Minor loss of function, or other problem where easy workaround is present.
            • Resolved

            Backport - A issue that is required to port a Bug or Feature into another product release. This issue type is generally associated with the main Bug/Feature to represent each individual release of the port. JDK-8285902 Lingering INCLUDE_ALL_GCS in test_oopStorage_parperf.cpp

            • P4 - Minor loss of function, or other problem where easy workaround is present.
            • Resolved
            relates to

            Enhancement - null JDK-8200557 OopStorage parallel iteration scales poorly

            • P4 - Minor loss of function, or other problem where easy workaround is present.
            • Resolved

            Enhancement - null JDK-8200729 Conditional compilation of GCs

            • P4 - Minor loss of function, or other problem where easy workaround is present.
            • Resolved
            links to

            Commit Commit openjdk/jdk11u-dev/580a9912

            Review Review openjdk/jdk11u-dev/1051

            Activity

              People

                kbarrett Kim Barrett
                kbarrett Kim Barrett
                Votes:
                0 Vote for this issue
                Watchers:
                4 Start watching this issue

                Dates

                  Created:
                  Updated:
                  Resolved: