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

Convert vmIntrinsics::ID to enum class

    XMLWordPrintable

Details

    • b27

    Description

      Convert vmIntrinsics::SID to enum class to provide better type safety.

      Also, put this enum class in the top level, so it can be forward-declared. I.e.,

      enum class vmIntrinsicID : int;

      This avoids excessive inclusion of vmIntrinsics.hpp (which currently is included indirectly in almost every hotspot source files).

      Attachments

        Issue Links

          relates to

          Bug - A problem which impairs or prevents the functions of the product. JDK-8257502 Builds fail with new warnings after JDK-8256254

          • P3 - Major loss of function.
          • Resolved

          Enhancement - null JDK-8256424 Move ciSymbol::symbol_name() to ciSymbols::symbol_name()

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

          Enhancement - null JDK-8253402 Convert vmSymbols::SID to enum class

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

          Commit Commit openjdk/jdk/f48737c7

          Review Review openjdk/jdk/1237

          Activity

            People

              iklam Ioi Lam
              iklam Ioi Lam
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: