From patchwork Sun Feb 26 12:48:10 2012 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Oleg Endo X-Patchwork-Id: 143082 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 DD49AB6FBD for ; Sun, 26 Feb 2012 23:48:40 +1100 (EST) Comment: DKIM? See http://www.dkim.org DKIM-Signature: v=1; a=rsa-sha1; c=relaxed/relaxed; d=gcc.gnu.org; s=default; x=1330865322; h=Comment: DomainKey-Signature:Received:Received:Received:Received:Received: Subject:From:To:Content-Type:Date:Message-ID:Mime-Version: Mailing-List:Precedence:List-Id:List-Unsubscribe:List-Archive: List-Post:List-Help:Sender:Delivered-To; bh=w3A6YVWZProxExX701qj 1Iev6XU=; b=M9aciC7QVZsIY0kD5lTd99v+X0181bPCdc7XLMwf0eIKFAILWRTb JjHf3Sty1pmE9IdKUf/m4qSOuvUGX7tnlFRvOg3I85P2hiBo1jxY/PQtTPD7t+a8 dI554U7G3JTe7QoJZiENHFvYe0ERE1xxNyvp7BxMqemJ3w3JPuC1gW8= Comment: DomainKeys? See http://antispam.yahoo.com/domainkeys DomainKey-Signature: a=rsa-sha1; q=dns; c=nofws; s=default; d=gcc.gnu.org; h=Received:Received:X-SWARE-Spam-Status:X-Spam-Check-By:Received:Received:Received:Subject:From:To:Content-Type:Date:Message-ID:Mime-Version:X-IsSubscribed:Mailing-List:Precedence:List-Id:List-Unsubscribe:List-Archive:List-Post:List-Help:Sender:Delivered-To; b=ZShq9XplbBSJjDGtXqvUvnncRADmVnui/T4RAxvmmVFsr6zJ7YLm2ECWANb02Z a6VV+HlNU9N4cU1wshJGxSdPqvR/O96dC3XOhaJRZVThMTj6b09+mzC/Qa2eF0Fk a6Lrhc7a7E9mVulUPF8IivfOoZ5qjwuQEzh7/4tQLNcx8=; Received: (qmail 12000 invoked by alias); 26 Feb 2012 12:48:32 -0000 Received: (qmail 11970 invoked by uid 22791); 26 Feb 2012 12:48:30 -0000 X-SWARE-Spam-Status: No, hits=-2.6 required=5.0 tests=AWL, BAYES_00, RCVD_IN_DNSWL_NONE, T_RP_MATCHES_RCVD, UNPARSEABLE_RELAY X-Spam-Check-By: sourceware.org Received: from mailout10.t-online.de (HELO mailout10.t-online.de) (194.25.134.21) by sourceware.org (qpsmtpd/0.43rc1) with ESMTP; Sun, 26 Feb 2012 12:48:15 +0000 Received: from fwd09.aul.t-online.de (fwd09.aul.t-online.de ) by mailout10.t-online.de with smtp id 1S1dWP-0005qI-0q; Sun, 26 Feb 2012 13:48:13 +0100 Received: from [192.168.0.104] (ZqeKWsZvohqYRZZ8ltLPrDBMvdwg7W43cY800TfF1lY++7JiRwRZBpMVCFg3cOvZBD@[87.157.42.20]) by fwd09.t-online.de with esmtp id 1S1dWN-03A3s00; Sun, 26 Feb 2012 13:48:11 +0100 Subject: [patch committed SH] Fix some comment typos and formattings From: Oleg Endo To: gcc-patches Date: Sun, 26 Feb 2012 13:48:10 +0100 Message-ID: <1330260490.2929.45.camel@yam-132-YW-E178-FTW> Mime-Version: 1.0 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 Hello, I've just committed the attached patch as obvious. Cheers, Oleg 2012-02-26 Oleg Endo * config/sh/predicates.md: Remove blank lines. * config/sh/sh.c: Fix typos in comments. * config/sh/constraints.md: Likewise. * config/sh/sh.md: Remove blank lines. Fix typos in comments. Use ;; as comment characters. Index: gcc/config/sh/predicates.md =================================================================== --- gcc/config/sh/predicates.md (revision 184582) +++ gcc/config/sh/predicates.md (working copy) @@ -448,7 +448,6 @@ return general_operand (op, mode); }) - ;; Returns 1 if OP is a POST_INC on stack pointer register. (define_predicate "sh_no_delay_pop_operand" @@ -466,7 +465,6 @@ return 0; }) - ;; Returns 1 if OP is a MEM that can be source of a simple move operation. (define_predicate "unaligned_load_operand" Index: gcc/config/sh/sh.c =================================================================== --- gcc/config/sh/sh.c (revision 184582) +++ gcc/config/sh/sh.c (working copy) @@ -1850,7 +1850,7 @@ } /* ??? How should we distribute probabilities when more than one branch - is generated. So far we only have soem ad-hoc observations: + is generated. So far we only have some ad-hoc observations: - If the operands are random, they are likely to differ in both parts. - If comparing items in a hash chain, the operands are random or equal; operation should be EQ or NE. @@ -5380,7 +5380,7 @@ /* If relaxing, generate pseudo-ops to associate function calls with the symbols they call. It does no harm to not generate these - pseudo-ops. However, when we can generate them, it enables to + pseudo-ops. However, when we can generate them, it enables the linker to potentially relax the jsr to a bsr, and eliminate the register load and, possibly, the constant pool entry. */ @@ -9259,7 +9259,7 @@ #if 0 /* If this is a label that existed before reload, then the register - if dead here. However, if this is a label added by reorg, then + is dead here. However, if this is a label added by reorg, then the register may still be live here. We can't tell the difference, so we just ignore labels completely. */ if (code == CODE_LABEL) @@ -9569,7 +9569,7 @@ { int size; - /* Check if this the address of an unaligned load / store. */ + /* Check if this is the address of an unaligned load / store. */ if (mode == VOIDmode) return CONST_OK_FOR_I06 (INTVAL (op)); Index: gcc/config/sh/constraints.md =================================================================== --- gcc/config/sh/constraints.md (revision 184582) +++ gcc/config/sh/constraints.md (working copy) @@ -139,7 +139,7 @@ (match_test "ival >= 0 && ival <= 255"))) (define_constraint "K12" - "An unsigned 8-bit constant, as used in SH2A 12-bit display." + "An unsigned 8-bit constant, as used in SH2A 12-bit displacement addressing." (and (match_code "const_int") (match_test "ival >= 0 && ival <= 4095"))) Index: gcc/config/sh/sh.md =================================================================== --- gcc/config/sh/sh.md (revision 184582) +++ gcc/config/sh/sh.md (working copy) @@ -4405,8 +4405,6 @@ "sub r63, %1, %0" [(set_attr "type" "arith_media")]) - - ;; Don't expand immediately because otherwise neg:DI (abs:DI) will not be ;; combined. (define_expand "negdi2" @@ -4497,7 +4495,6 @@ DONE; }") - ;; The SH4 202 can do zero-offset branches without pipeline stalls. ;; This can be used as some kind of conditional execution, which is useful ;; for abs. @@ -5468,7 +5465,7 @@ operands[3] = gen_rtx_REG (DImode, REGNO (operands[2])); }") -/* When storing r0, we have to avoid reg+reg addressing. */ +;; When storing r0, we have to avoid reg+reg addressing. (define_insn "movhi_i" [(set (match_operand:HI 0 "general_movdst_operand" "=r,r,r,r,m,r,l,r") (match_operand:HI 1 "general_movsrc_operand" "Q,rI08,m,t,r,l,r,i"))] @@ -7472,7 +7469,7 @@ (set_attr "fp_set" "unknown")]) ;; This is TBR relative jump instruction for SH2A architecture. -;; Its use is enabled assigning an attribute "function_vector" +;; Its use is enabled by assigning an attribute "function_vector" ;; and the vector number to a function during its declaration. (define_insn "call_valuei_tbr_rel" @@ -9581,8 +9578,6 @@ DONE; ") - - ;; sne moves the complement of the T reg to DEST like this: ;; cmp/eq ... ;; mov #-1,temp @@ -9605,7 +9600,6 @@ operands[1] = gen_reg_rtx (SImode); }") - ;; Recognize mov #-1/negc/neg sequence, and change it to movt/add #-1. ;; This prevents a regression that occurred when we switched from xor to ;; mov/neg for sne. @@ -9659,7 +9653,6 @@ DONE; ") - ;; ------------------------------------------------------------------------- ;; Instructions to cope with inline literal tables ;; ------------------------------------------------------------------------- @@ -12688,7 +12681,7 @@ [(set_attr "type" "arith_media") (set_attr "highpart" "ignore")]) -/* These are useful to expand ANDs and as combiner patterns. */ +;; These are useful to expand ANDs and as combiner patterns. (define_insn_and_split "mshfhi_l_di" [(set (match_operand:DI 0 "arith_reg_dest" "=r,f") (ior:DI (lshiftrt:DI (match_operand:DI 1 "arith_reg_or_0_operand" "rZ,f")