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

Move implementation of process_grey_object to concurrentMark.inline.hpp

    Details

    • Type: Bug
    • Status: Resolved
    • Priority: P4
    • Resolution: Fixed
    • Affects Version/s: 8u60, 9
    • Fix Version/s: 9
    • Component/s: hotspot
    • Labels:
      None
    • Subcomponent:
      gc
    • Resolved In Build:
      b81

      Backports

        Description

        We had a linker problem with a C++ - compiler that inlined process_grey_object at all uses but removed the symbol and the actual implementation.

        As there are explicit instantiations of this method in concurrentMark.cpp, this probably should not happen and can be considered of a bug of the c++ compiler.
        Though, it seems to be pointless to mark that method inline if there are explicit instantiations.

        However, considering performance, it would be better to move the implementation of process_grey_object to concurrentMark.inline.hpp in order to inline the method in other compilation units.

          Attachments

            Issue Links

              Activity

                People

                • Assignee:
                  asiebenborn Axel Siebenborn
                  Reporter:
                  asiebenborn Axel Siebenborn
                • Votes:
                  0 Vote for this issue
                  Watchers:
                  3 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved: