From patchwork Wed Jun 1 20:00:17 2011 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Diego Novillo X-Patchwork-Id: 98256 Return-Path: X-Original-To: incoming@patchwork.ozlabs.org Delivered-To: patchwork-incoming@bilbo.ozlabs.org Received: from sourceware.org (server1.sourceware.org [209.132.180.131]) by ozlabs.org (Postfix) with SMTP id 819EAB6F97 for ; Thu, 2 Jun 2011 06:00:54 +1000 (EST) Received: (qmail 2582 invoked by alias); 1 Jun 2011 20:00:52 -0000 Received: (qmail 2573 invoked by uid 22791); 1 Jun 2011 20:00:51 -0000 X-SWARE-Spam-Status: No, hits=-2.1 required=5.0 tests=AWL, BAYES_00, DKIM_SIGNED, DKIM_VALID, DKIM_VALID_AU, SPF_HELO_PASS, T_RP_MATCHES_RCVD X-Spam-Check-By: sourceware.org Received: from smtp-out.google.com (HELO smtp-out.google.com) (216.239.44.51) by sourceware.org (qpsmtpd/0.43rc1) with ESMTP; Wed, 01 Jun 2011 20:00:37 +0000 Received: from wpaz5.hot.corp.google.com (wpaz5.hot.corp.google.com [172.24.198.69]) by smtp-out.google.com with ESMTP id p51K0PZH028668; Wed, 1 Jun 2011 13:00:30 -0700 Received: from topo.tor.corp.google.com (topo.tor.corp.google.com [172.29.41.2]) by wpaz5.hot.corp.google.com with ESMTP id p51K0OQw028203; Wed, 1 Jun 2011 13:00:24 -0700 Received: by topo.tor.corp.google.com (Postfix, from userid 54752) id 18E401DA1C9; Wed, 1 Jun 2011 16:00:17 -0400 (EDT) To: reply@codereview.appspotmail.com, rguenther@suse.de, gcc-patches@gcc.gnu.org Subject: [lto] Remove unnecessary assertion for DECL_SAVED_TREE (issue4530094) Message-Id: <20110601200018.18E401DA1C9@topo.tor.corp.google.com> Date: Wed, 1 Jun 2011 16:00:17 -0400 (EDT) From: dnovillo@google.com (Diego Novillo) X-System-Of-Record: true X-IsSubscribed: yes Mailing-List: contact gcc-patches-help@gcc.gnu.org; run by ezmlm Precedence: bulk List-Id: List-Unsubscribe: List-Archive: List-Post: List-Help: Sender: gcc-patches-owner@gcc.gnu.org Delivered-To: mailing list gcc-patches@gcc.gnu.org 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. --- This patch is available for review at http://codereview.appspot.com/4530094 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); }