Skip to content

Commit

Permalink
8265443: IGV: disambiguate groups by emiting additional properties
Browse files Browse the repository at this point in the history
Reviewed-by: thartmann
  • Loading branch information
jtfuller111 authored and TobiHartmann committed Sep 8, 2021
1 parent 59c9f75 commit f2f8136
Show file tree
Hide file tree
Showing 2 changed files with 10 additions and 0 deletions.
8 changes: 8 additions & 0 deletions src/hotspot/share/opto/idealGraphPrinter.cpp
Expand Up @@ -48,6 +48,8 @@ const char *IdealGraphPrinter::NODE_ELEMENT = "node";
const char *IdealGraphPrinter::NODES_ELEMENT = "nodes";
const char *IdealGraphPrinter::REMOVE_EDGE_ELEMENT = "removeEdge";
const char *IdealGraphPrinter::REMOVE_NODE_ELEMENT = "removeNode";
const char *IdealGraphPrinter::COMPILATION_ID_PROPERTY = "compilationId";
const char *IdealGraphPrinter::COMPILATION_OSR_PROPERTY = "osr";
const char *IdealGraphPrinter::METHOD_NAME_PROPERTY = "name";
const char *IdealGraphPrinter::METHOD_IS_PUBLIC_PROPERTY = "public";
const char *IdealGraphPrinter::METHOD_IS_STATIC_PROPERTY = "static";
Expand Down Expand Up @@ -313,6 +315,12 @@ void IdealGraphPrinter::begin_method() {
print_prop(METHOD_IS_STATIC_PROPERTY, TRUE_VALUE);
}

if (C->is_osr_compilation()) {
print_prop(COMPILATION_OSR_PROPERTY, TRUE_VALUE);
}

print_prop(COMPILATION_ID_PROPERTY, C->compile_id());

tail(PROPERTIES_ELEMENT);

_should_send_method = true;
Expand Down
2 changes: 2 additions & 0 deletions src/hotspot/share/opto/idealGraphPrinter.hpp
Expand Up @@ -57,6 +57,8 @@ class IdealGraphPrinter : public CHeapObj<mtCompiler> {
static const char *CONTROL_FLOW_ELEMENT;
static const char *REMOVE_EDGE_ELEMENT;
static const char *REMOVE_NODE_ELEMENT;
static const char *COMPILATION_ID_PROPERTY;
static const char *COMPILATION_OSR_PROPERTY;
static const char *METHOD_NAME_PROPERTY;
static const char *BLOCK_NAME_PROPERTY;
static const char *BLOCK_DOMINATOR_PROPERTY;
Expand Down

1 comment on commit f2f8136

@openjdk-notifier
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please sign in to comment.