C2 asserts should provide more information

XMLWordPrintable

    • Type: Enhancement
    • Resolution: Unresolved
    • Priority: P4
    • tbd
    • Affects Version/s: 17
    • Component/s: hotspot

      I think we need wrappers for asserts in C2 to provide information about failure which may happens only once during testing.
      Example: JDK-8262093

      The assert code is this:

          } else if (mem->is_ArrayCopy()) {
            done = true;
          } else {
            assert(false, "unexpected node");
          }

      Without reproducer (which may impossible to make) it is hard to know which wrong Ideal node was there.

      There are a lot of other similar cases in C2 code. I think we should fix/improve assertion code.

            Assignee:
            Unassigned
            Reporter:
            Vladimir Kozlov
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

              Created:
              Updated: