Hello,

While working on something else, I noticed that debug_tree (vec), when
vec is a TREE_VEC, was crashing because TREE_NOTHROW was asserting that
its argument is not a TREE_VEC, so print_node would crash.

It turned out that TREE_NOTHROW was accidentally modified by this
change set:

commit 87d8f7b67c6a36c37e48e298f26e693520099b1e
Author: rguenth <rguenth@138bc75d-0d04-0410-961f-82ee72b054a4>
Date:   Tue Aug 21 10:03:38 2012 +0000

    2012-08-21  Richard Guenther  <rguent...@suse.de>

        cp/
        * cp-tree.h (TREE_INDIRECT_USING): Use TREE_LANG_FLAG_0 accessor.
        (ATTR_IS_DEPENDENT): Likewise.
        (ARGUMENT_PACK_INCOMPLETE_P): Use TREE_ADDRESSABLE instead of
        TREE_LANG_FLAG_0 on TREE_VECs.

        * tree.h (struct tree_base): Add union to make it possible to
        re-use the upper 4 bytes for tree codes that do not need as
        many flags as others.  Move visited and default_def_flag to
        common bits section in exchange for saturating_flag and
        unsigned_flag.  Add SSA name version and tree vec length
        fields here.
        (struct tree_vec): Remove length field here.
        (struct tree_ssa_name): Remove version field here.

    git-svn-id: svn+ssh://gcc.gnu.org/svn/gcc/trunk@190558 
138bc75d-0d04-0410-961f-82ee72b054a4

Richard Guenther pre-approved the reverting of the TREE_NOTHROW change
line.

Tested on x86_64-unknown-linux-gnu against master and applied to
trunk.

gcc/
        * tree.h (TREE_NOTHROW): Use the base.nothrow_flag.

git-svn-id: svn+ssh://gcc.gnu.org/svn/gcc/trunk@190595 
138bc75d-0d04-0410-961f-82ee72b054a4
---
 gcc/ChangeLog |    4 ++++
 gcc/tree.h    |    2 +-
 2 files changed, 5 insertions(+), 1 deletions(-)

diff --git a/gcc/tree.h b/gcc/tree.h
index f332fdd..bca0576 100644
--- a/gcc/tree.h
+++ b/gcc/tree.h
@@ -1305,7 +1305,7 @@ extern void omp_clause_range_check_failed (const_tree, 
const char *, int,
    throw an exception.  In a CALL_EXPR, nonzero means the call cannot
    throw.  We can't easily check the node type here as the C++
    frontend also uses this flag (for AGGR_INIT_EXPR).  */
-#define TREE_NOTHROW(NODE) (TREE_NOT_CHECK (NODE, TREE_VEC)->base.nothrow_flag)
+#define TREE_NOTHROW(NODE) ((NODE)->base.nothrow_flag)
 
 /* In a CALL_EXPR, means that it's safe to use the target of the call
    expansion as the return slot for a call that returns in memory.  */
-- 
                Dodji

Reply via email to