Skip to content

Commit

Permalink
Browse files Browse the repository at this point in the history
8273416: C2: assert(false) failed: bad AD file after JDK-8252372 with…
… UseSSE={0,1}

Reviewed-by: kvn, roland
  • Loading branch information
shipilev committed Nov 1, 2021
1 parent dbf5100 commit 89ade1d
Show file tree
Hide file tree
Showing 2 changed files with 22 additions and 2 deletions.
20 changes: 20 additions & 0 deletions src/hotspot/cpu/x86/x86_32.ad
Expand Up @@ -7203,6 +7203,7 @@ instruct castLL( eRegL dst ) %{
%}

instruct castFF( regF dst ) %{
predicate(UseSSE >= 2);
match(Set dst (CastFF dst));
format %{ "#castFF of $dst" %}
ins_encode( /*empty encoding*/ );
Expand All @@ -7211,6 +7212,25 @@ instruct castFF( regF dst ) %{
%}

instruct castDD( regD dst ) %{
predicate(UseSSE >= 2);
match(Set dst (CastDD dst));
format %{ "#castDD of $dst" %}
ins_encode( /*empty encoding*/ );
ins_cost(0);
ins_pipe( empty );
%}

instruct castFF_PR( regFPR dst ) %{
predicate(UseSSE < 2);
match(Set dst (CastFF dst));
format %{ "#castFF of $dst" %}
ins_encode( /*empty encoding*/ );
ins_cost(0);
ins_pipe( empty );
%}

instruct castDD_PR( regDPR dst ) %{
predicate(UseSSE < 2);
match(Set dst (CastDD dst));
format %{ "#castDD of $dst" %}
ins_encode( /*empty encoding*/ );
Expand Down
4 changes: 2 additions & 2 deletions src/hotspot/share/opto/castnode.hpp
Expand Up @@ -140,7 +140,7 @@ class CastFFNode: public ConstraintCastNode {
init_class_id(Class_CastFF);
}
virtual int Opcode() const;
virtual uint ideal_reg() const { return Op_RegF; }
virtual uint ideal_reg() const { return in(1)->ideal_reg(); }
};

class CastDDNode: public ConstraintCastNode {
Expand All @@ -150,7 +150,7 @@ class CastDDNode: public ConstraintCastNode {
init_class_id(Class_CastDD);
}
virtual int Opcode() const;
virtual uint ideal_reg() const { return Op_RegD; }
virtual uint ideal_reg() const { return in(1)->ideal_reg(); }
};

class CastVVNode: public ConstraintCastNode {
Expand Down

1 comment on commit 89ade1d

@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.