Message ID | AANLkTi=rLNyzCKTDJ+7JmxSG8yhsXgbTKFbjXyzxWZa5@mail.gmail.com |
---|---|
State | New |
Headers | show |
> ChangeLog gcc/ada/ > > 2010-12-31 Kai Tietz > > PR target/38662 > * gcc-interface/misc.c (gnat_type_hash_eq): > Allow METHOD_TYPE, too. No, Ada doesn't use METHOD_TYPE.
2010/12/31 Eric Botcazou <ebotcazou@adacore.com>: >> ChangeLog gcc/ada/ >> >> 2010-12-31 Kai Tietz >> >> PR target/38662 >> * gcc-interface/misc.c (gnat_type_hash_eq): >> Allow METHOD_TYPE, too. > > No, Ada doesn't use METHOD_TYPE. > > -- > Eric Botcazou > Good, I remove this part of the patch. I wasn't quite sure if there would be some use of METHOD_TYPE. As by grepping through ada's gcc tree I found some hits for it. Happy new year, Kai
On 12/31/2010 12:34 PM, Kai Tietz wrote: > Good, I remove this part of the patch. I wasn't quite sure if there > would be some use of METHOD_TYPE. As by grepping through ada's gcc > tree I found some hits for it. OK without the ada change. Jason
2011/1/1 Jason Merrill <jason@redhat.com>: > On 12/31/2010 12:34 PM, Kai Tietz wrote: >> >> Good, I remove this part of the patch. I wasn't quite sure if there >> would be some use of METHOD_TYPE. As by grepping through ada's gcc >> tree I found some hits for it. > > OK without the ada change. > > Jason > Applied at revision 168389. Thanks, Kai
Index: ada/gcc-interface/misc.c =================================================================== --- ada/gcc-interface/misc.c (revision 168365) +++ ada/gcc-interface/misc.c (working copy) @@ -529,7 +529,8 @@ static bool gnat_type_hash_eq (const_tree t1, const_tree t2) { - gcc_assert (TREE_CODE (t1) == FUNCTION_TYPE); + gcc_assert (TREE_CODE (t1) == FUNCTION_TYPE + || TREE_CODE (t1) == METHOD_TYPE); return fntype_same_flags_p (t1, TYPE_CI_CO_LIST (t2), TYPE_RETURN_UNCONSTRAINED_P (t2), TYPE_RETURN_BY_DIRECT_REF_P (t2),