cleanup and minor extensions of the debugging facilities in CodeStrings

XMLWordPrintable

    • Type: Enhancement
    • Resolution: Fixed
    • Priority: P4
    • 9
    • Affects Version/s: 9
    • Component/s: hotspot
    • b74

        The change does not impact the product mode.

        In non product mode, CodeStrings allows to associate a linked list of strings to a CodeBuffer, CodeBlob or Stub. In addition, with ASSERTS, it defines a boolean asserting whether the list of strings are valid.

        Here are the issues addressed by this CR:

        - The old code mentioned the fact that CodeStrings was not always correctly initialized. This will be addressed by the fix, allowing check_valid to be added at a few locations where it could currently failed due to uninitialized values.

        - As a minor extension, it will be possible for platform dependent code to modify the comment separator used by print_block_comment, which was hard coded to " ;; ".

        - As another minor extension of the validity assertion, freeing a code string will not necessarily mark it (and hence its Stub/CodeBlob/CodeBuffer) as invalid. If CodeStrings contains only comment, removing them does not change the validity of the CodeStrings.

              Assignee:
              Bertrand Delsart (Inactive)
              Reporter:
              Bertrand Delsart (Inactive)
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

                Created:
                Updated:
                Resolved:

                  Estimated:
                  Original Estimate - 1 week
                  1w
                  Remaining:
                  0m
                  Logged:
                  Time Not Required
                  Not Specified