Re: [PATCH, GCC] Deal with singular sentences in builtin-sprintf-warn-1.c regex

2016-09-23 Thread Thomas Preudhomme
Sorry, forgot the patch. Please find it attached. Best regards, Thomas On 23/09/16 16:40, Thomas Preudhomme wrote: Hi, New builtin-sprintf-warn-1.c testcase contains a few regex of the form "\[0-9\]+ bytes" or ". bytes". This does not account for the case where the nu

[PATCH] Fix builtin-sprintf-warn-2.c xfail pattern

2016-09-23 Thread Thomas Preudhomme
The new builtin-sprintf-warn-2.c xfail on line 83 and 84 for *-*-*-* which fails for arm-none-eabi for instance. The usual pattern for catchall xfail is *-*-*. This patch fixes the pattern as such. ChangeLog entry is as follows: *** gcc/testsuite/ChangeLog *** 2016-09-23 Thomas Preud'homme

Re: [PATCH, GCC] Deal with singular sentences in builtin-sprintf-warn-1.c regex

2016-09-23 Thread Thomas Preudhomme
Hi Martin, On 23/09/16 17:17, Martin Sebor wrote: On 09/23/2016 09:42 AM, Thomas Preudhomme wrote: Sorry, forgot the patch. Please find it attached. Best regards, Thomas On 23/09/16 16:40, Thomas Preudhomme wrote: Hi, New builtin-sprintf-warn-1.c testcase contains a few regex of the form

Re: [PATCH, GCC] Deal with singular sentences in builtin-sprintf-warn-1.c regex

2016-09-26 Thread Thomas Preudhomme
How about this reworked patch? Best regards, Thomas On 23/09/16 17:17, Martin Sebor wrote: On 09/23/2016 09:42 AM, Thomas Preudhomme wrote: Sorry, forgot the patch. Please find it attached. Best regards, Thomas On 23/09/16 16:40, Thomas Preudhomme wrote: Hi, New builtin-sprintf-warn-1.c

Re: [PATCH, 1/7] Move memory model declarations in memmodel.h

2016-09-26 Thread Thomas Preudhomme
On 26/09/16 10:15, Richard Biener wrote: On Thu, 22 Sep 2016, Thomas Preudhomme wrote: Hi, This patch is part of a patch series to add support for ARMv8-M[1] to GCC. This specific patch moves memory model declarations in memmodel.h. Currently, is_mm_* memory model related functions are

Re: [PATCH, 1/7] Move memory model declarations in memmodel.h

2016-09-26 Thread Thomas Preudhomme
On 26/09/16 11:53, Richard Biener wrote: On Mon, 26 Sep 2016, Thomas Preudhomme wrote: On 26/09/16 10:15, Richard Biener wrote: On Thu, 22 Sep 2016, Thomas Preudhomme wrote: Hi, This patch is part of a patch series to add support for ARMv8-M[1] to GCC. This specific patch moves memory

Re: [PATCH, 1/7] Move memory model declarations in memmodel.h

2016-09-26 Thread Thomas Preudhomme
On 26/09/16 18:22, Thomas Preudhomme wrote: I committed as is because emit-rtl.h uses enum memmodel so all files that includes it must be updated. This gets worse because tm.h uses emit-rtl.h so all target needs to be updated as well. The diff was becoming big so I decided to keep it separate

Re: [PATCH, ARM 2/7, ping] Adapt atomic and exclusive load and store to ARMv8-M Baseline

2016-10-03 Thread Thomas Preudhomme
Ping? Best regards, Thomas On 22/09/16 14:41, Thomas Preudhomme wrote: Hi, This patch is part of a patch series to add support for atomic operations on ARMv8-M Baseline targets in GCC. This specific patch adapts atomic and exclusive load and store patterns to the constraints of ARMv8-M

Re: [PATCH, ARM 3/7, ping] Refactor atomic compare_and_swap to make it fit for ARMv8-M Baseline

2016-10-03 Thread Thomas Preudhomme
Ping? Best regards, Thomas On 22/09/16 14:44, Thomas Preudhomme wrote: Hi, This patch is part of a patch series to add support for atomic operations on ARMv8-M Baseline targets in GCC. This specific patch refactors the expander and splitter for atomics to make the logic work with ARMv8-M

Re: [PATCH, ARM 4/7, ping] Adapt atomic compare and swap to ARMv8-M Baseline

2016-10-03 Thread Thomas Preudhomme
Ping? Best regards, Thomas On 22/09/16 14:46, Thomas Preudhomme wrote: Hi, This patch is part of a patch series to add support for atomic operations on ARMv8-M Baseline targets in GCC. This specific patch makes the necessary change for compare and swap to work for ARMv8-M Baseline

Re: [PATCH, ARM 5/7, ping] Adapt other atomic operations to ARMv8-M Baseline

