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

Redundant separate 'String' check in 'trust_final_non_static_fields' ciField.cpp

XMLWordPrintable

    • Icon: Enhancement Enhancement
    • Resolution: Unresolved
    • Icon: P5 P5
    • tbd
    • 26
    • hotspot

      In JDK-8234049 code was updated to trust all classes from 'java/lang' package.

        if (holder->is_in_package("java/lang/invoke") || holder->is_in_package("sun/invoke") ||
            holder->is_in_package("java/lang/reflect") || holder->is_in_package("jdk/internal/reflect") ||
            holder->is_in_package("jdk/internal/foreign/layout") || holder->is_in_package("jdk/internal/foreign") ||
            holder->is_in_package("jdk/internal/vm/vector") || holder->is_in_package("jdk/incubator/vector") ||
            holder->is_in_package("java/lang"))
          return true;

      It means separate String check now is not needed.

        if (holder->name() == ciSymbols::java_lang_String())
          return true;

            Unassigned Unassigned
            aturbanov Andrey Turbanov
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Created:
              Updated: