d: Fix ICE: in verify_gimple_in_seq on powerpc-darwin9 [PR112270]
This ICE was seen during stage2 on powerpc-darwin9 only. There were still some uses of GCC's boolean_type_node in the D front-end, which caused a type mismatch to trigger as D bool size is fixed to 1 byte on all targets. So two new nodes have been introduced - d_bool_false_node and d_bool_true_node - which have replaced all remaining uses of boolean_false_node and boolean_true_node respectively. PR d/112270 gcc/d/ChangeLog: * d-builtins.cc (d_build_d_type_nodes): Initialize d_bool_false_node, d_bool_true_node. * d-codegen.cc (build_array_struct_comparison): Use d_bool_false_node instead of boolean_false_node. * d-convert.cc (d_truthvalue_conversion): Use d_bool_false_node and d_bool_true_node instead of boolean_false_node and boolean_true_node. * d-tree.h (enum d_tree_index): Add DTI_BOOL_FALSE and DTI_BOOL_TRUE. (d_bool_false_node): New macro. (d_bool_true_node): New macro. * modules.cc (build_dso_cdtor_fn): Use d_bool_false_node and d_bool_true_node instead of boolean_false_node and boolean_true_node. (register_moduleinfo): Use d_bool_type instead of boolean_type_node. gcc/testsuite/ChangeLog: * gdc.dg/pr112270.d: New test.
This commit is contained in:
parent
5d2a360f0a
commit
10f1489dcb
6 changed files with 24 additions and 8 deletions
|
@ -956,6 +956,9 @@ d_build_d_type_nodes (void)
|
|||
d_bool_type = make_unsigned_type (1);
|
||||
TREE_SET_CODE (d_bool_type, BOOLEAN_TYPE);
|
||||
|
||||
d_bool_false_node = TYPE_MIN_VALUE (d_bool_type);
|
||||
d_bool_true_node = TYPE_MAX_VALUE (d_bool_type);
|
||||
|
||||
char8_type_node = make_unsigned_type (8);
|
||||
TYPE_STRING_FLAG (char8_type_node) = 1;
|
||||
|
||||
|
|
|
@ -1115,7 +1115,7 @@ build_array_struct_comparison (tree_code code, StructDeclaration *sd,
|
|||
if (length == 0 || result OP 0) break; */
|
||||
t = build_boolop (EQ_EXPR, length, d_convert (lentype, integer_zero_node));
|
||||
t = build_boolop (TRUTH_ORIF_EXPR, t, build_boolop (code, result,
|
||||
boolean_false_node));
|
||||
d_bool_false_node));
|
||||
t = build1 (EXIT_EXPR, void_type_node, t);
|
||||
add_stmt (t);
|
||||
|
||||
|
|
|
@ -132,13 +132,13 @@ d_truthvalue_conversion (tree expr)
|
|||
return expr;
|
||||
|
||||
case INTEGER_CST:
|
||||
return integer_zerop (expr) ? boolean_false_node
|
||||
: boolean_true_node;
|
||||
return integer_zerop (expr) ? d_bool_false_node
|
||||
: d_bool_true_node;
|
||||
|
||||
case REAL_CST:
|
||||
return real_compare (NE_EXPR, &TREE_REAL_CST (expr), &dconst0)
|
||||
? boolean_true_node
|
||||
: boolean_false_node;
|
||||
? d_bool_true_node
|
||||
: d_bool_false_node;
|
||||
|
||||
case ADDR_EXPR:
|
||||
/* If we are taking the address of a decl that can never be null,
|
||||
|
@ -148,7 +148,7 @@ d_truthvalue_conversion (tree expr)
|
|||
warning (OPT_Waddress,
|
||||
"the address of %qD will always evaluate as %<true%>",
|
||||
TREE_OPERAND (expr, 0));
|
||||
return boolean_true_node;
|
||||
return d_bool_true_node;
|
||||
}
|
||||
break;
|
||||
|
||||
|
|
|
@ -444,6 +444,9 @@ enum d_tree_index
|
|||
DTI_NULL_ARRAY,
|
||||
DTI_BOTTOM_TYPE,
|
||||
|
||||
DTI_BOOL_FALSE,
|
||||
DTI_BOOL_TRUE,
|
||||
|
||||
DTI_MAX
|
||||
};
|
||||
|
||||
|
@ -480,6 +483,9 @@ extern GTY(()) tree d_global_trees[DTI_MAX];
|
|||
#define null_array_node d_global_trees[DTI_NULL_ARRAY]
|
||||
/* The bottom type, referred to as `noreturn` in code. */
|
||||
#define noreturn_type_node d_global_trees[DTI_BOTTOM_TYPE]
|
||||
/* D boolean values are always byte-sized, unlike boolean_type_node. */
|
||||
#define d_bool_false_node d_global_trees[DTI_BOOL_FALSE]
|
||||
#define d_bool_true_node d_global_trees[DTI_BOOL_TRUE]
|
||||
|
||||
/* A prefix for internal variables, which are not user-visible. */
|
||||
#if !defined (NO_DOT_IN_LABEL)
|
||||
|
|
|
@ -330,7 +330,7 @@ static tree
|
|||
build_dso_cdtor_fn (bool ctor_p)
|
||||
{
|
||||
const char *name = ctor_p ? GDC_PREFIX ("dso_ctor") : GDC_PREFIX ("dso_dtor");
|
||||
tree condition = ctor_p ? boolean_true_node : boolean_false_node;
|
||||
tree condition = ctor_p ? d_bool_true_node : d_bool_false_node;
|
||||
|
||||
/* Declaration of dso_ctor/dso_dtor is:
|
||||
|
||||
|
@ -453,7 +453,7 @@ register_moduleinfo (Module *decl, tree minfo)
|
|||
d_finish_decl (dso_slot_node);
|
||||
|
||||
dso_initialized_node = build_dso_registry_var (GDC_PREFIX ("dso_initialized"),
|
||||
boolean_type_node);
|
||||
d_bool_type);
|
||||
d_finish_decl (dso_initialized_node);
|
||||
|
||||
/* Declare dso_ctor() and dso_dtor(). */
|
||||
|
|
7
gcc/testsuite/gdc.dg/pr112270.d
Normal file
7
gcc/testsuite/gdc.dg/pr112270.d
Normal file
|
@ -0,0 +1,7 @@
|
|||
// https://gcc.gnu.org/bugzilla/show_bug.cgi?id=112270
|
||||
// { dg-do compile }
|
||||
class CPPNamespaceDeclaration { }
|
||||
bool isNamespaceEqual (CPPNamespaceDeclaration a)
|
||||
{
|
||||
return a ? true : isNamespaceEqual(a);
|
||||
}
|
Loading…
Add table
Reference in a new issue