HI Jakub,

I have updated the changelog entries as per your suggestion.
Is this OK? Thank you!

Best Regards,
Sebastian

Index: ChangeLog
===================================================================
--- ChangeLog   (revision 257583)
+++ ChangeLog   (working copy)
@@ -129,8 +129,7 @@
 
 2018-02-09  Sebastian Perta  <sebastian.pe...@renesas.com>
 
-       * config/rx.md: updated "movsicc" expand to be matched by GCC
-       * testsuite/gcc.target/rx/movsicc.c: new test case
+       * config/rx/rx.md (movsicc): Update expander to be matched by GCC.
 
 2018-02-09  Peter Bergner  <berg...@vnet.ibm.com>
 
@@ -143,10 +142,10 @@
 
 2018-02-09  Sebastian Perta  <sebastian.pe...@renesas.com>
 
-       * config/rx/constraints.md: added new constraint CALL_OP_SYMBOL_REF 
-       to allow or block "symbol_ref" depending on value of TARGET_JSR
-       * config/rx/rx.md: use CALL_OP_SYMBOL_REF in call_internal and 
-       call_value_internal insns
+       * config/rx/constraints.md (CALL_OP_SYMBOL_REF): Added new
constraint 
+       to allow or block "symbol_ref" depending on the value of TARGET_JSR.
+       * config/rx/rx.md (call_internal): Use CALL_OP_SYMBOL_REF.
+       * config/rx/rx.md (call_value_internal): Use CALL_OP_SYMBOL_REF.
 
 2018-02-09  Pierre-Marie de Rodat  <dero...@adacore.com>
 
@@ -1342,9 +1341,8 @@
 
 2018-01-26  Sebastian Perta  <sebastian.pe...@renesas.com>
 
-       * config/rl78/rl78.c: if operand 2 is const avoid addition with 0
-       and use incw and decw where possible
-       * testsuite/gcc.target/rl78/test_addsi3_internal.c: new file
+       * config/rl78/rl78.c (rl78_addsi3_internal): If operand 2 is const 
+       avoid addition with 0 and use incw and decw where possible.
 
 2018-01-26  Richard Biener  <rguent...@suse.de>
 
@@ -1675,15 +1673,15 @@
 
 2018-01-22  Sebastian Perta  <sebastian.pe...@renesas.com>
 
-       * config/rl78/rl78-expand.md: New define_expand "bswaphi2"
-       * config/rl78/rl78-virt.md: New define_insn "*bswaphi2_virt"
-       * config/rl78/rl78-real.md: New define_insn "*bswaphi2_real"
+       * config/rl78/rl78-expand.md (bswaphi2): New define_expand.
+       * config/rl78/rl78-virt.md (*bswaphi2_virt): New define_insn.
+       * config/rl78/rl78-real.md (*bswaphi2_real): New define_insn.
 
 2018-01-22  Sebastian Perta  <sebastian.pe...@renesas.com>
 
-       * config/rl78/rl78-protos.h: New function declaration
rl78_split_movdi
-       * config/rl78/rl78.md: New define_expand "movdi"
-       * config/rl78/rl78.c: New function definition rl78_split_movdi
+       * config/rl78/rl78-protos.h (rl78_split_movdi): New function
declaration.
+       * config/rl78/rl78.md (movdi): New define_expand.
+       * config/rl78/rl78.c (rl78_split_movdi): New function.
 
 2018-01-22  Michael Meissner  <meiss...@linux.vnet.ibm.com>
 
@@ -1706,19 +1704,19 @@
 
 2018-01-22  Sebastian Perta  <sebastian.pe...@renesas.com>
 
-       * config/rl78/rl78.md: New define_expand "anddi3".
+       * config/rl78/rl78.md (anddi3): New define_expand.
 
 2018-01-22  Sebastian Perta  <sebastian.pe...@renesas.com>
 
-       * config/rl78/rl78.md: New define_expand "umindi3".
+       * config/rl78/rl78.md (umindi3): New define_expand.
 
 2018-01-22  Sebastian Perta  <sebastian.pe...@renesas.com>
 
-       * config/rl78/rl78.md: New define_expand "smindi3".
+       * config/rl78/rl78.md (smindi3): New define_expand.
 
 2018-01-22  Sebastian Perta  <sebastian.pe...@renesas.com>
 
-       * config/rl78/rl78.md: New define_expand "smaxdi3".
+       * config/rl78/rl78.md (smaxdi3): New define_expand.
 
 2018-01-22 Carl Love <c...@us.ibm.com>
 
@@ -1738,12 +1736,12 @@
 
 2018-01-22  Sebastian Perta  <sebastian.pe...@renesas.com>
 
-       * config/rl78/rl78.md: New define_expand "umaxdi3".
+       * config/rl78/rl78.md (umaxdi3): New define_expand.
 
 2018-01-22  Sebastian Perta  <sebastian.pe...@renesas.com>
 
-       * config/rl78/rl78.c (rl78_note_reg_set): fixed dead reg check
-       for non-QImode registers
+       * config/rl78/rl78.c (rl78_note_reg_set): Fixed dead reg check
+       for non-QImode registers.
 
 2018-01-22  Richard Biener  <rguent...@suse.de>
 
Index: testsuite/ChangeLog
===================================================================
--- testsuite/ChangeLog (revision 257583)
+++ testsuite/ChangeLog (working copy)
@@ -80,7 +80,11 @@
 
        PR sanitizer/83987
        * g++.dg/ubsan/pr83987-2.C: New test.
+       
+2018-02-09  Sebastian Perta  <sebastian.pe...@renesas.com>
+
+       * gcc.target/rx/movsicc.c: New test.
+
 2018-02-09  Peter Bergner  <berg...@vnet.ibm.com>
 
        PR target/83926
@@ -945,6 +949,10 @@
        PR c++/83924
        * g++.dg/warn/Wduplicated-branches5.C: New.
 
+2018-01-26  Sebastian Perta  <sebastian.pe...@renesas.com>
+
+       * gcc.target/rl78/test_addsi3_internal.c: New test.
+
 2018-01-26  Segher Boessenkool  <seg...@kernel.crashing.org>
 
        * gcc.target/powerpc/safe-indirect-jump-1.c: Build on all targets.

> -----Original Message-----
> From: Jakub Jelinek [mailto:ja...@redhat.com]
> Sent: 12 February 2018 12:45
> To: Sebastian Perta <sebastian.pe...@renesas.com>
> Cc: Nick Clifton <ni...@redhat.com>; gcc-patches <gcc-
> patc...@gcc.gnu.org>
> Subject: Re: PING [PATCH] RX movsicc degrade fix
> 
> On Mon, Feb 12, 2018 at 11:06:35AM -0000, Sebastian Perta wrote:
> > Hi Jakub,
> >
> > Thank you for pointing this out, I'm sorry!
> > Can I create a patch to correct the changelog entries?
> >
> > Best Regards,
> > Sebastian
> >
> > >>1) there should be a space between * and the filename
> > The spaces are there (see the changelog), the renesas mail server
removes
> > them sometimes
> 
> They weren't there in what I've fixed.
> See
> https://gcc.gnu.org/viewcvs/gcc/trunk/gcc/ChangeLog?r1=257536&r2=25753
> 5&pathrev=257536
> 
>       Jakub

Reply via email to