From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 18189 invoked by alias); 5 Mar 2008 21:24:07 -0000 Received: (qmail 18181 invoked by uid 22791); 5 Mar 2008 21:24:06 -0000 X-Spam-Check-By: sourceware.org Received: from smtp-out.google.com (HELO smtp-out.google.com) (216.239.33.17) by sourceware.org (qpsmtpd/0.31) with ESMTP; Wed, 05 Mar 2008 21:23:49 +0000 Received: from zps38.corp.google.com (zps38.corp.google.com [172.25.146.38]) by smtp-out.google.com with ESMTP id m25LNd64006574; Wed, 5 Mar 2008 21:23:39 GMT Received: from localhost.localdomain.google.com (dhcp-172-18-118-183.corp.google.com [172.18.118.183]) (authenticated bits=0) by zps38.corp.google.com with ESMTP id m25LNbBM000734 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=NOT); Wed, 5 Mar 2008 13:23:38 -0800 To: Kenneth Zadeck Cc: gcc-patches , "Park, Seongbae" , "Bonzini, Paolo" , Steven Bosscher , Richard Sandiford Subject: Re: [trunk] first of three patches to get rid of no conflict blocks. References: <47CDB6AF.9090707@naturalbridge.com> <47CDB78B.1020100@naturalbridge.com> From: Ian Lance Taylor Date: Wed, 05 Mar 2008 21:24:00 -0000 In-Reply-To: <47CDB78B.1020100@naturalbridge.com> Message-ID: User-Agent: Gnus/5.09 (Gnus v5.9.0) Emacs/21.4 MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii X-IsSubscribed: yes Mailing-List: contact gcc-patches-help@gcc.gnu.org; run by ezmlm Precedence: bulk List-Id: List-Archive: List-Post: List-Help: Sender: gcc-patches-owner@gcc.gnu.org X-SW-Source: 2008-03/txt/msg00364.txt.bz2 Kenneth Zadeck writes: > 2008-03-04 Kenneth Zadeck > > * fwprop.c (update_df): Support width and offset parameters of > df_ref_create. > * ra-conflict.c (mark_reg_store, clear_reg_in_live, > global_conflicts): Change DF_REF_EXTRACT to either > DF_REF_ZERO_EXTRACT or DF_REF_SIGN_EXTRACT. Change > DF_REF_STRICT_LOWER_PART to DF_REF_STRICT_LOW_PART. > * df-scan.c (df_ref_record, df_defs_record, > df_ref_create_structure, df_def_record_1, df_uses_record, > df_get_conditional_uses, df_get_call_refs, df_insn_refs_collect, > df_bb_refs_collect, df_entry_block_defs_collect, > df_exit_block_uses_collect): Support new width and offset fields. > (ref_extract_pool): New storage pool. > (df_free_ref): New function. > (df_reg_chain_unlink, df_free_collection_rec, > df_sort_and_compress_refs): Call df_free_ref. > (df_ref_equal_p, df_ref_compare): Compare offset and width fields > of df_ref_extract. > (df_ref_create_structure): Allocate df_ref_extract if offset and > width fields are used. > (df_def_record_1): Get offset and width from ZERO_EXTRACT. > (df_uses_record): Get offset and width from ZERO_EXTRACT > and SIGN_EXTRACT. > * global.c (build_insn_chain): Change DF_REF_EXTRACT to either > DF_REF_ZERO_EXTRACT or DF_REF_SIGN_EXTRACT. Change > DF_REF_STRICT_LOWER_PART to DF_REF_STRICT_LOW_PART. > * df.h (df_ref_flags): Change DF_REF_EXTRACT to either > DF_REF_ZERO_EXTRACT or DF_REF_SIGN_EXTRACT. Change > DF_REF_STRICT_LOWER_PART to DF_REF_STRICT_LOW_PART. > (df_ref_extract): New structure. > (DF_REF_WIDTH, DF_REF_OFFSET): New macros. > (df_ref_create): Add width and offset parameters. This looks OK to me. Thanks. Ian