2016-10-03 Thread Thomas Preudhomme
Ping? Best regards, Thomas On 22/09/16 14:47, Thomas Preudhomme wrote: Hi, This patch is part of a patch series to add support for atomic operations on ARMv8-M Baseline targets in GCC. This specific patch adds support for remaining atomic operations (exchange, addition, substraction, bitwise

Re: [PATCH, ARM/testsuite 6/7, ping] Force soft float in ARMv6-M and ARMv8-M Baseline options

2016-10-03 Thread Thomas Preudhomme
On 22/09/16 17:15, Thomas Preudhomme wrote: On 22/09/16 16:47, Richard Earnshaw (lists) wrote: On 22/09/16 15:51, Thomas Preudhomme wrote: Sorry, noticed an error in the patch. It was not caught during testing because GCC was built with --with-mode=thumb. Correct patch attached. Best regards

Re: [PATCH, ARM 7/7, ping] Enable ARMv8-M atomic and synchronization support for ARMv8-M Baseline

2016-10-03 Thread Thomas Preudhomme
Ping? Best regards, Thomas On 22/09/16 14:50, Thomas Preudhomme wrote: Hi, This patch is part of a patch series to add support for atomic operations on ARMv8-M Baseline targets in GCC. This specific patch enables atomic and synchronization support added in previous patches of the series and

[PATCH, testsuite] Fix PR77710: incorrect wildchar target triplet

2016-10-11 Thread Thomas Preudhomme
I've applied the following commit as obvious to fix builtin-sprintf-warn-4.c excess error failure on arm-none-eabi due to incorrect wildchar target triplet. ChangeLog entry is as follows: *** gcc/testsuite/ChangeLog *** 2016-10-11 Thomas Preud'homme PR testsuite/PR77710 * g

Re: [PATCH, ARM 5/7] Add support for MOVT/MOVW to ARMv8-M Baseline

2016-10-11 Thread Thomas Preudhomme
On 10/10/16 13:35, Christophe Lyon wrote: Hi Thomas, Hi Christophe, On 13 July 2016 at 17:34, Thomas Preudhomme wrote: On Wednesday 13 July 2016 17:14:52 Christophe Lyon wrote: Hi Thomas, Hi Christophe, I'm seeing: gcc.target/arm/pr42574.c: syntax error in target sel

[PATCH, GCC/testsuite/ARM] Enable mov?i_movw tests on ARM and big endian

2016-10-12 Thread Thomas Preudhomme
Hi, As reported by Christophe Lyon, gcc.target/arm/movdi_movw test fails on big endian targets. This is because on big endian targets the low bits of a 64bit value would be in the highest numbered register of a pair rather than the lowest numbered register as on little endian targets. This pat

Re: [PATCH, GCC/testsuite/ARM] Enable mov?i_movw tests on ARM and big endian

2016-10-12 Thread Thomas Preudhomme
Sorry :-( Here you are. Cheers, Thomas On 12/10/16 16:25, Kyrill Tkachov wrote: On 12/10/16 16:21, Thomas Preudhomme wrote: Hi, As reported by Christophe Lyon, gcc.target/arm/movdi_movw test fails on big endian targets. This is because on big endian targets the low bits of a 64bit value

Re: [PATCH, GCC/testsuite/ARM] Enable mov?i_movw tests on ARM and big endian

2016-10-12 Thread Thomas Preudhomme
On 12/10/16 17:01, Christophe Lyon wrote: On 12 October 2016 at 17:50, Kyrill Tkachov wrote: On 12/10/16 16:29, Thomas Preudhomme wrote: Sorry :-( Here you are. Cheers, Thomas On 12/10/16 16:25, Kyrill Tkachov wrote: On 12/10/16 16:21, Thomas Preudhomme wrote: Hi, As reported by

[PATCH, GCC] Move MEMMODEL_* and enum memmodel from coretypes.h to memmodel.h

2016-10-12 Thread Thomas Preudhomme
Hi, This patch is a follow up of [1] which aims to have all memory model related declarations in memmodel.h. To achieve that, this patch moves memory model related declaration from coretypes.h into memmodel.h. Note that since memmodel.h is now included from libgcc it needs to have a runtime li

Re: [PATCH, GCC] Move MEMMODEL_* and enum memmodel from coretypes.h to memmodel.h

2016-10-13 Thread Thomas Preudhomme
On 12/10/16 21:00, Joseph Myers wrote: On Wed, 12 Oct 2016, Thomas Preudhomme wrote: This patch is a follow up of [1] which aims to have all memory model related declarations in memmodel.h. To achieve that, this patch moves memory model related declaration from coretypes.h into memmodel.h

[PATCH, GCC/ARM 1/2] Add multilib support for embedded bare-metal targets

2016-10-13 Thread Thomas Preudhomme
Hi ARM maintainers, This patchset aims at adding multilib support for R and M profile ARM architectures and allowing it to be built alongside multilib for A profile ARM architectures. This specific patch adds the t-rmprofile multilib Makefile fragment for the former objective. Multilib are bui

[PATCH, GCC/ARM 2/2] Allow combination of aprofile and rmprofile multilibs

2016-10-13 Thread Thomas Preudhomme
Hi ARM maintainers, This patchset aims at adding multilib support for R and M profile ARM architectures and allowing it to be built alongside multilib for A profile ARM architectures. This specific patch is concerned with the latter. The patch works by moving the bits shared by both aprofile a

Re: [PATCH, ARM 2/7, ping2] Adapt atomic and exclusive load and store to ARMv8-M Baseline

2016-10-14 Thread Thomas Preudhomme
Ping? Best regards, Thomas On 03/10/16 17:42, Thomas Preudhomme wrote: Ping? Best regards, Thomas On 22/09/16 14:41, Thomas Preudhomme wrote: Hi, This patch is part of a patch series to add support for atomic operations on ARMv8-M Baseline targets in GCC. This specific patch adapts

Re: [PATCH, ARM 4/7, ping] Adapt atomic compare and swap to ARMv8-M Baseline

2016-10-14 Thread Thomas Preudhomme
Ping? Best regards, Thomas On 03/10/16 17:45, Thomas Preudhomme wrote: Ping? Best regards, Thomas On 22/09/16 14:46, Thomas Preudhomme wrote: Hi, This patch is part of a patch series to add support for atomic operations on ARMv8-M Baseline targets in GCC. This specific patch makes the

Re: [PATCH, ARM 3/7, ping] Refactor atomic compare_and_swap to make it fit for ARMv8-M Baseline

2016-10-14 Thread Thomas Preudhomme
Ping? Best regards, Thomas On 03/10/16 17:44, Thomas Preudhomme wrote: Ping? Best regards, Thomas On 22/09/16 14:44, Thomas Preudhomme wrote: Hi, This patch is part of a patch series to add support for atomic operations on ARMv8-M Baseline targets in GCC. This specific patch refactors

Re: [PATCH, ARM 5/7, ping2] Adapt other atomic operations to ARMv8-M Baseline

2016-10-14 Thread Thomas Preudhomme
Ping? Best regards, Thomas On 03/10/16 17:45, Thomas Preudhomme wrote: Ping? Best regards, Thomas On 22/09/16 14:47, Thomas Preudhomme wrote: Hi, This patch is part of a patch series to add support for atomic operations on ARMv8-M Baseline targets in GCC. This specific patch adds support

Re: [PATCH, ARM 7/7, ping2] Enable ARMv8-M atomic and synchronization support for ARMv8-M Baseline

2016-10-14 Thread Thomas Preudhomme
Ping? Best regards, Thomas On 03/10/16 17:46, Thomas Preudhomme wrote: Ping? Best regards, Thomas On 22/09/16 14:50, Thomas Preudhomme wrote: Hi, This patch is part of a patch series to add support for atomic operations on ARMv8-M Baseline targets in GCC. This specific patch enables

Re: [PATCH, ARM 2/7, ping3] Adapt atomic and exclusive load and store to ARMv8-M Baseline

2016-10-24 Thread Thomas Preudhomme
Ping? Best regards, Thomas On 14/10/16 14:48, Thomas Preudhomme wrote: Ping? Best regards, Thomas On 03/10/16 17:42, Thomas Preudhomme wrote: Ping? Best regards, Thomas On 22/09/16 14:41, Thomas Preudhomme wrote: Hi, This patch is part of a patch series to add support for atomic

Re: [PATCH, ARM 3/7, ping3] Refactor atomic compare_and_swap to make it fit for ARMv8-M Baseline

2016-10-24 Thread Thomas Preudhomme
Ping? Best regards, Thomas On 14/10/16 14:50, Thomas Preudhomme wrote: Ping? Best regards, Thomas On 03/10/16 17:44, Thomas Preudhomme wrote: Ping? Best regards, Thomas On 22/09/16 14:44, Thomas Preudhomme wrote: Hi, This patch is part of a patch series to add support for atomic

Re: [PATCH, ARM 4/7, ping3] Adapt atomic compare and swap to ARMv8-M Baseline

2016-10-24 Thread Thomas Preudhomme
Ping? Best regards, Thomas On 14/10/16 14:50, Thomas Preudhomme wrote: Ping? Best regards, Thomas On 03/10/16 17:45, Thomas Preudhomme wrote: Ping? Best regards, Thomas On 22/09/16 14:46, Thomas Preudhomme wrote: Hi, This patch is part of a patch series to add support for atomic

Re: [PATCH, ARM 5/7, ping3] Adapt other atomic operations to ARMv8-M Baseline

2016-10-24 Thread Thomas Preudhomme
Ping? Best regards, Thomas On 14/10/16 14:51, Thomas Preudhomme wrote: Ping? Best regards, Thomas On 03/10/16 17:45, Thomas Preudhomme wrote: Ping? Best regards, Thomas On 22/09/16 14:47, Thomas Preudhomme wrote: Hi, This patch is part of a patch series to add support for atomic

Re: [PATCH, ARM 7/7, ping3] Enable ARMv8-M atomic and synchronization support for ARMv8-M Baseline

2016-10-24 Thread Thomas Preudhomme
Ping? Best regards, Thomas On 14/10/16 14:51, Thomas Preudhomme wrote: Ping? Best regards, Thomas On 03/10/16 17:46, Thomas Preudhomme wrote: Ping? Best regards, Thomas On 22/09/16 14:50, Thomas Preudhomme wrote: Hi, This patch is part of a patch series to add support for atomic

Re: [PATCH, GCC/ARM 2/2, ping] Allow combination of aprofile and rmprofile multilibs

2016-10-24 Thread Thomas Preudhomme
Ping? Best regards, Thomas On 13/10/16 16:35, Thomas Preudhomme wrote: Hi ARM maintainers, This patchset aims at adding multilib support for R and M profile ARM architectures and allowing it to be built alongside multilib for A profile ARM architectures. This specific patch is concerned with

Re: [PATCH, GCC/ARM 1/2, ping] Add multilib support for embedded bare-metal targets

2016-10-24 Thread Thomas Preudhomme
Ping? Best regards, Thomas On 13/10/16 16:35, Thomas Preudhomme wrote: Hi ARM maintainers, This patchset aims at adding multilib support for R and M profile ARM architectures and allowing it to be built alongside multilib for A profile ARM architectures. This specific patch adds the t

Re: [PATCH, ARM 2/7, ping3] Adapt atomic and exclusive load and store to ARMv8-M Baseline

2016-10-24 Thread Thomas Preudhomme
Hi Kyrill, On 24/10/16 17:40, Kyrill Tkachov wrote: Hi Thomas, On 24/10/16 09:04, Thomas Preudhomme wrote: Ping? Best regards, Thomas On 14/10/16 14:48, Thomas Preudhomme wrote: Ping? Best regards, Thomas On 03/10/16 17:42, Thomas Preudhomme wrote: Ping? Best regards, Thomas On 22

[PATCH, GCC/ARM] Optional -mthumb for Thumb only targets

2016-10-25 Thread Thomas Preudhomme
Hi, Currently when a user compiles for a thumb-only target (such as Cortex-M processors) without specifying the -mthumb option GCC throws the error "target CPU does not support ARM mode". This is suboptimal from a usability point of view: the -mthumb could be deduced from the -march or -mcpu o

[PATCH, gcc/ARM] Add support for Cortex-M33

2016-10-26 Thread Thomas Preudhomme
Hi, This patch adds support for the Cortex-M33 processor launched by ARM [1]. The patch adds support for the name and wires it up to the ARMv8-M Mainline with DSP extensions architecture and arm_v7m_tune tuning parameters for the time being. It also updates documentation to mention this new pr

[PATCH, gcc/ARM] Add support for Cortex-M23

2016-10-26 Thread Thomas Preudhomme
Hi, This patch adds support for the Cortex-M23 processor launched by ARM [1]. The patch adds support for the name and wires it up to the ARMv8-M Baseline architecture and arm_v6m_tune tuning parameters for the time being. It also updates documentation to mention this new processor. [1] http:

Re: [PATCH, ARM 7/7, ping3] Enable ARMv8-M atomic and synchronization support for ARMv8-M Baseline

2016-10-27 Thread Thomas Preudhomme
On 27/10/16 10:05, Kyrill Tkachov wrote: Hi Thomas, On 24/10/16 09:06, Thomas Preudhomme wrote: Ping? Best regards, Thomas On 14/10/16 14:51, Thomas Preudhomme wrote: Ping? Best regards, Thomas On 03/10/16 17:46, Thomas Preudhomme wrote: Ping? Best regards, Thomas On 22/09/16 14

Re: [PATCH, ARM 5/7, ping3] Adapt other atomic operations to ARMv8-M Baseline

2016-10-27 Thread Thomas Preudhomme
On 27/10/16 09:50, Kyrill Tkachov wrote: Hi Thomas, On 24/10/16 09:05, Thomas Preudhomme wrote: Ping? Best regards, Thomas On 14/10/16 14:51, Thomas Preudhomme wrote: Ping? Best regards, Thomas On 03/10/16 17:45, Thomas Preudhomme wrote: Ping? Best regards, Thomas On 22/09/16 14

Re: [arm-embedded] [PATCH, 1/7] Move memory model declarations in memmodel.h

2016-10-27 Thread Thomas Preudhomme
On 22/09/16 17:41, Thomas Preudhomme wrote: Hi, We've decided to apply the following patch to ARM/embedded-6-branch. Sorry I meant ARM/embedded-5-branch. This has just been applied on ARM/embedded-6-branch as well 2 days ago (2016-10-25). Best regards, Thomas

Re: [arm-embedded] [PATCH, ARM 2/7] Adapt atomic and exclusive load and store to ARMv8-M Baseline

2016-10-27 Thread Thomas Preudhomme
On 22/09/16 17:41, Thomas Preudhomme wrote: Hi, We've decided to apply the following patch to ARM/embedded-6-branch. Sorry I meant ARM/embedded-5-branch. This has just been applied on ARM/embedded-6-branch as well 1 day ago (2016-10-26). Best regards, Thomas

Re: [arm-embedded] [PATCH, ARM 3/7] Refactor atomic compare_and_swap to make it fit for ARMv8-M Baseline

2016-10-27 Thread Thomas Preudhomme
On 22/09/16 17:41, Thomas Preudhomme wrote: Hi, We've decided to apply the following patch to ARM/embedded-6-branch. Sorry I meant ARM/embedded-5-branch. This has just been applied on ARM/embedded-6-branch as well 1 day ago (2016-10-26). Best regards, Thomas

Re: [arm-embedded] [PATCH, ARM 4/7] Adapt atomic compare and swap to ARMv8-M Baseline

2016-10-27 Thread Thomas Preudhomme
On 22/09/16 17:42, Thomas Preudhomme wrote: Hi, We've decided to apply the following patch to ARM/embedded-6-branch. Sorry I meant ARM/embedded-5-branch. This has just been applied on ARM/embedded-6-branch as well 1 day ago (2016-10-26). Best regards, Thomas

Re: [arm-embedded] [PATCH, ARM 5/7] Adapt other atomic operations to ARMv8-M Baseline

2016-10-27 Thread Thomas Preudhomme
On 22/09/16 17:42, Thomas Preudhomme wrote: Hi, We've decided to apply the following patch to ARM/embedded-6-branch. Sorry I meant ARM/embedded-5-branch. This has just been applied on ARM/embedded-6-branch as well today. Best regards, Thomas

Re: [arm-embedded] [PATCH, ARM/testsuite 6/7] Force soft float in ARMv6-M and ARMv8-M Baseline options

2016-10-27 Thread Thomas Preudhomme
On 22/09/16 17:42, Thomas Preudhomme wrote: Hi, We've decided to apply the following patch to ARM/embedded-6-branch. Sorry, I meant ARM/embedded-5-branch. Best regards, Thomas

Re: [arm-embedded] [PATCH, ARM 7/7] Enable ARMv8-M atomic and synchronization support for ARMv8-M Baseline

2016-10-27 Thread Thomas Preudhomme
On 22/09/16 17:43, Thomas Preudhomme wrote: Hi, We've decided to apply the following patch to ARM/embedded-6-branch. Sorry I meant ARM/embedded-5-branch. This has just been applied on ARM/embedded-6-branch as well today. Best regards, Thomas

Re: [PATCH, GCC/ARM 1/2, ping] Add multilib support for embedded bare-metal targets

2016-10-27 Thread Thomas Preudhomme
Hi Kyrill, On 27/10/16 10:45, Kyrill Tkachov wrote: Hi Thomas, On 24/10/16 09:06, Thomas Preudhomme wrote: Ping? Best regards, Thomas On 13/10/16 16:35, Thomas Preudhomme wrote: Hi ARM maintainers, This patchset aims at adding multilib support for R and M profile ARM architectures and

Re: [PATCH, ARM/testsuite 6/7] Force soft float in ARMv6-M and ARMv8-M Baseline options

2016-10-28 Thread Thomas Preudhomme
On 22/09/16 16:47, Richard Earnshaw (lists) wrote: On 22/09/16 15:51, Thomas Preudhomme wrote: Sorry, noticed an error in the patch. It was not caught during testing because GCC was built with --with-mode=thumb. Correct patch attached. Best regards, Thomas On 22/09/16 14:49, Thomas

Re: [PATCH, ARM/testsuite 6/7, ping] Force soft float in ARMv6-M and ARMv8-M Baseline options

2016-11-02 Thread Thomas Preudhomme
Ping? Best regards, Thomas On 28/10/16 10:49, Thomas Preudhomme wrote: On 22/09/16 16:47, Richard Earnshaw (lists) wrote: On 22/09/16 15:51, Thomas Preudhomme wrote: Sorry, noticed an error in the patch. It was not caught during testing because GCC was built with --with-mode=thumb. Correct

Re: [PATCH, GCC/ARM 1/2, ping] Add multilib support for embedded bare-metal targets

2016-11-02 Thread Thomas Preudhomme
Ping? Best regards, Thomas On 27/10/16 15:26, Thomas Preudhomme wrote: Hi Kyrill, On 27/10/16 10:45, Kyrill Tkachov wrote: Hi Thomas, On 24/10/16 09:06, Thomas Preudhomme wrote: Ping? Best regards, Thomas On 13/10/16 16:35, Thomas Preudhomme wrote: Hi ARM maintainers, This patchset

Re: [PATCH, GCC/ARM 2/2, ping2] Allow combination of aprofile and rmprofile multilibs

2016-11-02 Thread Thomas Preudhomme
Ping? Best regards, Thomas On 24/10/16 09:07, Thomas Preudhomme wrote: Ping? Best regards, Thomas On 13/10/16 16:35, Thomas Preudhomme wrote: Hi ARM maintainers, This patchset aims at adding multilib support for R and M profile ARM architectures and allowing it to be built alongside

Re: [PATCH, gcc/ARM, ping] Add support for Cortex-M33

2016-11-02 Thread Thomas Preudhomme
Ping? Best regards, Thomas On 26/10/16 17:42, Thomas Preudhomme wrote: Hi, This patch adds support for the Cortex-M33 processor launched by ARM [1]. The patch adds support for the name and wires it up to the ARMv8-M Mainline with DSP extensions architecture and arm_v7m_tune tuning parameters

Re: [PATCH, gcc/ARM, ping] Add support for Cortex-M23

2016-11-02 Thread Thomas Preudhomme
Ping? Best regards, Thomas On 26/10/16 17:42, Thomas Preudhomme wrote: Hi, This patch adds support for the Cortex-M23 processor launched by ARM [1]. The patch adds support for the name and wires it up to the ARMv8-M Baseline architecture and arm_v6m_tune tuning parameters for the time being

[PATCH, GCC/ARM] Fix PR77933: stack corruption on ARM when using high registers and lr

2016-11-02 Thread Thomas Preudhomme
Hi, When saving registers, function thumb1_expand_prologue () aims at minimizing the number of push instructions. One of the optimization it does is to push lr alongside high register(s) (after having moved them to low register(s)) when there is no low register to save. The way this is impleme

[PATCH, GCC/ARM] Fix PR77904: callee-saved register trashed when clobbering sp

2016-11-03 Thread Thomas Preudhomme
Hi, When using a callee-saved register to save the frame pointer the Thumb-1 prologue fails to save the callee-saved register before that. For ARM and Thumb-2 targets the frame pointer is handled as a special case but nothing is done for Thumb-1 targets. This patch adds the same logic for Thum

Re: [PATCH, libstdc++/testsuite, ping2] 29_atomics/atomic/65913.cc: require atomic-builtins rather than specific target

2016-06-30 Thread Thomas Preudhomme
On Wednesday 29 June 2016 21:58:40 Jonathan Wakely wrote: > On 29/06/16 13:49 -0700, Mike Stump wrote: > >Please include the libstdc++ list, they don't all read the other list. > > And the documentation clearly says (in two places) that all libstdc++ > patches must go to the libstdc++ list. Oops

Re: [PATCH, libgcc/ARM 1a/6, ping2] Fix Thumb-1 only == ARMv6-M & Thumb-2 only == ARMv7-M assumptions

2016-07-04 Thread Thomas Preudhomme
Ping? Best regards, Thomas On Monday 27 June 2016 16:52:50 Thomas Preudhomme wrote: > Ping? > > Best regards, > > Thomas > > On Friday 17 June 2016 18:21:44 Thomas Preudhomme wrote: > > On Wednesday 01 June 2016 10:00:52 Ramana Radhakrishnan wrote: > > >

Re: [PATCH, libgcc/ARM 1b/7] Check CLZ availability with ISA support and architecture level macros

2016-07-05 Thread Thomas Preudhomme
[Fixed subject to reflect patch] Ping? Best regards, Thomas On Monday 27 June 2016 17:51:34 Thomas Preudhomme wrote: > Hi Ramana, > > On Wednesday 01 June 2016 10:00:52 Ramana Radhakrishnan wrote: > > From here down to > > > > > -#if ((__ARM_ARCH__ &g

Re: [PATCH, ARM 6/7, ping1] Add support for CB(N)Z and (U|S)DIV to ARMv8-M Baseline

2016-07-06 Thread Thomas Preudhomme
On Friday 20 May 2016 14:22:48 Kyrill Tkachov wrote: > Hi Thomas, > > > Hmm, I'm not a fan of this change. arm_print_operand_punct_valid_p is an > implementation of a target hook that is used to validate user-provided > inline asm as well and is therefore the right place to reject such invalid >

Re: [PATCH, ARM 2/7, ping1] Add support for ARMv8-M

2016-07-07 Thread Thomas Preudhomme
On Wednesday 25 May 2016 14:32:54 Kyrill Tkachov wrote: > Hi Thomas, > > On 25/05/16 14:26, Thomas Preudhomme wrote: > > On Thursday 19 May 2016 17:59:26 Kyrill Tkachov wrote: > >> Hi Thomas, > > > > Hi Kyrill, > > > > Please find an updated p

Re: [PATCH, ARM 4/7, ping1] Factor out MOVW/MOVT availability and desirability checks

2016-07-07 Thread Thomas Preudhomme
On Friday 20 May 2016 13:41:30 Kyrill Tkachov wrote: > Hi Thomas, > > On 19/05/16 17:10, Thomas Preudhomme wrote: > > On Wednesday 18 May 2016 11:47:47 Kyrill Tkachov wrote: > >> Hi Thomas, > > > > Hi Kyrill, > > > > Please find below

Re: [PATCH, ARM 5/7] Add support for MOVT/MOVW to ARMv8-M Baseline

2016-07-07 Thread Thomas Preudhomme
gt; > > > (define_insn "*thumb1_movhi_insn" > > > > - [(set (match_operand:HI 0 "nonimmediate_operand" "=l,l,m,l*r,*h,l") > > - (match_operand:HI 1 "general_operand" "l,m,l,k*h,*r,I"))] > > + [(set (match_operand:HI 0

[PATCH, GCC/LRA] Teach LRA to not use same register value for multiple output operands of an insn

2016-07-08 Thread Thomas Preudhomme
Hi, While investigating the root cause a testsuite regression for the ARM/embedded-5-branch GCC in gcc.dg/vect/slp-perm-5.c, we found that the bug seems to also affect trunk. The bug manifests itself as an ICE in cselib due to a parallel insn with two SET to the same register. When processing t

[arm-embedded] [PATCH, libgcc/ARM 1a/6] Fix Thumb-1 only == ARMv6-M & Thumb-2 only == ARMv7-M assumptions

2016-07-11 Thread Thomas Preudhomme
m: Ramana Radhakrishnan To: Thomas Preudhomme CC: gcc-patches On Fri, Jun 17, 2016 at 6:21 PM, Thomas Preudhomme wrote: > On Wednesday 01 June 2016 10:00:52 Ramana Radhakrishnan wrote: >> Please fix up the macros, post back and redo the test. Otherwise this >> is ok from a quick

[arm-embedded] [PATCH, libgcc/ARM 1/6] Fix Thumb-1 only == ARMv6-M & Thumb-2 only == ARMv7-M assumptions

2016-07-11 Thread Thomas Preudhomme
m: Ramana Radhakrishnan To: Thomas Preudhomme CC: gcc-patches On Mon, Jun 27, 2016 at 5:51 PM, Thomas Preudhomme wrote: > Hi Ramana, > > On Wednesday 01 June 2016 10:00:52 Ramana Radhakrishnan wrote: >> >> From here down to >> >> > -#if ((

[arm-embedded] [PATCH, ARM 2/7, ping1] Add support for ARMv8-M

2016-07-11 Thread Thomas Preudhomme
We've decided to apply the following patch to ARM/embedded-6-branch. Best regards, Thomas -- Forwarded Message -- Subject: Re: [PATCH, ARM 2/7, ping1] Add support for ARMv8-M Date: Thursday 07 July 2016, 09:57:08 From: Thomas Preudhomme To: Kyrill Tkachov CC: Ri

[arm-embedded] [PATCH, ARM 4/7, ping1] Factor out MOVW/MOVT availability and desirability checks

2016-07-11 Thread Thomas Preudhomme
We've decided to apply the following patch to ARM/embedded-6-branch. Best regards, Thomas -- Forwarded Message -- Subject: Re: [PATCH, ARM 4/7, ping1] Factor out MOVW/MOVT availability and desirability checks Date: Thursday 07 July 2016, 09:59:53 From: Thomas Preudhomm

[arm-embedded] [PATCH, ARM 3/7, ping1] Fix indentation of FL_FOR_ARCH* definition after adding support for ARMv8-M

2016-07-11 Thread Thomas Preudhomme
From: Kyrill Tkachov To: Thomas Preudhomme CC: ramana.radhakrish...@arm.com, richard.earns...@arm.com, gcc- patc...@gcc.gnu.org On 18/05/16 14:45, Thomas Preudhomme wrote: > On Wednesday 18 May 2016 11:30:43 Kyrill Tkachov wrote: >> Hi Thomas, >> >> On 17/05/16 11:10, Thomas

Re: [PATCH, ARM 6/7, ping1] Add support for CB(N)Z and (U|S)DIV to ARMv8-M Baseline

2016-07-12 Thread Thomas Preudhomme
Hi Kyrill, On Friday 20 May 2016 14:22:48 Kyrill Tkachov wrote: > Hi Thomas, > > On 17/05/16 11:14, Thomas Preudhomme wrote: > > Ping? > > > > *** gcc/ChangeLog *** > > > > 2015-11-13 Thomas Preud'homme > > > > *

Re: [PATCH, ARM 6/7, ping1] Add support for CB(N)Z and (U|S)DIV to ARMv8-M Baseline

2016-07-13 Thread Thomas Preudhomme
On Tuesday 12 July 2016 15:57:41 Kyrill Tkachov wrote: > On 12/07/16 11:26, Thomas Preudhomme wrote: > > Hi Kyrill, > > Hi Thomas, > > > On Friday 20 May 2016 14:22:48 Kyrill Tkachov wrote: > >> Hi Thomas, > >> > >> On 17/05/16 11:14, Thomas

Re: [PATCH, ARM 5/7] Add support for MOVT/MOVW to ARMv8-M Baseline

2016-07-13 Thread Thomas Preudhomme
On Wednesday 13 July 2016 17:14:52 Christophe Lyon wrote: > Hi Thomas, Hi Christophe, > > I'm seeing: > gcc.target/arm/pr42574.c: syntax error in target selector > "arm_thumb1_ok && { ! arm_thumb1_movt_ok }" for " dg-do 1 compile { > arm_thumb1_ok && { ! arm_thumb1_movt_ok } } " Oops. I remembe

[PATCH, GCC/ARM testsuite] Fix pr42574.c selector syntax error

2016-07-14 Thread Thomas Preudhomme
This patch fixes a syntax error in the dg-do selector of pr42574.c: it is missing the target keyword, with the following boolean expression enclosed in curly braces. Test fails to be run without this patch and successfully pass with it. Patch is in attachment. ChangeLog entry is as follows: *

Re: [PATCH, GCC/ARM testsuite] Fix pr42574.c selector syntax error

2016-07-14 Thread Thomas Preudhomme
On Thursday 14 July 2016 10:14:52 Kyrill Tkachov wrote: > Hi Thomas, > > On 14/07/16 10:12, Thomas Preudhomme wrote: > > This patch fixes a syntax error in the dg-do selector of pr42574.c: it is > > missing the target keyword, with the following boolean expression enclose

Re: [PATCH, GCC/LRA] Teach LRA to not use same register value for multiple output operands of an insn

2016-07-14 Thread Thomas Preudhomme
On Friday 08 July 2016 09:05:55 Mike Stump wrote: > On Jul 8, 2016, at 8:07 AM, Thomas Preudhomme wrote: > > While investigating the root cause a testsuite regression for the > > ARM/embedded-5-branch GCC in gcc.dg/vect/slp-perm-5.c, we found that the > > bug seems to also a

Re: [PATCH, GCC/LRA] Teach LRA to not use same register value for multiple output operands of an insn

2016-07-14 Thread Thomas Preudhomme
On Thursday 14 July 2016 10:32:48 Thomas Preudhomme wrote: > On Friday 08 July 2016 09:05:55 Mike Stump wrote: > > On Jul 8, 2016, at 8:07 AM, Thomas Preudhomme > > wrote: > > > While investigating the root cause a testsuite regression for the > > > ARM/embedde

Re: [PATCH, ARM 7/7] Enable atomics for ARMv8-M Mainline

2016-07-14 Thread Thomas Preudhomme
Hi Kyrill, On Thursday 19 May 2016 17:18:29 Kyrill Tkachov wrote: > Hi Thomas, > > On 17/05/16 11:15, Thomas Preudhomme wrote: > > Ping? > > > > *** gcc/ChangeLog *** > > > > 2015-12-17 Thomas Preud'homme > > > > *

Re: [PATCH, ARM 7/7] Enable atomics for ARMv8-M Mainline

2016-07-14 Thread Thomas Preudhomme
On Thursday 14 July 2016 17:23:46 Kyrill Tkachov wrote: > Hi Thomas, > > On 14/07/16 14:37, Thomas Preudhomme wrote: > > Hi Kyrill, > > > > On Thursday 19 May 2016 17:18:29 Kyrill Tkachov wrote: > >> Hi Thomas, > >> > >>

[arm-embedded] Backport of resolution handling fix WRT incremental linking

2016-07-27 Thread Thomas Preudhomme
The following backport has been made from gcc-6-branch to ARM/embedded-5- branch in order to fix an ICE in LTO observed when running g++.dg/lto/20081219 testcase: 2016-07-27 Thomas Preud'homme Backport from mainline 2015-11-29 Jan Hubicka * cgraph.c (cgraph_node::m

Re: [PATCH, ARM/testsuite 6/7, ping4] Force soft float in ARMv6-M and ARMv8-M Baseline options

2016-11-28 Thread Thomas Preudhomme
Hi Richard, Ping? Best regards, Thomas On 17/11/16 20:42, Thomas Preudhomme wrote: Ping? Best regards, Thomas On 08/11/16 13:35, Thomas Preudhomme wrote: Ping, Best regards, Thomas On 02/11/16 10:04, Thomas Preudhomme wrote: Ping? Best regards, Thomas On 28/10/16 10:49, Thomas

Re: [PATCH, GCC/ARM] Fix PR77933: stack corruption on ARM when using high registers and lr

2016-11-30 Thread Thomas Preudhomme
3-1.c: New test. * gcc.target/arm/pr77933-2.c: Likewise. Best regards, Thomas On 17/11/16 20:15, Thomas Preudhomme wrote: Hi Kyrill, I've committed the following updated patch where the test is restricted to Thumb execution mode and skipping it if not possible since -mtpcs-leaf-frame is

Re: [PATCH, GCC/ARM] Fix ICE when compiling empty FIQ interrupt handler in ARM mode

2016-11-30 Thread Thomas Preudhomme
gcc/testsuite/ * gcc.target/arm/empty_fiq_handler.c: Skip if -mthumb is passed in and target is Thumb-only. Best regards, Thomas On 16/11/16 09:39, Kyrill Tkachov wrote: On 09/11/16 16:19, Thomas Preudhomme wrote: Hi, This patch fixes the following ICE when building when compili

Re: [PATCH, GCC/ARM] Fix PR77933: stack corruption on ARM when using high registers and lr

2016-11-30 Thread Thomas Preudhomme
On 30/11/16 10:04, Richard Earnshaw (lists) wrote: On 30/11/16 09:50, Thomas Preudhomme wrote: Hi, Is this ok to backport to gcc-5-branch and gcc-6-branch? Patch applies cleanly (patches attached for reference). 2016-11-17 Thomas Preud'homme Backport from mainline 2016-

Re: [PATCH, GCC/ARM] Fix PR77904: callee-saved register trashed when clobbering sp

2016-11-30 Thread Thomas Preudhomme
/arm/arm.c (thumb1_compute_save_reg_mask): Mark frame pointer in save register mask if it is needed. gcc/testsuite/ PR target/77904 * gcc.target/arm/pr77904.c: New test. Best regards, Thomas On 22/11/16 10:45, Thomas Preudhomme wrote: On 17/11/16 09:11, Kyrill Tkachov wrote: On 17/11/16 08:56, Thomas

Re: [PATCH, GCC/ARM] Fix PR77904: callee-saved register trashed when clobbering sp

2016-11-30 Thread Thomas Preudhomme
Sorry, the bug cannot be reproduced on gcc-5-branch so it's probably better to only do a backport to gcc-6-branch. Ok for a backport to gcc-6-branch? Best regards, Thomas On 30/11/16 10:42, Thomas Preudhomme wrote: Hi, Is this ok to backport to gcc-5-branch and gcc-6-branch? Patch ap

[PATCH, GCC/ARM] Add multilib mapping for Cortex-M23 & Cortex-M33

2016-11-30 Thread Thomas Preudhomme
Hi, With ARM Cortex-M23 and Cortex-M33 and the support for RM profile multilib added recently, it's time to add the corresponding CPU to architecture mappings in config/arm/t-rmprofile. Note that Cortex-M33 is mapped to ARMv8-M Mainline because there is no transitive closure of mappings and th

Fwd: Re: [PATCH, ARM/testsuite 6/7] Force soft float in ARMv6-M and ARMv8-M Baseline options

2016-12-01 Thread Thomas Preudhomme
ption. (add_options_for_arm_arch_v8m_base): Likewise. Best regards, Thomas --- Begin Message --- On 22/09/16 16:47, Richard Earnshaw (lists) wrote: On 22/09/16 15:51, Thomas Preudhomme wrote: Sorry, noticed an error in the patch. It was not caught during testing because GCC was built with --with-mode=

[arm-embedded] [PATCH, GCC/ARM] Add multilib mapping for Cortex-M23 & Cortex-M33

2016-12-01 Thread Thomas Preudhomme
Hi, We have decided to backport this patch to add multilib support for ARM Cortex-M23 and Cortex-M33 to our embedded-6-branch. *** gcc/ChangeLog *** 2016-11-30 Thomas Preud'homme * config/arm/t-rmprofile: Add mappings for Cortex-M23 and Cortex-M33. Best regards, Thomas --- Beg

[PATCH, GCC/LRA] Fix PR78617: Fix conflict detection in rematerialization

2016-12-01 Thread Thomas Preudhomme
Hi, When considering a candidate for rematerialization, LRA verifies if the candidate clobbers a live register before going forward with the rematerialization (see code starting with comment "Check clobbers do not kill something living."). To do this check, the set of live registers at any giv

[PATCH, GCC/testsuite/ARM] XFAIL optional_thumb-1 and optional_thumb-2 testcases

2016-12-02 Thread Thomas Preudhomme
Hi, The logic to make -mthumb optional for Thumb-only targets was designed to only apply when no mode is specified either at compile time or when the toolchain was configured (using --with-mode). The dg-skip-if in the testcases optional_thumb-1 and optional_thumb-2 catch the former case but no

Re: [PATCH, GCC/testsuite/ARM] XFAIL optional_thumb-1 and optional_thumb-2 testcases

2016-12-05 Thread Thomas Preudhomme
On 05/12/16 09:01, Christophe Lyon wrote: On 2 December 2016 at 17:35, Thomas Preudhomme wrote: Hi, The logic to make -mthumb optional for Thumb-only targets was designed to only apply when no mode is specified either at compile time or when the toolchain was configured (using --with-mode

Re: [PATCH, GCC/ARM 2/2, ping3] Allow combination of aprofile and rmprofile multilibs

2016-12-06 Thread Thomas Preudhomme
ls in doing that. Best regards, Thomas On 17/11/16 20:43, Thomas Preudhomme wrote: Ping? Best regards, Thomas On 08/11/16 13:36, Thomas Preudhomme wrote: Ping? Best regards, Thomas On 02/11/16 10:05, Thomas Preudhomme wrote: Ping? Best regards, Thomas On 24/10/16 09:07, Thomas Preud

Re: [PATCH, GCC/ARM, gcc-5/6-branch, ping] Fix ICE when compiling empty FIQ interrupt handler in ARM mode

2016-12-06 Thread Thomas Preudhomme
Ping? Best regards, Thomas On 30/11/16 10:20, Thomas Preudhomme wrote: Hi, Is this ok to backport this fix together with its follow-up testcase fix to gcc-5-branch and gcc-6-branch? Both patches apply cleanly (patches attached for reference). 2016-11-30 Thomas Preud'homme Bac

Re: [PATCH, GCC/ARM, gcc-5/6-branch, ping] Fix PR77933: stack corruption on ARM when using high registers and lr

2016-12-06 Thread Thomas Preudhomme
Ping? Best regards, Thomas On 30/11/16 09:50, Thomas Preudhomme wrote: Hi, Is this ok to backport to gcc-5-branch and gcc-6-branch? Patch applies cleanly (patches attached for reference). 2016-11-17 Thomas Preud'homme Backport from mainline 2016-11-17 Thomas Preud&

Re: [PATCH, GCC/ARM, gcc-5/6-branch, ping] Fix PR77904: callee-saved register trashed when clobbering sp

2016-12-06 Thread Thomas Preudhomme
Ping? Best regards, Thomas On 30/11/16 10:44, Thomas Preudhomme wrote: Sorry, the bug cannot be reproduced on gcc-5-branch so it's probably better to only do a backport to gcc-6-branch. Ok for a backport to gcc-6-branch? Best regards, Thomas On 30/11/16 10:42, Thomas Preudhomme wrote

Re: [PATCH, GCC/ARM, gcc-5/6-branch, ping] Fix ICE when compiling empty FIQ interrupt handler in ARM mode

2016-12-07 Thread Thomas Preudhomme
On 06/12/16 11:37, Kyrill Tkachov wrote: On 06/12/16 11:36, Thomas Preudhomme wrote: Ping? Best regards, Ok if bootstrap and testing on those branches doesn't reveal any issues. Both backport bootstrapped fine when configured with: --with-arch=armv7-a --with-mode=arm -with-fpu

[arm-embedded] [PATCH, GCC/LRA] Fix PR78617: Fix conflict detection in rematerialization

2016-12-07 Thread Thomas Preudhomme
We decided to also apply this patch to fix a wrong code generation to the ARM embedded 6 branch. 2016-12-07 Thomas Preud'homme Backport from mainline 2016-12-07 Thomas Preud'homme gcc/ PR rtl-optimization/78617 * lra-remat.c (do_remat): Initialize live_hard_regs from

Re: [PATCH, GCC/ARM] Fix ICE when compiling empty FIQ interrupt handler in ARM mode

2016-12-08 Thread Thomas Preudhomme
On 08/12/16 08:46, Christophe Lyon wrote: On 21 November 2016 at 12:03, Thomas Preudhomme wrote: On 17/11/16 20:04, Thomas Preudhomme wrote: Hi Christophe, On 17/11/16 13:36, Christophe Lyon wrote: On 16 November 2016 at 10:39, Kyrill Tkachov wrote: On 09/11/16 16:19, Thomas

Re: [PATCH, GCC/ARM, gcc-5/6-branch, ping2] Fix PR77933: stack corruption on ARM when using high registers and lr

2016-12-09 Thread Thomas Preudhomme
[Seeing as an RC for GCC 6.3 was suggested on IRC for mid next week] Ping? backport for 6 bootstraps on Thumb-1 and testsuite shows no regression for either 5 or 6. Bootstrap for 5 is ongoing. Best regards, Thomas On 06/12/16 11:37, Thomas Preudhomme wrote: Ping? Best regards, Thomas

Re: [PATCH, GCC/ARM, gcc-5/6-branch, ping2] Fix PR77933: stack corruption on ARM when using high registers and lr

2016-12-09 Thread Thomas Preudhomme
On 09/12/16 14:30, Kyrill Tkachov wrote: On 09/12/16 14:24, Thomas Preudhomme wrote: [Seeing as an RC for GCC 6.3 was suggested on IRC for mid next week] Ping? backport for 6 bootstraps on Thumb-1 and testsuite shows no regression for either 5 or 6. Bootstrap for 5 is ongoing. Ok

<    1   2   3   4   5   >