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

Move ciSymbol::symbol_name() to ciSymbols::symbol_name()

    XMLWordPrintable

Details

    • b01

    Description

      ciSymbol.hpp use a macro to wrap symbols exported by vmSymbols.hpp. E.g., we have functions like

      ciSymbol* ciSymbol::java_lang_System();
      ciSymbol* ciSymbol::java_lang_String();

      See https://github.com/openjdk/jdk/blob/0357db358133b38d82ad959261bc3175ea442043/src/hotspot/share/ci/ciSymbol.hpp#L101

      This causes vmSymbols.hpp and vmIntrinsics.hpp to be recursively included by a large number of CPP files. Since only small number of CPP files actually use these functions, we should move this to a new header file, ciSymbols.hpp, so we can reduce the unnecessary includes.

      Attachments

        Issue Links

          Activity

            People

              redestad Claes Redestad
              iklam Ioi Lam
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: