Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(1826)

Issue 4530094: [lto] Remove unnecessary assertion for DECL_SAVED_TREE (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
12 years, 11 months ago by Diego Novillo
Modified:
12 years, 11 months ago
Reviewers:
CC:
rguenther_suse.de, gcc-patches_gcc.gnu.org
Visibility:
Public.

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+0 lines, -5 lines) Patch
M gcc/lto-streamer-out.c View 1 chunk +0 lines, -5 lines 0 comments Download

Messages

Total messages: 1
Diego Novillo
12 years, 11 months ago (2011-06-01 20:00:36 UTC) #1
This patchlet removes the assertion that DECL_SAVED_TREE should
be NULL.

As we discussed in http://gcc.gnu.org/ml/gcc-patches/2011-06/msg00004.html,
it is no longer necessary.

Committed to trunk.


Diego.


	* lto-streamer-out.c (lto_output_ts_decl_non_common_tree_pointers):
	Remove assertion for DECL_SAVED_TREE in FUNCTION_DECL nodes.

diff --git a/gcc/lto-streamer-out.c b/gcc/lto-streamer-out.c
index 7f3217b..3d42483 100644
--- a/gcc/lto-streamer-out.c
+++ b/gcc/lto-streamer-out.c
@@ -931,11 +931,6 @@ lto_output_ts_decl_non_common_tree_pointers (struct
output_block *ob,
 {
   if (TREE_CODE (expr) == FUNCTION_DECL)
     {
-      /* DECL_SAVED_TREE holds the GENERIC representation for DECL.
-	 At this point, it should not exist.  Either because it was
-	 converted to gimple or because DECL didn't have a GENERIC
-	 representation in this TU.  */
-      gcc_assert (DECL_SAVED_TREE (expr) == NULL_TREE);
       lto_output_tree_or_ref (ob, DECL_ARGUMENTS (expr), ref_p);
       lto_output_tree_or_ref (ob, DECL_RESULT (expr), ref_p);
     }

--
This patch is available for review at http://codereview.appspot.com/4530094
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b