From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 84600 invoked by alias); 1 Sep 2015 22:43:22 -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 84589 invoked by uid 89); 1 Sep 2015 22:43:21 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-2.3 required=5.0 tests=AWL,BAYES_00,RCVD_IN_DNSWL_LOW,SPF_PASS,T_RP_MATCHES_RCVD autolearn=ham version=3.3.2 X-HELO: mailapp01.imgtec.com Received: from mailapp01.imgtec.com (HELO mailapp01.imgtec.com) (195.59.15.196) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Tue, 01 Sep 2015 22:43:20 +0000 Received: from KLMAIL01.kl.imgtec.org (unknown [192.168.5.35]) by Websense Email Security Gateway with ESMTPS id 260D2101E46A0; Tue, 1 Sep 2015 23:43:14 +0100 (IST) Received: from LEMAIL01.le.imgtec.org (192.168.152.62) by KLMAIL01.kl.imgtec.org (192.168.5.35) with Microsoft SMTP Server (TLS) id 14.3.195.1; Tue, 1 Sep 2015 23:43:17 +0100 Received: from LEMAIL01.le.imgtec.org ([fe80::5ae:ee16:f4b9:cda9]) by LEMAIL01.le.imgtec.org ([fe80::5ae:ee16:f4b9:cda9%17]) with mapi id 14.03.0210.002; Tue, 1 Sep 2015 23:43:17 +0100 From: Matthew Fortune To: "Moore, Catherine" , "'gcc-patches@gcc.gnu.org' (gcc-patches@gcc.gnu.org)" Subject: RE: [PATCH, MIPS] Compact branch support for MIPS32R6/MIPS64R6 Date: Tue, 01 Sep 2015 22:43:00 -0000 Message-ID: <6D39441BF12EF246A7ABCE6654B023532125E01C@LEMAIL01.le.imgtec.org> References: <6D39441BF12EF246A7ABCE6654B02353211F2232@LEMAIL01.le.imgtec.org> <6D39441BF12EF246A7ABCE6654B023532122F2E0@LEMAIL01.le.imgtec.org> In-Reply-To: Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 X-IsSubscribed: yes X-SW-Source: 2015-09/txt/msg00104.txt.bz2 Moore, Catherine writes: > Hi Matthew: >=20 > > -----Original Message----- > > From: Matthew Fortune [mailto:Matthew.Fortune@imgtec.com] > > Sent: Monday, August 17, 2015 6:47 PM > > To: Moore, Catherine; 'gcc-patches@gcc.gnu.org' (gcc-patches@gcc.gnu.or= g) > > Subject: RE: [PATCH, MIPS] Compact branch support for MIPS32R6/MIPS64R6 > > >=20 > One comment on the updated patch: >=20 > > diff --git a/gcc/doc/invoke.texi b/gcc/doc/invoke.texi index > > 27be317..d7d9586 100644 > > --- a/gcc/doc/invoke.texi > > +++ b/gcc/doc/invoke.texi > > @@ -781,6 +781,7 @@ Objective-C and Objective-C++ Dialects}. > > -mgp32 -mgp64 -mfp32 -mfpxx -mfp64 -mhard-float -msoft-float @go= l - > > mno-float -msingle-float -mdouble-float @gol -modd-spreg -mno-odd- > > spreg @gol > > +-mcompact-branches=3D@var{policy} @gol > > -mabs=3D@var{mode} -mnan=3D@var{encoding} @gol -mdsp -mno-dsp - > > mdspr2 -mno-dspr2 @gol -mmcu -mmno-mcu @gol @@ -17387,6 +17388,27 > > @@ for the o32 ABI. This is the default for processors that are known = to > > support these registers. When using the o32 FPXX ABI, @option{-mno-odd- > > spreg} is set by default. > > > > +@item -mcompact-branches=3Dnever > > +@itemx -mcompact-branches=3Doptimal > > +@itemx -mcompact-branches=3Dalways > > +@opindex mcompact-branches=3Dnever > > +@opindex mcompact-branches=3Doptimal > > +@opindex mcompact-branches=3Dalways > > +These options control which form of branches will be generated. The > > +default is @option{-mcompact-branches=3Doptimal}. > > + > > +The @option{-mcompact-branches=3Dnever} option >=20 > Change: > ensures that no compact branch instructions are generated. > > + > To: > ensures that compact branch instructions will never be generated. >=20 > > +The @option{-mcompact-branches=3Dalways} option >=20 > Change: > ensures that only compact branch instructions are used unless there is o= nly a delay slot > form of a branch. >=20 > To: > ensures that a compact branch instruction will be generated if availab= le. If a compact > branch instruction is not available, a delay slot form of the branch will= be used instead. >=20 > This option is supported from MIPS Release 6 onwards. >=20 > > +The @option{-mcompact-branches=3Doptimal} option >=20 > Change: > will use delay slot > > +branches if available in the current ISA and the delay slot filler > > +successfully fills a delay slot. Otherwise, a compact branch will be > > +used if available. > > + > To: > Will cause a delay slot branch to be used if one is available in the curr= ent ISA and the > delay slot is successfully filled. > If the delay slot is not filled, a compact branch will be chosen if one = is available. All done and committed as r227385. Thanks, Matthew