From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 36780 invoked by alias); 12 Feb 2018 14:08:01 -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 36766 invoked by uid 89); 12 Feb 2018 14:08:01 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-11.1 required=5.0 tests=BAYES_00,GIT_PATCH_2,GIT_PATCH_3,KAM_ASCII_DIVIDERS,RCVD_IN_DNSWL_NONE,SPF_PASS,T_RP_MATCHES_RCVD autolearn=ham version=3.3.2 spammy=H*UA:Outlook, H*x:Outlook, H*UA:Microsoft, H*x:Microsoft X-HELO: relmlie1.idc.renesas.com Received: from relmlor2.renesas.com (HELO relmlie1.idc.renesas.com) (210.160.252.172) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Mon, 12 Feb 2018 14:07:59 +0000 Received: from unknown (HELO relmlir4.idc.renesas.com) ([10.200.68.154]) by relmlie1.idc.renesas.com with ESMTP; 12 Feb 2018 23:07:56 +0900 Received: from relmlii1.idc.renesas.com (relmlii1.idc.renesas.com [10.200.68.65]) by relmlir4.idc.renesas.com (Postfix) with ESMTP id F165A690BF; Mon, 12 Feb 2018 23:07:56 +0900 (JST) Received: from unknown (HELO REEBED03978) ([10.226.37.54]) by relmlii1.idc.renesas.com with ESMTP; 12 Feb 2018 23:07:55 +0900 From: "Sebastian Perta" To: "'DJ Delorie'" Cc: , "'Jakub Jelinek'" References: (message from Sebastian Perta on Tue, 6 Feb 2018 13:39:14 +0000) In-Reply-To: Subject: RE: [PATCH] RL78 new "vector" function attribute Date: Mon, 12 Feb 2018 14:08:00 -0000 Message-ID: <000201d3a40a$e1c8fc70$a55af550$@renesas.com> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit X-SW-Source: 2018-02/txt/msg00646.txt.bz2 Hi DJ, >>Looks OK to me, but wait a day or two for a docs person to comment on... 6 days no comments so far, can I check in now? >>if the new line is too long There are many other lines which have the same length or are even longer this is why I let it as it is. Also based on comments from Jakub (on a different patch) I corrected the Changelog entry for this patch (see below). Is this OK? Best Regards, Sebastian Index: ChangeLog =================================================================== --- ChangeLog (revision 257588) +++ ChangeLog (working copy) @@ -1,3 +1,13 @@ +2018-02-12 Sebastian Perta + + * config/rl78/rl78.c (add_vector_labels): New function. + * config/rl78/rl78.c (rl78_handle_vector_attribute): New function. + * config/rl78/rl78.c (rl78_start_function): Call add_vector_labels. + * config/rl78/rl78.c (rl78_handle_func_attribute): Removed the assert + which checks that no arguments are passed. + * config/rl78/rl78.c (rl78_attribute_table): Add "vector" attribute. + * doc/extend.texi: Documentation for the new attribute. + 2018-02-12 Richard Biener PR tree-optimization/84037 Index: testsuite/ChangeLog =================================================================== --- testsuite/ChangeLog (revision 257588) +++ testsuite/ChangeLog (working copy) @@ -1,3 +1,7 @@ +2018-02-12 Sebastian Perta + + * gcc.target/rl78/test_auto_vector.c: New test. + 2018-02-12 Tamar Christina PR target/82641 > -----Original Message----- > From: DJ Delorie [mailto:dj@redhat.com] > Sent: 06 February 2018 22:57 > To: Sebastian Perta > Cc: gcc-patches@gcc.gnu.org > Subject: Re: [PATCH] RL78 new "vector" function attribute > > > Sebastian Perta writes: > > I've updated the patch (extend.texi) as you suggested. > > Please let me know if this is OK to check-in, thank you! > > Looks OK to me, but wait a day or two for a docs person to comment on... > > > -On RX targets, you may specify one or more vector numbers as arguments > > +On RX and RL78 targets, you may specify one or more vector numbers as > arguments > > ...if the new line is too long and if a paragraph reformat is warranted. > > Thanks!