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

Instance field load is replaced by wrong data Phi

    XMLWordPrintable

Details

    • b02
    • b137
    • generic
    • generic

    Backports

      Description

        In C2, instance field loads may be replaced by a corresponding, already existing data Phi in LoadNode::identity():
          // Search for an existing data phi which was generated before for the same
          // instance's field to avoid infinite generation of phis in a loop.

        The problem is, that we only check if the Phi is from the same region and corresponds to the same instance field. We don't check if it contains the actual "up to date" value from the last store to the input memory:
          if (phi->is_Phi() && phi != mem &&
              phi->as_Phi()->is_same_inst_field(this_type, this_iid, this_index, this_offset)) {
            return phi;
          }

        This problem was reported by the Apache Lucene because it causes "impossible" assertions in one of their tests:
        https://issues.apache.org/jira/browse/LUCENE-5168

        The problem is still reproducible with JDK 7, 8 and 9.

        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-8173347 Instance field load is replaced by wrong data Phi

            • P2 - Crashes, loss of data, severe memory leak.
            • 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-8175151 Instance field load is replaced by wrong data Phi

            • P2 - Crashes, loss of data, severe memory leak.
            • 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-8190023 Instance field load is replaced by wrong data Phi

            • P2 - Crashes, loss of data, severe memory leak.
            • 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-8192169 Instance field load is replaced by wrong data Phi

            • P2 - Crashes, loss of data, severe memory leak.
            • 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-8195257 Instance field load is replaced by wrong data Phi

            • P2 - Crashes, loss of data, severe memory leak.
            • 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-8197252 Instance field load is replaced by wrong data Phi

            • P2 - Crashes, loss of data, severe memory leak.
            • 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-8168920 Instance field load is replaced by wrong data Phi

            • P2 - Crashes, loss of data, severe memory leak.
            • Closed

            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-8168948 Instance field load is replaced by wrong data Phi

            • P2 - Crashes, loss of data, severe memory leak.
            • Closed

            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-8175903 Instance field load is replaced by wrong data Phi

            • P2 - Crashes, loss of data, severe memory leak.
            • Closed

            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-8177491 Instance field load is replaced by wrong data Phi

            • P2 - Crashes, loss of data, severe memory leak.
            • Closed

            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-8183053 Instance field load is replaced by wrong data Phi

            • P2 - Crashes, loss of data, severe memory leak.
            • Closed

            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-8185181 Instance field load is replaced by wrong data Phi

            • P2 - Crashes, loss of data, severe memory leak.
            • Closed
            relates to

            Enhancement - null JDK-8164632 Node indices should be treated as unsigned integers

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

            Enhancement - null JDK-6673473 Escape Analysis: Add the instance's field information to PhiNode.

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

            Bug - A problem which impairs or prevents the functions of the product. JDK-8217918 C2: -XX:+AggressiveUnboxing is broken

            • P3 - Major loss of function.
            • Resolved

            Activity

              People

                thartmann Tobias Hartmann
                kvn Vladimir Kozlov
                Votes:
                0 Vote for this issue
                Watchers:
                9 Start watching this issue

                Dates

                  Created:
                  Updated:
                  Resolved: