public inbox for gcc-bugs@sourceware.org
help / color / mirror / Atom feed
From: "rguenther at suse dot de" <gcc-bugzilla@gcc.gnu.org>
To: gcc-bugs@gcc.gnu.org
Subject: [Bug lto/52178] [4.7 regression] Ada bootstrap failure in LTO mode
Date: Mon, 13 Feb 2012 12:29:00 -0000 [thread overview]
Message-ID: <bug-52178-4-JFNl2cyVGg@http.gcc.gnu.org/bugzilla/> (raw)
In-Reply-To: <bug-52178-4@http.gcc.gnu.org/bugzilla/>
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=52178
--- Comment #7 from rguenther at suse dot de <rguenther at suse dot de> 2012-02-13 12:29:36 UTC ---
On Mon, 13 Feb 2012, ebotcazou at gcc dot gnu.org wrote:
> http://gcc.gnu.org/bugzilla/show_bug.cgi?id=52178
>
> --- Comment #6 from Eric Botcazou <ebotcazou at gcc dot gnu.org> 2012-02-13 12:17:46 UTC ---
> > Sth I'm not very familiar is the QUAL_UNION record kinds - maybe you
> > can eye the two merging machineries for obvious errors here?
>
> I did, and only came up with the following no-op patchlet:
>
> Index: gimple.c
> ===================================================================
> --- gimple.c (revision 184143)
> +++ gimple.c (working copy)
> @@ -4140,9 +4140,7 @@ iterative_hash_gimple_type (tree type, h
> v = iterative_hash_hashval_t (na, v);
> }
>
> - if (TREE_CODE (type) == RECORD_TYPE
> - || TREE_CODE (type) == UNION_TYPE
> - || TREE_CODE (type) == QUAL_UNION_TYPE)
> + if (RECORD_OR_UNION_TYPE_P (type))
> {
> unsigned nf;
> tree f;
> @@ -4373,9 +4371,7 @@ iterative_hash_canonical_type (tree type
> v = iterative_hash_hashval_t (na, v);
> }
>
> - if (TREE_CODE (type) == RECORD_TYPE
> - || TREE_CODE (type) == UNION_TYPE
> - || TREE_CODE (type) == QUAL_UNION_TYPE)
> + if (RECORD_OR_UNION_TYPE_P (type))
> {
> unsigned nf;
> tree f;
I see. I came up with the following, but it's unlikely to make a
difference either
Index: gcc/gimple.c
===================================================================
--- gcc/gimple.c (revision 184151)
+++ gcc/gimple.c (working copy)
@@ -3549,10 +3549,6 @@ gimple_types_compatible_p_1 (tree t1, tr
goto different_types;
}
- /* If their attributes are not the same they can't be the same type.
*/
- if (!attribute_list_equal (TYPE_ATTRIBUTES (t1), TYPE_ATTRIBUTES (t2)))
- goto different_types;
-
/* Do type-specific comparisons. */
switch (TREE_CODE (t1))
{
@@ -4589,10 +4585,6 @@ gimple_canonical_types_compatible_p (tre
return true;
}
- /* If their attributes are not the same they can't be the same type.
*/
- if (!attribute_list_equal (TYPE_ATTRIBUTES (t1), TYPE_ATTRIBUTES (t2)))
- return false;
-
/* Do type-specific comparisons. */
switch (TREE_CODE (t1))
{
@@ -4646,13 +4638,6 @@ gimple_canonical_types_compatible_p (tre
}
case METHOD_TYPE:
- /* Method types should belong to the same class. */
- if (!gimple_canonical_types_compatible_p
- (TYPE_METHOD_BASETYPE (t1), TYPE_METHOD_BASETYPE (t2)))
- return false;
-
- /* Fallthru */
-
case FUNCTION_TYPE:
/* Function types are the same if the return type and arguments
types
are the same. */
next prev parent reply other threads:[~2012-02-13 12:29 UTC|newest]
Thread overview: 27+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-02-08 21:37 [Bug lto/52178] New: " ebotcazou at gcc dot gnu.org
2012-02-10 15:10 ` [Bug lto/52178] " rguenth at gcc dot gnu.org
2012-02-10 17:51 ` ebotcazou at gcc dot gnu.org
2012-02-13 11:35 ` rguenth at gcc dot gnu.org
2012-02-13 11:54 ` ebotcazou at gcc dot gnu.org
2012-02-13 11:58 ` rguenther at suse dot de
2012-02-13 12:18 ` ebotcazou at gcc dot gnu.org
2012-02-13 12:29 ` rguenther at suse dot de [this message]
2012-02-13 12:39 ` ebotcazou at gcc dot gnu.org
2012-02-13 13:29 ` rguenth at gcc dot gnu.org
2012-02-13 13:43 ` rguenth at gcc dot gnu.org
2012-02-13 13:45 ` rguenth at gcc dot gnu.org
2012-02-13 13:53 ` ebotcazou at gcc dot gnu.org
2012-02-13 13:54 ` ebotcazou at gcc dot gnu.org
2012-02-13 13:56 ` ebotcazou at gcc dot gnu.org
2012-02-13 14:04 ` rguenther at suse dot de
2012-02-13 14:13 ` ebotcazou at gcc dot gnu.org
2012-02-13 14:14 ` rguenther at suse dot de
2012-02-13 14:19 ` rguenther at suse dot de
2012-02-13 14:22 ` ebotcazou at gcc dot gnu.org
2012-02-13 14:44 ` rguenth at gcc dot gnu.org
2012-02-13 17:07 ` ebotcazou at gcc dot gnu.org
2012-02-14 8:41 ` rguenth at gcc dot gnu.org
2012-02-15 0:11 ` ebotcazou at gcc dot gnu.org
2012-02-15 0:12 ` ebotcazou at gcc dot gnu.org
2012-05-25 20:28 ` ebotcazou at gcc dot gnu.org
2012-05-25 20:30 ` ebotcazou at gcc dot gnu.org
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=bug-52178-4-JFNl2cyVGg@http.gcc.gnu.org/bugzilla/ \
--to=gcc-bugzilla@gcc.gnu.org \
--cc=gcc-bugs@gcc.gnu.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).