From patchwork Thu Mar 9 17:45:25 2017 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: David Malcolm X-Patchwork-Id: 737073 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]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ozlabs.org (Postfix) with ESMTPS id 3vfHBK5Wrfz9s0Z for ; Fri, 10 Mar 2017 04:16:57 +1100 (AEDT) Authentication-Results: ozlabs.org; dkim=pass (1024-bit key; unprotected) header.d=gcc.gnu.org header.i=@gcc.gnu.org header.b="Oi+Kf/Ny"; dkim-atps=neutral DomainKey-Signature: a=rsa-sha1; c=nofws; d=gcc.gnu.org; h=list-id :list-unsubscribe:list-archive:list-post:list-help:sender:from :to:cc:subject:date:message-id:in-reply-to:references; q=dns; s= default; b=ZtZZYI/pHIxmnXNO0dZaRv6AvXMIqVDrgYVuUtfI1SwPo4iFPrwTP Xzq5m+/ZLoNKtp3KG/6SsgIiXIgPK/I3QjmsD6Mi47EuEJbakRHYt1Pu7TLXkCdP i9KgSL7H2M0UdNhTeQ/S6I4XhVgZQWbERe+qsBlGMQQyQBmylB+GgA= DKIM-Signature: v=1; a=rsa-sha1; c=relaxed; d=gcc.gnu.org; h=list-id :list-unsubscribe:list-archive:list-post:list-help:sender:from :to:cc:subject:date:message-id:in-reply-to:references; s= default; bh=sYWEt6tdZ8onBAm00RjB81m8cng=; b=Oi+Kf/Ny9NR+92qHAD71 SZO2DxSsig+lm5cLNp3Vv1uGvb1s3u5wQtbRVnOcGwSGaRpcEuWpK4/IqbJ2iW0N E+H+5rZ28Mam9JMcmivRbLu5INxDQfAqEMSJ5RJJSvxAS3vqGVkggImiUo5TwyXG HO3BrcWeGdhfsGLv7OZ2PWM= Received: (qmail 3245 invoked by alias); 9 Mar 2017 17:15:55 -0000 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 Received: (qmail 2998 invoked by uid 89); 9 Mar 2017 17:15:54 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-25.9 required=5.0 tests=BAYES_00, GIT_PATCH_0, GIT_PATCH_1, GIT_PATCH_2, GIT_PATCH_3, KAM_LAZY_DOMAIN_SECURITY, RP_MATCHES_RCVD, SPF_HELO_PASS autolearn=ham version=3.3.2 spammy=Hx-languages-length:9863, 9437 X-HELO: mx1.redhat.com Received: from mx1.redhat.com (HELO mx1.redhat.com) (209.132.183.28) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Thu, 09 Mar 2017 17:15:52 +0000 Received: from int-mx11.intmail.prod.int.phx2.redhat.com (int-mx11.intmail.prod.int.phx2.redhat.com [10.5.11.24]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mx1.redhat.com (Postfix) with ESMTPS id E85EAC04B935; Thu, 9 Mar 2017 17:15:52 +0000 (UTC) Received: from c64.redhat.com (vpn-235-134.phx2.redhat.com [10.3.235.134]) by int-mx11.intmail.prod.int.phx2.redhat.com (8.14.4/8.14.4) with ESMTP id v29HFnKp025892; Thu, 9 Mar 2017 12:15:52 -0500 From: David Malcolm To: gcc-patches@gcc.gnu.org Cc: roland.illig@gmx.de, Joseph Myers , David Malcolm Subject: [PATCH 3/7] Remove trailing period from various diagnostic messages (PR translation/79923) Date: Thu, 9 Mar 2017 12:45:25 -0500 Message-Id: <1489081529-22256-4-git-send-email-dmalcolm@redhat.com> In-Reply-To: <1489081529-22256-1-git-send-email-dmalcolm@redhat.com> References: <1489081529-22256-1-git-send-email-dmalcolm@redhat.com> X-IsSubscribed: yes gcc/ChangeLog: PR translation/79923 * auto-profile.c (get_combined_location): Convert leading character of diagnostics to lower case and remove trailing period. (read_profile): Likewise for various diagnostics. * config/arm/arm-builtins.c (arm_expand_builtin): Remove trailing period from various diagnostics. * config/arm/arm.c (arm_option_override): Likewise. * config/msp430/msp430.c (msp430_expand_delay_cycles): Likewise. (msp430_expand_delay_cycles): Likewise. --- gcc/auto-profile.c | 14 ++++++------- gcc/config/arm/arm-builtins.c | 48 +++++++++++++++++++++---------------------- gcc/config/arm/arm.c | 2 +- gcc/config/msp430/msp430.c | 4 ++-- 4 files changed, 34 insertions(+), 34 deletions(-) diff --git a/gcc/auto-profile.c b/gcc/auto-profile.c index 6255f94..4b21340 100644 --- a/gcc/auto-profile.c +++ b/gcc/auto-profile.c @@ -344,7 +344,7 @@ get_combined_location (location_t loc, tree decl) { /* TODO: allow more bits for line and less bits for discriminator. */ if (LOCATION_LINE (loc) - DECL_SOURCE_LINE (decl) >= (1<<16)) - warning_at (loc, OPT_Woverflow, "Offset exceeds 16 bytes."); + warning_at (loc, OPT_Woverflow, "offset exceeds 16 bytes"); return ((LOCATION_LINE (loc) - DECL_SOURCE_LINE (decl)) << 16); } @@ -917,13 +917,13 @@ read_profile (void) { if (gcov_open (auto_profile_file, 1) == 0) { - error ("Cannot open profile file %s.", auto_profile_file); + error ("cannot open profile file %s", auto_profile_file); return; } if (gcov_read_unsigned () != GCOV_DATA_MAGIC) { - error ("AutoFDO profile magic number does not match."); + error ("AutoFDO profile magic number does not match"); return; } @@ -931,7 +931,7 @@ read_profile (void) unsigned version = gcov_read_unsigned (); if (version != AUTO_PROFILE_VERSION) { - error ("AutoFDO profile version %u does match %u.", + error ("AutoFDO profile version %u does match %u", version, AUTO_PROFILE_VERSION); return; } @@ -943,7 +943,7 @@ read_profile (void) afdo_string_table = new string_table (); if (!afdo_string_table->read()) { - error ("Cannot read string table from %s.", auto_profile_file); + error ("cannot read string table from %s", auto_profile_file); return; } @@ -951,7 +951,7 @@ read_profile (void) afdo_source_profile = autofdo_source_profile::create (); if (afdo_source_profile == NULL) { - error ("Cannot read function profile from %s.", auto_profile_file); + error ("cannot read function profile from %s", auto_profile_file); return; } @@ -961,7 +961,7 @@ read_profile (void) /* Read in the working set. */ if (gcov_read_unsigned () != GCOV_TAG_AFDO_WORKING_SET) { - error ("Cannot read working set from %s.", auto_profile_file); + error ("cannot read working set from %s", auto_profile_file); return; } diff --git a/gcc/config/arm/arm-builtins.c b/gcc/config/arm/arm-builtins.c index 689219c..661c028 100644 --- a/gcc/config/arm/arm-builtins.c +++ b/gcc/config/arm/arm-builtins.c @@ -2990,60 +2990,60 @@ arm_expand_builtin (tree exp, && (imm < 0 || imm > 32)) { if (fcode == ARM_BUILTIN_WRORHI) - error ("the range of count should be in 0 to 32. please check the intrinsic _mm_rori_pi16 in code."); + error ("the range of count should be in 0 to 32. please check the intrinsic _mm_rori_pi16 in code"); else if (fcode == ARM_BUILTIN_WRORWI) - error ("the range of count should be in 0 to 32. please check the intrinsic _mm_rori_pi32 in code."); + error ("the range of count should be in 0 to 32. please check the intrinsic _mm_rori_pi32 in code"); else if (fcode == ARM_BUILTIN_WRORH) - error ("the range of count should be in 0 to 32. please check the intrinsic _mm_ror_pi16 in code."); + error ("the range of count should be in 0 to 32. please check the intrinsic _mm_ror_pi16 in code"); else - error ("the range of count should be in 0 to 32. please check the intrinsic _mm_ror_pi32 in code."); + error ("the range of count should be in 0 to 32. please check the intrinsic _mm_ror_pi32 in code"); } else if ((fcode == ARM_BUILTIN_WRORDI || fcode == ARM_BUILTIN_WRORD) && (imm < 0 || imm > 64)) { if (fcode == ARM_BUILTIN_WRORDI) - error ("the range of count should be in 0 to 64. please check the intrinsic _mm_rori_si64 in code."); + error ("the range of count should be in 0 to 64. please check the intrinsic _mm_rori_si64 in code"); else - error ("the range of count should be in 0 to 64. please check the intrinsic _mm_ror_si64 in code."); + error ("the range of count should be in 0 to 64. please check the intrinsic _mm_ror_si64 in code"); } else if (imm < 0) { if (fcode == ARM_BUILTIN_WSRLHI) - error ("the count should be no less than 0. please check the intrinsic _mm_srli_pi16 in code."); + error ("the count should be no less than 0. please check the intrinsic _mm_srli_pi16 in code"); else if (fcode == ARM_BUILTIN_WSRLWI) - error ("the count should be no less than 0. please check the intrinsic _mm_srli_pi32 in code."); + error ("the count should be no less than 0. please check the intrinsic _mm_srli_pi32 in code"); else if (fcode == ARM_BUILTIN_WSRLDI) - error ("the count should be no less than 0. please check the intrinsic _mm_srli_si64 in code."); + error ("the count should be no less than 0. please check the intrinsic _mm_srli_si64 in code"); else if (fcode == ARM_BUILTIN_WSLLHI) - error ("the count should be no less than 0. please check the intrinsic _mm_slli_pi16 in code."); + error ("the count should be no less than 0. please check the intrinsic _mm_slli_pi16 in code"); else if (fcode == ARM_BUILTIN_WSLLWI) - error ("the count should be no less than 0. please check the intrinsic _mm_slli_pi32 in code."); + error ("the count should be no less than 0. please check the intrinsic _mm_slli_pi32 in code"); else if (fcode == ARM_BUILTIN_WSLLDI) - error ("the count should be no less than 0. please check the intrinsic _mm_slli_si64 in code."); + error ("the count should be no less than 0. please check the intrinsic _mm_slli_si64 in code"); else if (fcode == ARM_BUILTIN_WSRAHI) - error ("the count should be no less than 0. please check the intrinsic _mm_srai_pi16 in code."); + error ("the count should be no less than 0. please check the intrinsic _mm_srai_pi16 in code"); else if (fcode == ARM_BUILTIN_WSRAWI) - error ("the count should be no less than 0. please check the intrinsic _mm_srai_pi32 in code."); + error ("the count should be no less than 0. please check the intrinsic _mm_srai_pi32 in code"); else if (fcode == ARM_BUILTIN_WSRADI) - error ("the count should be no less than 0. please check the intrinsic _mm_srai_si64 in code."); + error ("the count should be no less than 0. please check the intrinsic _mm_srai_si64 in code"); else if (fcode == ARM_BUILTIN_WSRLH) - error ("the count should be no less than 0. please check the intrinsic _mm_srl_pi16 in code."); + error ("the count should be no less than 0. please check the intrinsic _mm_srl_pi16 in code"); else if (fcode == ARM_BUILTIN_WSRLW) - error ("the count should be no less than 0. please check the intrinsic _mm_srl_pi32 in code."); + error ("the count should be no less than 0. please check the intrinsic _mm_srl_pi32 in code"); else if (fcode == ARM_BUILTIN_WSRLD) - error ("the count should be no less than 0. please check the intrinsic _mm_srl_si64 in code."); + error ("the count should be no less than 0. please check the intrinsic _mm_srl_si64 in code"); else if (fcode == ARM_BUILTIN_WSLLH) - error ("the count should be no less than 0. please check the intrinsic _mm_sll_pi16 in code."); + error ("the count should be no less than 0. please check the intrinsic _mm_sll_pi16 in code"); else if (fcode == ARM_BUILTIN_WSLLW) - error ("the count should be no less than 0. please check the intrinsic _mm_sll_pi32 in code."); + error ("the count should be no less than 0. please check the intrinsic _mm_sll_pi32 in code"); else if (fcode == ARM_BUILTIN_WSLLD) - error ("the count should be no less than 0. please check the intrinsic _mm_sll_si64 in code."); + error ("the count should be no less than 0. please check the intrinsic _mm_sll_si64 in code"); else if (fcode == ARM_BUILTIN_WSRAH) - error ("the count should be no less than 0. please check the intrinsic _mm_sra_pi16 in code."); + error ("the count should be no less than 0. please check the intrinsic _mm_sra_pi16 in code"); else if (fcode == ARM_BUILTIN_WSRAW) - error ("the count should be no less than 0. please check the intrinsic _mm_sra_pi32 in code."); + error ("the count should be no less than 0. please check the intrinsic _mm_sra_pi32 in code"); else - error ("the count should be no less than 0. please check the intrinsic _mm_sra_si64 in code."); + error ("the count should be no less than 0. please check the intrinsic _mm_sra_si64 in code"); } } return arm_expand_binop_builtin (icode, exp, target); diff --git a/gcc/config/arm/arm.c b/gcc/config/arm/arm.c index b397a73..511e163 100644 --- a/gcc/config/arm/arm.c +++ b/gcc/config/arm/arm.c @@ -3370,7 +3370,7 @@ arm_option_override (void) if (arm_fp16_inst) { if (arm_fp16_format == ARM_FP16_FORMAT_ALTERNATIVE) - error ("selected fp16 options are incompatible."); + error ("selected fp16 options are incompatible"); arm_fp16_format = ARM_FP16_FORMAT_IEEE; } diff --git a/gcc/config/msp430/msp430.c b/gcc/config/msp430/msp430.c index 15d9678..710a97b 100644 --- a/gcc/config/msp430/msp430.c +++ b/gcc/config/msp430/msp430.c @@ -2501,7 +2501,7 @@ msp430_expand_delay_cycles (rtx arg) { if (c < 0) { - error ("__delay_cycles only takes non-negative cycle counts."); + error ("__delay_cycles only takes non-negative cycle counts"); return NULL_RTX; } } @@ -2521,7 +2521,7 @@ msp430_expand_delay_cycles (rtx arg) c %= 4; if ((unsigned long long) i > 0xffffffffULL) { - error ("__delay_cycles is limited to 32-bit loop counts."); + error ("__delay_cycles is limited to 32-bit loop counts"); return NULL_RTX; } }