From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 27285 invoked by alias); 16 Sep 2003 17:06:37 -0000 Mailing-List: contact gdb-patches-help@sources.redhat.com; run by ezmlm Precedence: bulk List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sources.redhat.com Received: (qmail 27272 invoked from network); 16 Sep 2003 17:06:36 -0000 Received: from unknown (HELO mx1.redhat.com) (66.187.233.31) by sources.redhat.com with SMTP; 16 Sep 2003 17:06:36 -0000 Received: from int-mx1.corp.redhat.com (int-mx1.corp.redhat.com [172.16.52.254]) by mx1.redhat.com (8.11.6/8.11.6) with ESMTP id h8GH6Z415264 for ; Tue, 16 Sep 2003 13:06:35 -0400 Received: from pobox.corp.redhat.com (pobox.corp.redhat.com [172.16.52.156]) by int-mx1.corp.redhat.com (8.11.6/8.11.6) with ESMTP id h8GH6Ye26452 for ; Tue, 16 Sep 2003 13:06:34 -0400 Received: from localhost.redhat.com (devserv.devel.redhat.com [172.16.58.1]) by pobox.corp.redhat.com (8.12.8/8.12.8) with ESMTP id h8GH6YxI019472 for ; Tue, 16 Sep 2003 13:06:34 -0400 Received: by localhost.redhat.com (Postfix, from userid 469) id 7D8342CA3B; Tue, 16 Sep 2003 13:15:47 -0400 (EDT) From: Elena Zannoni MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Transfer-Encoding: 7bit Message-ID: <16231.17730.828347.741094@localhost.redhat.com> Date: Tue, 16 Sep 2003 17:06:00 -0000 To: Corinna Vinschen Subject: Re: SH patch 2 (was Re: [RFA] SH: Deprecate deprecated functions, use new frame interface) In-Reply-To: <20030915160111.GA9283@cygbert.vinschen.de> References: <20030908165510.GK1859@cygbert.vinschen.de> <16226.1322.230352.450541@localhost.redhat.com> <20030915160111.GA9283@cygbert.vinschen.de> X-SW-Source: 2003-09/txt/msg00358.txt.bz2 Corinna Vinschen writes: > On Fri, Sep 12, 2003 at 01:40:58PM -0400, Elena Zannoni wrote: > > Seems ok, but....could you split the renaming of insn to inst and > > removal of tdep into a separate patch, and repost the 2 patches? > > I.e. the renaming/tdep and the real cleanups after that. It's not easy > > to see what's going on with this diff. > > > > thanks > > elena > > ...and this is patch two, removing old deprecated functionality in > favor of shiny new frame stuff. > > Corinna > > ChangeLog: > > * sh-tdep.c (struct frame_extra_info): Remove. > (struct sh_frame_cache): New structure. > (GET_SOURCE_REG): New macro extracting source register of an opcode. > (GET_TARGET_REG): Ditto but target register. > (GET_PUSHED_REG): Remove. > (IS_MOV_ARG_TO_REG): New macro. > (IS_MOV_ARG_TO_IND_R14): New macro. > (IS_MOV_ARG_TO_IND_R14_WITH_DISP): New macro. > (IS_MOVW_PCREL_TO_REG): New macro. > (IS_MOVL_PCREL_TO_REG): New macro. > (IS_SUB_REG_FROM_SP): New macro. > (IS_ARG_MOV): Remove. > (IS_MOV_TO_R14): Remove. > (IS_RESTORE_FP): New macro. > (IS_RTS): New macro. > (IS_LDS): New macro. > (IS_MOV_FP_SP): New macro. > (IS_ADD_REG_TO_FP): New macro. > (IS_ADD_IMM_FP): New macro. > (sh_skip_prologue_hard_way): Remove. > (sh_saved_pc_after_call): Remove. > (sh_frame_chain): Remove. > (sh_find_callers_reg): Remove. > (sh_nofp_frame_init_saved_regs): Remove. > (sh_fp_frame_init_saved_regs): Remove. > (sh_init_extra_frame_info): Remove. > (sh_analyze_prologue): New function. > (sh_skip_prologue): Remove deprecated code. Rely on new function > sh_analyze_prologue when after_prologue fails. > (sh_frame_saved_pc): Remove. > (sh_alloc_frame_cache): New function. > (sh_frame_cache): Ditto. > (sh_frame_prev_register): Ditto. > (sh_frame_this_id): Ditto. > (sh_frame_unwind): New structure defining the heuristic frame > sniffer interface. > (sh_frame_sniffer): New function. > (sh_unwind_sp): Ditto. > (sh_unwind_pc): Ditto. > (sh_unwind_dummy_id): Ditto. > (sh_frame_base_address): Ditto. > (sh_frame_base): New structure defining new frame base code. > (sh_in_function_epilogue_p): New function. > (sh_gdbarch_init): Restructure and simplify to eliminate deprecated > code and to call all new code instead. Initialize dwarf2 and > heuristic frame sniffer. It looks ok, I assume that the sh64 prologue analysis is unaffected, right? I.e. the macros that were used for decomposing the instructions in sh64 case are still there, etc etc. How are the test results? Did they change? elena