From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 70157 invoked by alias); 27 Jul 2015 09:17:55 -0000 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 Received: (qmail 70144 invoked by uid 89); 27 Jul 2015 09:17:54 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-0.6 required=5.0 tests=AWL,BAYES_50,FREEMAIL_FROM,RCVD_IN_DNSWL_LOW,SPF_PASS autolearn=ham version=3.3.2 X-HELO: mail-ig0-f180.google.com Received: from mail-ig0-f180.google.com (HELO mail-ig0-f180.google.com) (209.85.213.180) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with (AES128-GCM-SHA256 encrypted) ESMTPS; Mon, 27 Jul 2015 09:17:53 +0000 Received: by iggf3 with SMTP id f3so68362527igg.1 for ; Mon, 27 Jul 2015 02:17:51 -0700 (PDT) MIME-Version: 1.0 X-Received: by 10.50.27.39 with SMTP id q7mr16241066igg.73.1437988671730; Mon, 27 Jul 2015 02:17:51 -0700 (PDT) Received: by 10.107.142.7 with HTTP; Mon, 27 Jul 2015 02:17:51 -0700 (PDT) In-Reply-To: <1437966615-21093-1-git-send-email-tbsaunde+gcc@tbsaunde.org> References: <1437966615-21093-1-git-send-email-tbsaunde+gcc@tbsaunde.org> Date: Mon, 27 Jul 2015 09:26:00 -0000 Message-ID: Subject: Re: [PATCH 0/9] start converting POINTER_SIZE to a hook From: Richard Biener To: tbsaunde+gcc@tbsaunde.org Cc: GCC Patches Content-Type: text/plain; charset=UTF-8 X-IsSubscribed: yes X-SW-Source: 2015-07/txt/msg02223.txt.bz2 On Mon, Jul 27, 2015 at 5:10 AM, wrote: > From: Trevor Saunders > > Hi, > > $subject. > > patches individually bootstrapped + regtested on x86_64-linux-gnu, and run > through config-list.mk with more patches removing usage of the macro. Ok? With POINTER_SIZE now being expensive (target hook) you might consider moving most users to use pointer_sized_int_node or some other global derived from POINTER_SIZE. Which of course raises the question of why we are hookizing this... if you'd want a truly switchable target you'd have to switch all global trees as well (or hookize them individually). Richard. > Trev > > Trevor Saunders (9): > remove POINTER_SIZE_UNITS macro > add pointer_size target hook > target.h: change to use targetm.pointer_size instead of POINTER_SIZE > varasm.c: switch from POINTER_SIZE to targetm.pointer_size () > ubsan.c: switch from POINTER_SIZE to targetm.pointer_size () > tree-chkp.c: switch to targetm.pointer_size () > stor-layout.c: switch to targetm.pointer_size () > tree.c: switch to targetm.pointer_size () > emit-rtl.c: switch to targetm.pointer_size () > > gcc/c-family/c-cppbuiltin.c | 2 +- > gcc/defaults.h | 3 --- > gcc/doc/tm.texi | 7 +++++++ > gcc/doc/tm.texi.in | 2 ++ > gcc/dwarf2asm.c | 4 ++-- > gcc/emit-rtl.c | 5 +++-- > gcc/lto/lto-object.c | 3 ++- > gcc/stor-layout.c | 9 +++++---- > gcc/target.def | 8 ++++++++ > gcc/target.h | 8 ++++++++ > gcc/targhooks.c | 8 ++++++++ > gcc/targhooks.h | 1 + > gcc/tree-chkp.c | 14 ++++++++------ > gcc/tree.c | 3 ++- > gcc/ubsan.c | 3 ++- > gcc/varasm.c | 12 ++++++------ > 16 files changed, 65 insertions(+), 27 deletions(-) > > -- > 2.4.0 >