Re: [PATCH] [ARC] Add support for HS4x cpus.

2018-07-06 Thread Andrew Burgess
* Claudiu Zissulescu [2018-06-13 12:09:18 +0300]: > From: Claudiu Zissulescu > > This patch adds support for two ARCHS variations. > > Ok to apply? > Claudiu Sorry for the delay, this looks fine. Thanks, Andrew > > gcc/ > 2017-03-10 Claudiu Zissulescu > > * config/arc/arc-arch.

Re: [PATCH 1/4] [ARC] Add more additional register names

2018-07-25 Thread Andrew Burgess
All the patches in this series look fine. Thanks, Andrew * Claudiu Zissulescu [2018-07-16 15:29:42 +0300]: > From: claziss > > gcc/ > 2017-06-14 Claudiu Zissulescu > > * config/arc/arc.h (ADDITIONAL_REGISTER_NAMES): Add additional > register names. > --- > gcc/config/arc/arc

[PATCH] gdb: Add a dependency between gdb and libbacktrace

2021-08-30 Thread Andrew Burgess
I plan to make use of libbacktrace within GDB. I believe that the patch below needs to be merged into GCCs toplevel directory and then back-ported to the binutils-gdb repository. Is this OK to merge? Thanks, Andrew --- GDB is going to start using libbacktrace, so add a build dependency between

Re: [PATCH 2/2] gcc/riscv: Add a mechanism to remove some calls to _riscv_save_0

2019-10-28 Thread Andrew Burgess
* Jim Wilson [2019-10-22 16:34:53 -0700]: > On Mon, Oct 21, 2019 at 5:26 AM Andrew Burgess > wrote: > > Below is a new versions of this patch, I believe that this addresses > > the review comments from the earlier version. In addition this has > > been tested using Jim&

[PATCH] gcc: Add new configure options to allow static libraries to be selected

2020-01-22 Thread Andrew Burgess
The motivation behind this change is to make it easier for a user to link against static libraries on a target where dynamic libraries are the default library type (for example GNU/Linux). Further, my motivation is really for linking libraries into GDB, however, the binutils-gdb/config/ directory

Re: [PATCH] gcc: Add new configure options to allow static libraries to be selected

2020-01-27 Thread Andrew Burgess
* Jeff Law [2020-01-22 13:52:27 -0700]: > On Wed, 2020-01-22 at 15:39 +0000, Andrew Burgess wrote: > > The motivation behind this change is to make it easier for a user to > > link against static libraries on a target where dynamic libraries are > > the default library

[PATCH] libiberty/hashtab: More const parameters

2020-01-27 Thread Andrew Burgess
I know that the tree's currently closed to non-bugfix changes, but I was hoping this might be accpeted anyway so it can be backported to binutils-gdb. --- Makes some parameters const in libiberty's hashtab library. include/ChangeLog: * hashtab.h (htab_remove_elt): Make a parameter const

Re: [PATCH] gcc: Add new configure options to allow static libraries to be selected

2020-01-30 Thread Andrew Burgess
* Iain Sandoe [2020-01-28 10:34:52 +]: > Hi Andrew, > > Andrew Burgess wrote: > > > * Jeff Law [2020-01-22 13:52:27 -0700]: > > > > > On Wed, 2020-01-22 at 15:39 +, Andrew Burgess wrote: > > > > The motivation behind this change is

Re: [PATCH] gcc: Add new configure options to allow static libraries to be selected

2020-01-30 Thread Andrew Burgess
* Iain Sandoe [2020-01-30 15:21:08 +]: > >> Can you clarify why there’s no need to match the configury changes in > >> libcpp > >> / gcc / libstdc++ ? > > > > This is the same issue that Tobias pointed out, and was a result of me > > incorrectly trying to regenerate the configure files. >

Re: [PATCH] gcc: Add new configure options to allow static libraries to be selected

2020-01-30 Thread Andrew Burgess
Here's a cleaned up version of the previous patch I posted. If Iain reports this fixes the regressions he saw then I will push this. All feedback welcome. Thanks, Andrew --- >From 876996580d64d31407357787fc5df7bd5699b2c0 Mon Sep 17 00:00:00 2001 From: Andrew Burgess Date: Thu, 30 Jan

Re: [PATCH] gcc: Add new configure options to allow static libraries to be selected

2020-01-31 Thread Andrew Burgess
* Iain Sandoe [2020-01-31 20:18:58 +]: > Hello Andrew, > > Andrew Burgess wrote: > > > Here's a cleaned up version of the previous patch I posted. If Iain > > reports this fixes the regressions he saw then I will push this. > > I applied this to r10-6

[PATCH] top-level configure: setup target_configdirs based on repository

2021-09-22 Thread Andrew Burgess
The top-level configure script is shared between the gcc repository and the binutils-gdb repository. The target_configdirs variable in the configure.ac script, defines sub-directories that contain components that should be built for the target using the target tools. Some components, e.g. zlib, a

Re: [PATCH] top-level configure: setup target_configdirs based on repository

2021-09-24 Thread Andrew Burgess
* Richard Biener [2021-09-23 10:53:16 +0200]: > On Wed, Sep 22, 2021 at 5:47 PM Andrew Burgess > wrote: > > > > The top-level configure script is shared between the gcc repository > > and the binutils-gdb repository. > > > > The target_configdirs variable

[PATCHv2] top-level configure: setup target_configdirs based on repository

2021-09-24 Thread Andrew Burgess
* Thomas Schwinge [2021-09-23 11:29:05 +0200]: > Hi! > > I only had a curious look here; hope that's still useful. > > On 2021-09-22T16:30:42+0100, Andrew Burgess > wrote: > > The top-level configure script is shared between the gcc repository > > and the b

[PATCH] top-level: merge Makefile.def patches from binutils-gdb repository

2021-09-24 Thread Andrew Burgess
commit 755ba58ebef02e1be9fc6770d00243ba6ed0223c Date: Thu Mar 18 12:37:52 2021 + Add install dependencies for ld -> bfd and libctf -> bfd OK to merge? 2021-09-07 Andrew Burgess Merge from binutils-gdb: 2021-09-08 Nick Alcock PR libctf

Re: [PATCH] top-level: merge Makefile.def patches from binutils-gdb repository

2021-09-24 Thread Andrew Burgess
* Richard Biener [2021-09-24 13:58:20 +0200]: > On Fri, Sep 24, 2021 at 12:49 PM Andrew Burgess > wrote: > > > > This commit back-ports two patches to Makefile.def from the > > binutils-gdb repository, these patches were committed over there > > without fi

Re: [PATCHv2] top-level configure: setup target_configdirs based on repository

2021-09-28 Thread Andrew Burgess
* Richard Biener [2021-09-27 10:23:50 +0200]: > On Fri, Sep 24, 2021 at 12:34 PM Andrew Burgess > wrote: > > > > * Thomas Schwinge [2021-09-23 11:29:05 +0200]: > > > > > Hi! > > > > > > I only had a curious look here; hope that's still

Re: [PATCH 0/13] v2 warning control by group and location (PR 74765)

2021-07-28 Thread Andrew Burgess
* Martin Sebor via Gcc-patches [2021-07-19 09:08:35 -0600]: > On 7/17/21 2:36 PM, Jan-Benedict Glaw wrote: > > Hi Martin! > > > > On Fri, 2021-06-04 15:27:04 -0600, Martin Sebor wrote: > > > This is a revised patch series to add warning control by group and > > > location, updated based on fee

Re: [PATCH 0/13] v2 warning control by group and location (PR 74765)

2021-07-29 Thread Andrew Burgess
* Martin Sebor [2021-07-28 10:16:59 -0600]: > On 7/28/21 5:14 AM, Andrew Burgess wrote: > > * Martin Sebor via Gcc-patches [2021-07-19 > > 09:08:35 -0600]: > > > > > On 7/17/21 2:36 PM, Jan-Benedict Glaw wrote: > > > > Hi Martin! > > > &

Re: [PATCH] Revert "Pass GUILE down to subdirectories"

2024-03-06 Thread Andrew Burgess
our guile scripts using a suitable version of guile. This could be applied irrespective of whether you revert b7e5a29602143b53267efcd9c8d5ecc78cd5a62f or not. What do you think? Thanks, Andrew --- commit 6d326010217a2c94ad27d4063f90f689f7c11615 Author: Andrew Burgess Date: Tue Mar 5 12:35:05 2024

Re: [PATCH] Revert "Pass GUILE down to subdirectories"

2024-03-11 Thread Andrew Burgess
Tom Tromey writes: >>>>>> "Andrew" == Andrew Burgess writes: > > Andrew> Tom Tromey writes: >>> This reverts commit b7e5a29602143b53267efcd9c8d5ecc78cd5a62f. >>> >>> This patch caused problems for some users when building gdb,

Re: [PATCH] Revert "Pass GUILE down to subdirectories"

2024-02-07 Thread Andrew Burgess
Tom Tromey writes: > This reverts commit b7e5a29602143b53267efcd9c8d5ecc78cd5a62f. > > This patch caused problems for some users when building gdb, because > it would cause 'guild' to be invoked with the wrong versin of guile. > On the whole it seems simpler to just back this out. > > * Mak

Re: [PATCH] Revert "Pass GUILE down to subdirectories"

2024-02-08 Thread Andrew Burgess
Christian Biesinger writes: > On Mon, Jan 22, 2024 at 7:21 PM Tom Tromey wrote: > >> This reverts commit b7e5a29602143b53267efcd9c8d5ecc78cd5a62f. >> >> This patch caused problems for some users when building gdb, because >> it would cause 'guild' to be invoked with the wrong versin of guile. >>

[PATCHv2 0/2] Changes to libiberty buildargv

2024-02-10 Thread Andrew Burgess
I realise that these patches are not going to get merged until GCC is back in stage 1, but thought I'd post my latest set of changes for the libiberty buildargv function. Patch #1 is unchanged from V1. Patch #2 is new in V2. Thanks, Andrew --- Andrew Burgess (2): libiberty/buildargv:

[PATCHv2 2/2] libiberty/buildargv: handle input consisting of only white space

2024-02-10 Thread Andrew Burgess
GDB makes use of the libiberty function buildargv for splitting the inferior (program being debugged) argument string in the case where the inferior is not being started under a shell. I have recently been working to improve this area of GDB, and noticed some unexpected behaviour to the libiberty

[PATCHv2 1/2] libiberty/buildargv: POSIX behaviour for backslash handling

2024-02-10 Thread Andrew Burgess
GDB makes use of the libiberty function buildargv for splitting the inferior (program being debugged) argument string in the case where the inferior is not being started under a shell. I have recently been working to improve this area of GDB, and have tracked done some of the unexpected behaviour

Re: [PATCH] Pass GUILE down to subdirectories

2024-01-13 Thread Andrew Burgess
ne 'GUILE = guile' to instead be just 'GUILE ='. With this in place I can still override the choice of guile executable with: make GUILE=guile but, if I don't do this then instead of forcing 'guile' as the default, we allow the guild compiler t

Ping: Re: [PATCH] libiberty/buildargv: POSIX behaviour for backslash handling

2024-01-02 Thread Andrew Burgess
Ping! Thanks, Andrew Andrew Burgess writes: > GDB makes use of the libiberty function buildargv for splitting the > inferior (program being debugged) argument string in the case where > the inferior is not being started under a shell. > > I have recently been working to impro

[PATCH] libiberty/buildargv: POSIX behaviour for backslash handling

2023-12-06 Thread Andrew Burgess
GDB makes use of the libiberty function buildargv for splitting the inferior (program being debugged) argument string in the case where the inferior is not being started under a shell. I have recently been working to improve this area of GDB, and have tracked done some of the unexpected behaviour

Re: [PATCHv2 2/2] libiberty/buildargv: handle input consisting of only white space

2024-06-28 Thread Andrew Burgess
Hi, Am I OK to push these patches given the testing went OK? I'm thinking probably, but I don't want to overstep. Thanks, Andrew Andrew Burgess writes: > Jeff Law writes: > >> On 2/10/24 10:26 AM, Andrew Burgess wrote: >>> GDB makes use of the libiberty f

Re: [PATCHv2 2/2] libiberty/buildargv: handle input consisting of only white space

2024-07-16 Thread Andrew Burgess
Jeff Law writes: > On 6/11/24 4:39 AM, Andrew Burgess wrote: > >> >> Jeff, >> >> Thanks for looking these patches over. >> >> For testing, using current(ish) gcc HEAD, on x86-64 GNU/Linux, I: >> >>../src/configure --prefix=$(cd .. &a

Re: [PATCHv2 2/2] libiberty/buildargv: handle input consisting of only white space

2024-06-11 Thread Andrew Burgess
Jeff Law writes: > On 2/10/24 10:26 AM, Andrew Burgess wrote: >> GDB makes use of the libiberty function buildargv for splitting the >> inferior (program being debugged) argument string in the case where >> the inferior is not being started under a shell. >> >>

Re: [PATCHv2 2/2] libiberty/buildargv: handle input consisting of only white space

2024-06-11 Thread Andrew Burgess
Jeff Law writes: > On 2/10/24 10:26 AM, Andrew Burgess wrote: >> GDB makes use of the libiberty function buildargv for splitting the >> inferior (program being debugged) argument string in the case where >> the inferior is not being started under a shell. >> >>

Re: [PATCHv2 2/2] libiberty/buildargv: handle input consisting of only white space

2024-07-29 Thread Andrew Burgess
Thomas Schwinge writes: > Hi! > > On 2024-02-10T17:26:01+0000, Andrew Burgess wrote: >> --- a/libiberty/argv.c >> +++ b/libiberty/argv.c > >> @@ -439,17 +442,8 @@ expandargv (int *argcp, char ***argvp) >> } >>/* Add a NUL t

Re: [PATCHv2 2/2] libiberty/buildargv: handle input consisting of only white space

2024-08-05 Thread Andrew Burgess
Jeff Law writes: > On 7/29/24 6:51 AM, Andrew Burgess wrote: >> Thomas Schwinge writes: >> >>> Hi! >>> >>> On 2024-02-10T17:26:01+, Andrew Burgess wrote: >>>> --- a/libiberty/argv.c >>>> +++ b/libiberty/argv.c &

Re: [PATCHv2 0/2] Changes to libiberty buildargv

2024-04-27 Thread Andrew Burgess
Ping! Any thoughts on these patches? Thanks, Andrew Andrew Burgess writes: > I realise that these patches are not going to get merged until GCC is > back in stage 1, but thought I'd post my latest set of changes for the > libiberty buildargv function. > > Patch #1

[PATCH 03/10] gcc/arc: generate jump tables in code section for nps400

2016-03-04 Thread Andrew Burgess
@@ -1,3 +1,10 @@ +2015-09-08 Andrew Burgess + Joern Rennecke + Noam Camus + + * config/arc/arc.opt (TARGET_CASE_VECTOR_PC_RELATIVE): Default on + for NPS400. + 2016-02-02 Joern Rennecke Andrew Burgess diff --git a/gcc/config/arc/arc.opt b/gcc

[PATCH 01/10] gcc: Add support for mellanox nps400 arc variant

2016-03-04 Thread Andrew Burgess
.286f2dd 100644 --- a/gcc/ChangeLog.NPS400 +++ b/gcc/ChangeLog.NPS400 @@ -1,3 +1,8 @@ +2016-02-01 Andrew Burgess + + * config.gcc: Add support for arc*-mellanox-* nps400 targets. + * config/arc/t-nps400: New file. + 2016-02-02 Andrew Burgess * common.opt (ffat-lto-objects): Make t

[PATCH 04/10] gcc/arc: Replace rI constraint with r & Cm2 for ld and update insns

2016-03-04 Thread Andrew Burgess
--- 2 files changed, 60 insertions(+), 48 deletions(-) diff --git a/gcc/ChangeLog.NPS400 b/gcc/ChangeLog.NPS400 index 716e413..71463df 100644 --- a/gcc/ChangeLog.NPS400 +++ b/gcc/ChangeLog.NPS400 @@ -1,3 +1,15 @@ +2016-02-01 Andrew Burgess + + * config/arc/arc

[PATCH 00/10] ARC: Add support for NPS400 variant

2016-03-04 Thread Andrew Burgess
that I could get an early review, especially on patch #1, the build infrastructure, then if I need to rework anything I can get started on it sooner. I've run regression tests against a standard arc-elf target, and the results look good. All feedback appreciated. Thanks, Andrew --- Andrew Bu

[PATCH 05/10] gcc/arc: convert some constraints to define_constraint

2016-03-04 Thread Andrew Burgess
, 7 deletions(-) diff --git a/gcc/ChangeLog.NPS400 b/gcc/ChangeLog.NPS400 index 71463df..5d1533c 100644 --- a/gcc/ChangeLog.NPS400 +++ b/gcc/ChangeLog.NPS400 @@ -1,3 +1,10 @@ +2016-03-01 Joern Rennecke + Andrew Burgess + + * config/arc/constraints.md (Usd): Convert to define_constraint.

[PATCH 02/10] gcc/arc: Add -munaligned-access option for nps400

2016-03-04 Thread Andrew Burgess
/gcc/ChangeLog.NPS400 b/gcc/ChangeLog.NPS400 index 286f2dd..0281640 100644 --- a/gcc/ChangeLog.NPS400 +++ b/gcc/ChangeLog.NPS400 @@ -1,3 +1,14 @@ +2016-02-02 Joern Rennecke + Andrew Burgess + + * config/arc/arc.h (ARC_NPS400): Define if not already defined

[PATCH 09/10] gcc/arc: Add an nps400 specific testcase

2016-03-04 Thread Andrew Burgess
..20f88d0 100644 --- a/gcc/testsuite/ChangeLog.NPS400 +++ b/gcc/testsuite/ChangeLog.NPS400 @@ -1,3 +1,7 @@ +2016-02-06 Andrew Burgess + + * gcc.target/arc/nps400-1.c: New file. + 2016-01-19 Andrew Burgess * gcc.target/arc/movh_cl-1.c: New file. diff --git a/gcc/testsuite

[PATCH 08/10] gcc/arc: Mask integer 'L' operands to 32-bit

2016-03-04 Thread Andrew Burgess
x 8229d67..146370c 100644 --- a/gcc/ChangeLog.NPS400 +++ b/gcc/ChangeLog.NPS400 @@ -1,3 +1,9 @@ +2016-01-19 Joern Rennecke + Andrew Burgess + + * config/arc/arc.c (arc_print_operand): Print integer 'L' operands + as 32-bits. + 2013-02-19 Joern Rennecke

[PATCH 06/10] gcc/arc: Add support for nps400 cmem xld/xst instructions

2016-03-04 Thread Andrew Burgess
/testsuite/gcc.target/arc/cmem-st.inc diff --git a/gcc/ChangeLog.NPS400 b/gcc/ChangeLog.NPS400 index 5d1533c..2a0f820 100644 --- a/gcc/ChangeLog.NPS400 +++ b/gcc/ChangeLog.NPS400 @@ -1,3 +1,31 @@ +2013-08-31 Joern Rennecke + Andrew Burgess + + * config/arc/ar

[PATCH 10/10] gcc/arc: Add __NPS400__ define for nps400 targets

2016-03-04 Thread Andrew Burgess
/gcc/ChangeLog.NPS400 b/gcc/ChangeLog.NPS400 index 146370c..e1889b9 100644 --- a/gcc/ChangeLog.NPS400 +++ b/gcc/ChangeLog.NPS400 @@ -1,3 +1,7 @@ +2016-02-08 Andrew Burgess + + * conig/arc/arc.h (TARGET_CPU_CPP_BUILTINS): Add __NPS400__. + 2016-01-19 Joern Rennecke Andrew

[PATCH 07/10] gcc/arc: Add nps400 bitops support

2016-03-04 Thread Andrew Burgess
rget/arc/mrgb-1.c diff --git a/gcc/ChangeLog.NPS400 b/gcc/ChangeLog.NPS400 index 2a0f820..8229d67 100644 --- a/gcc/ChangeLog.NPS400 +++ b/gcc/ChangeLog.NPS400 @@ -1,3 +1,45 @@ +2013-02-19 Joern Rennecke + Andrew Burgess + + * config/arc/arc.c (arc_conditional_register_usage): Take +

[PATCH] contrib: Add KPASS support to dg-extract-results.{sh,py}

2019-10-03 Thread Andrew Burgess
My motivation for the patch below comes from GDB. The binutils-gdb project maintains a copy of the contrib directory that it keeps in sync with upstream GCC, and patches to contrib/ are ideally first applied to GCC then backported to binutils-gdb. This patch extends the dg-extract-results.* scrip

Re: [PATCH] RISC-V: Include more registers in SIBCALL_REGS.

2019-10-17 Thread Andrew Burgess
* Jim Wilson [2019-10-16 14:04:45 -0700]: > This finishes the part 1 of 2 patch submitted by Andrew Burgess on Aug 19. > This adds the argument registers but not t0 (aka x5) to SIBCALL_REGS. It > also adds the missing riscv_regno_to_class change. > > Tested with cross riscv32-

Re: [PATCH] RISC-V: Include more registers in SIBCALL_REGS.

2019-10-17 Thread Andrew Burgess
* Jim Wilson [2019-10-17 14:55:34 -0700]: > On Thu, Oct 17, 2019 at 7:09 AM Andrew Burgess > wrote: > > I'm still working on part 2, I'm hoping to have a revised patch posted > > by Monday next week. > > I started looking at the part 2 patch also

Re: [PATCH 2/2] gcc/riscv: Add a mechanism to remove some calls to _riscv_save_0

2019-10-21 Thread Andrew Burgess
don't know if a separate flag is required. Thanks, Andrew --- commit f1b824e94f3ea396bd0ef46692d5211f855d4b4c Author: Andrew Burgess Date: Thu Jul 18 16:03:10 2019 +0100 gcc/riscv: Add a mechanism to remove some calls to _riscv_save_0 When using the -msave-restore flag we

[PATCH] contrib: Handle GDB specific test result types

2020-05-11 Thread Andrew Burgess
This commit is for the benefit of GDB, but as the binutils-gdb repository shares the contrib/ directory with gcc, this commit must first be applied to gcc then copied back to binutils-gdb. This commit extends the two scripts contrib/dg-extract-results.{py,sh} to handle some new, GDB specific test

Re: [PATCH 1/7] [ARC] Add support for naked functions.

2017-06-16 Thread Andrew Burgess
* Claudiu Zissulescu [2017-06-01 15:34:51 +0200]: > gcc/ > 2016-12-13 Claudiu Zissulescu > Andrew Burgess > > * config/arc/arc-protos.h (arc_compute_function_type): Change prototype. > (arc_return_address_register): New function. >

Re: [PATCH 2/7] [ARC] Define ADDITIONAL_REGISTER_NAMES.

2017-06-16 Thread Andrew Burgess
* Claudiu Zissulescu [2017-06-01 15:34:52 +0200]: > This macro is needed to be used with -ffixed- option, and inline asm. > > gcc/ > 2017-01-09 Claudiu Zissulescu > > * config/arc/arc.h (ADDITIONAL_REGISTER_NAMES): Define. This looks fine and could be applied straight away I think.

Re: [PATCH 3/7] [ARC] [LRA] Fix tests asm constraints.

2017-06-16 Thread Andrew Burgess
* Claudiu Zissulescu [2017-06-01 15:34:53 +0200]: > LRA doesn't like the 'X' constraint as used in our tests, remove it. > > gcc/testsuite > 2017-01-09 Claudiu Zissulescu > > * gcc.target/arc/mulsi3_highpart-1.c: Remove 'X' constraint. > * gcc.target/arc/mulsi3_highpart-2.c: Lik

Re: [PATCH] [ARC] Reimplement exception handling support.

2017-11-07 Thread Andrew Burgess
is fixed in this commit too. can probably be deleted completely. Otherwise, I'm happy. Thanks, Andrew > Claudiu > > 2016-06-09 Claudiu Zissulescu > Andrew Burgess > > * config/arc/arc-protos.h (arc_compute_frame_size): Delete >

Re: [PATCH 01/10] [ARC][LRA] Use TARGET_CANNOT_SUBSTITUTE_MEM_EQUIV.

2017-11-27 Thread Andrew Burgess
* Claudiu Zissulescu [2017-11-27 12:09:50 +0100]: > From: Claudiu Zissulescu > > Sometimes the memory equivalent is not valid due to a large offset. > For example replacing the ap register with its fp/sp-equivalent during > LRA step. To solve this we introduced TARGET_CANNOT_SUBSTITUTE_MEM_EQU

Re: [PATCH 02/10] [ARC][ZOL] Update uses for hw-loop labels.

2017-11-27 Thread Andrew Burgess
* Claudiu Zissulescu [2017-11-27 12:09:51 +0100]: > From: claziss > > Make sure we mark the hw-loop labels as beeing used. > > gcc/ > 2017-09-19 Claudiu Zissulescu > > * config/arc/arc.c (hwloop_optimize): Update hw-loop's end/start > labels number of usages. > > gcc/testsuit

Re: [PATCH 03/10] [ARC] Don't allow the last ZOL insn to be in a delay slot.

2017-11-27 Thread Andrew Burgess
* Claudiu Zissulescu [2017-11-27 12:09:52 +0100]: > From: Claudiu Zissulescu > > The ARC ZOL implementation doesn't allow the last instruction to be a > control instruction or part of a delay slot. Thus, we add a note to > the last ZOL instruction which will prevent it to finish into a delay

Re: [PATCH 04/10] [ARC] Add ARCv2 core3 tune option.

2017-11-27 Thread Andrew Burgess
* Claudiu Zissulescu [2017-11-27 12:09:53 +0100]: > From: claziss > > ARCv2 Core3 cpus are comming with dbnz support. Add this feature on > the tune option. > > gcc/ > 2017-09-14 Claudiu Zissulescu > > * config/arc/arc-arch.h (arc_tune_attr): Add ARC_TUNE_CORE_3. > * config/ar

Re: [PATCH 05/10] [ARC] Add trap instruction.

2017-11-27 Thread Andrew Burgess
* Claudiu Zissulescu [2017-11-27 12:09:54 +0100]: > From: Claudiu Zissulescu > > 2017-11-07 Claudiu Zissulescu > > * config/arc/arc.md (trap): New pattern. Looks good. Thanks, Andrew > --- > gcc/config/arc/arc.md | 7 +++ > 1 file changed, 7 insertions(+) > > diff --git a/gc

Re: [PATCH 06/10] [ARC] Update legitimate constant hook.

2017-12-07 Thread Andrew Burgess
* Claudiu Zissulescu [2017-11-27 12:09:55 +0100]: > From: Claudiu Zissulescu > > Make sure we check the constants in all cases. > > gcc/ > 2017-10-14 Claudiu Zissulescu > > * config/arc/arc.c (arc_legitimate_constant_p): Always check all > constants. > > testsuite/ > 2017-10-

Re: [PATCH 07/10] [ARC][FIX] Consider command line ffixed- option.

2017-12-07 Thread Andrew Burgess
* Claudiu Zissulescu [2017-11-27 12:09:56 +0100]: > From: claziss > > Track which regs are set fixed/call saved/call used from commnad line. > Do not try to override their properties if user says otherwise. > > gcc/ > 2017-06-08 Claudiu Zissulescu > > * config/arc/arc.c (overridereg

Re: [PATCH 09/10] [ARC] Update (u)maddsidi patterns.

2017-12-07 Thread Andrew Burgess
* Claudiu Zissulescu [2017-11-27 12:09:58 +0100]: > From: claziss > > The accumulator registers are freely used by the compiler. However, > there are a number of instructions which are having an intrinsic use > of these registers. Update patterns to inform the compiler which ones. > > gcc/ >

Re: [PATCH 08/10] [ARC] Enable unaligned access.

2018-01-02 Thread Andrew Burgess
* Claudiu Zissulescu [2017-11-27 12:09:57 +0100]: > From: Claudiu Zissulescu > > Use munaligned-access to control if we can have unaligned accesses. For ARC > HS family unaligned access is always on. > > 2017-10-19 Claudiu Zissulescu > > * config/arc/arc-c.def (__ARC_UNALIGNED__):

Re: [PATCH 10/10] [ARC] Revamp trampoline implementation.

2018-01-02 Thread Andrew Burgess
* Claudiu Zissulescu [2017-11-27 12:09:59 +0100]: > From: Claudiu Zissulescu > > The new implementation attempts to clean up the existing trampoline > implementation for ARC making it to work for linux type of systems. > > gcc/ > 2017-11-10 Claudiu Zissulescu > > * config/arc/arc.c

Re: [PATCH 1/2] [ARC][QuarkSE] Update

2018-06-04 Thread Andrew Burgess
* Claudiu Zissulescu [2018-05-21 13:20:28 +0300]: > From: claziss > > QuarkSE has lp_count width set to 16 bits. Update the compiler to > consider it. > > Ok to apply? > Claudiu > > gcc/ > 2017-07-11 Claudiu Zissulescu > > * config/arc/arc-arch.h (arc_extras): New enum. > (arc

Re: [PATCH 2/2] [ARC] Update warning messges.

2018-06-04 Thread Andrew Burgess
* Claudiu Zissulescu [2018-05-21 13:20:29 +0300]: > From: claziss > > When we pass an mcpu to the compiler we have two types of (hardware > configuration) flags that are set: > > 1. Architecture specific, for example code-density is always enabled > for ARCHS architectures. These options are o

Re: [PATCH 4/6] [ARC] Define LINK_GCC_C_SEQUENCE_SPEC.

2018-06-12 Thread Andrew Burgess
* Claudiu Zissulescu [2018-05-21 13:18:37 +0300]: > From: claziss > > If no specs file is provided, default to nosys library. > > Ok to apply? Looks good. Thanks, Andrew > Claudiu > > gcc/ > 2017-05-03 Claudiu Zissulescu > > * config/arc/elf.h (LINK_GCC_C_SEQUENCE_SPEC): Define.

Re: [PATCH 3/6] [ARC] SYNC instruction is valid on all ARC cores.

2018-06-12 Thread Andrew Burgess
* Claudiu Zissulescu [2018-05-21 13:18:36 +0300]: > From: claziss > > The sync instruction is part of all ARC architectures. Fix this in the > compiler. > > Ok to apply? Looks good. Thanks, Andrew > Claudiu > > gcc/ > 2017-05-03 Claudiu Zissulescu > > * config/arc/builtins.def

Re: [PATCH 1/6] [ARC] Add modes scheduler HS

2018-06-12 Thread Andrew Burgess
* Claudiu Zissulescu [2018-05-21 13:18:34 +0300]: > From: Claudiu Zissulescu > > An update on how the instructions are scheduled for HS processor. > > Ok to apply? Looks good. Thanks, Andrew > Claudiu > > --- > gcc/config/arc/arcHS.md | 21 +++-- > 1 file changed, 15 inse

Re: [PATCH 2/6] [ARC] Allow EX instruction for ARC700 and ARCv2.

2018-06-12 Thread Andrew Burgess
* Claudiu Zissulescu [2018-05-21 13:18:35 +0300]: > From: claziss > > The EX instruction is base line for both architectures. Reflect this in the > compiler. > > OK to apply? Looks good. Thanks, Andrew > Claudiu > > gcc/ > 2017-05-02 Claudiu Zissulescu > > * config/arc/arc.c (a

Re: [PATCH 5/6] [ARC] Update fma expansions.

2018-06-12 Thread Andrew Burgess
* Claudiu Zissulescu [2018-05-21 13:18:38 +0300]: > Make sure only one operand has an immediate. > > OK to apply? Looks good. Thanks, Andrew > Claudiu > > gcc/ > 2018-03-21 Claudiu Zissulescu > > * config/arc/fpu.md (fmasf4): Force operand to register. > (fnmasf4): Likewise.

Re: [PATCH 6/6] [ARC] Reimplement return padding operation for ARC700.

2018-06-12 Thread Andrew Burgess
* Claudiu Zissulescu [2018-05-21 13:18:39 +0300]: > From: Claudiu Zissulescu > > For ARC700, adding padding if necessary to avoid a mispredict. A > return could happen immediately after the function start. A > call/return and return/return must be 6 bytes apart to avoid > mispredict. > > The

Re: [PATCH 00/10][ARC] Critical fixes

2018-01-16 Thread Andrew Burgess
* Claudiu Zissulescu [2018-01-08 15:18:30 +]: > > [ARC][LRA] Use TARGET_CANNOT_SUBSTITUTE_MEM_EQUIV. > > [ARC] Don't allow the last ZOL insn to be in a delay slot. > > [ARC] Add trap instruction. > > [ARC] Update legitimate constant hook. > > [ARC] Enable unaligned access. > > [A

Re: [PATCH 1/6] [ARC] Add JLI support.

2018-01-16 Thread Andrew Burgess
* Claudiu Zissulescu [2017-11-02 13:30:30 +0100]: > The ARCv2 ISA provides the JLI instruction, which is two-byte instructions > that can be used to reduce code size in an application. To make use of it, > we provide two new function attributes 'jli_always' and 'jli_fixed' which > will force the

Re: [PATCH 2/6] [ARC] Add SJLI support.

2018-01-16 Thread Andrew Burgess
* Claudiu Zissulescu [2017-11-02 13:30:31 +0100]: > gcc/ > 2017-02-20 Claudiu Zissulescu > > * config/arc/arc-protos.h: Add arc_is_secure_call_p proto. > * config/arc/arc.c (arc_handle_secure_attribute): New function. > (arc_attribute_table): Add 'secure_call' attribute. >

Re: [PATCH 3/6] [ARC] Add support for "register file 16" reduced register set

2018-01-16 Thread Andrew Burgess
* Claudiu Zissulescu [2017-11-02 13:30:32 +0100]: > gcc/ > 2017-03-20 Claudiu Zissulescu > > * config/arc/arc-arches.def: Option mrf16 valid for all > architectures. > * config/arc/arc-c.def (__ARC_RF16__): New predefined macro. > * config/arc/arc-cpus.def (em_mini):

Re: [PATCH 4/6] [ARC] Rework delegitimate_address hook

2018-01-23 Thread Andrew Burgess
* Claudiu Zissulescu [2017-11-02 13:30:33 +0100]: > From: claziss > > Delegitimize address is used to undo the obfuscating effect of PIC > addresses, returning the address in a way which is understood by the > compiler. > > gcc/ > 2017-04-25 Claudiu Zissulescu > > * config/arc/arc.c

Re: [PATCH 5/6] [ARC] Add 'uncached' attribute.

2018-01-29 Thread Andrew Burgess
* Claudiu Zissulescu [2017-11-02 13:30:34 +0100]: > From: claziss > > The _Uncached type qualifier can be used to bypass the cache without > resorting to declaring variables as volatile. > > gcc/ > 2017-07-12 Claudiu Zissulescu > > * config/arc/arc-protos.h (arc_is_uncached_mem_p):

Re: [PATCH 6/6] [ARC] Add 'aux' variable attribute.

2018-01-29 Thread Andrew Burgess
* Claudiu Zissulescu [2017-11-02 13:30:35 +0100]: > From: claziss > > The 'aux' variable attribute is used to directly access the auxiliary > register space from C. > > gcc/ > 2017-07-25 Claudiu Zissulescu > > * config/arc/arc.c (arc_handle_aux_attribute): New function. > (arc

Re: [PATCH] [ARC] Cleanup unused functions.

2018-03-05 Thread Andrew Burgess
* Claudiu Zissulescu [2018-02-20 15:38:32 +0200]: > From: Claudiu Zissulescu > > Cleanup unsed functions and macros. > > OK to apply? > Claudiu Looks good. Nice cleanup. Thanks, Andrew > > gcc/ > 2018-01-26 Claudiu Zissulescu > > * config/arc/arc.c (arc_finalize_pic): Remove fu

Re: [PATCH] [ARC] Add/update combiner patterns.

2018-04-07 Thread Andrew Burgess
* Claudiu Zissulescu [2018-03-07 13:59:03 +0200]: > From: claziss > > Hi Andrew, > > Please find the following patch which improves generating more instructions > with .f flag (i.e., compare with zero). > > Ok to apply? Looks good. Thanks. Andrew > Claudiu > > gcc/ > 2018-01-26 Claudiu

Re: [PATCH] [ARC]: core3 features are default for core4

2018-09-11 Thread Andrew Burgess
* Vineet Gupta [2018-09-10 11:26:35 -0700]: >* config/arc/arc.c: object attributes for core4 not reflected correctly >* config/arc/arc.h: Don't restrict DBNZ to core3 (core4 includes core3) > > Signed-off-by: Vineet Gupta > --- > gcc/ChangeLog| 7 +++ > gcc/config/arc/arc.c

Re: [PATCH 1/2] [ARC] Check for odd-even register when emitting double mac ops.

2018-09-20 Thread Andrew Burgess
* Claudiu Zissulescu [2018-09-17 15:50:26 +0300]: > Avoid generate dmac instructions when the register is not odd-even, > use instead the equivalent mac instruction. > > gcc/ > Claudiu Zissulescu > > * config/arc/arc.md (maddsidi4_split): Don't use dmac if the > destination

Re: [PATCH 2/2] [ARC] Avoid specific constants to end in limm field.

2018-09-20 Thread Andrew Burgess
* Claudiu Zissulescu [2018-09-17 15:50:27 +0300]: > The 3-operand instructions accepts to place an immediate into the > second operand. However, this immediate will end up in the long > immediate field. This patch avoids constants to end up in the limm > field for particular instructions when com

Re: [PATCH 2/6] [ARC] Cleanup TLS implementation.

2018-10-11 Thread Andrew Burgess
* Claudiu Zissulescu [2018-10-10 11:00:12 +0300]: > Cleanup TLS implementation and add a number of tests. > > gcc/ > 2018-07-25 Claudiu Zissulescu > > * config/arc/arc.c (arc_get_tp): Remove function. > (arc_emit_call_tls_get_addr): Likewise. > (arc_call_tls_get_addr): New

Re: [PATCH 1/6] [ARC] Remove non standard funcions calls.

2018-10-11 Thread Andrew Burgess
* Claudiu Zissulescu [2018-10-10 11:00:11 +0300]: > Replace all custom "library" calls with compiler known patterns. > > gcc/ > -xx-xx Claudiu Zissulescu > > * config/arc/arc.md (mulsi3): Remove call to mulsi_600_lib. > (mulsi3_600_lib): Remove pattern. > (umulsi3_highp

Re: [PATCH 1/5] [ARC] Update sleep builtin.

2018-04-18 Thread Andrew Burgess
* Claudiu Zissulescu [2018-04-06 11:00:10 +0200]: > From: claziss > > gcc/ > 2017-05-09 Claudiu Zissulescu > > * config/arc/arc-protos.h (check_if_valid_sleep_operand): Remove. > * config/arc/arc.c (arc_expand_builtin): Sleep accepts registers > and short u6 immediate. >

Re: [PATCH 2/5] [ARC] Fix FLS, SETI patterns.

2018-04-18 Thread Andrew Burgess
* Claudiu Zissulescu [2018-04-06 11:00:11 +0200]: > From: claziss > > Claudiu Zissulescu > > * config/arc/arc.md ("vunspec"): Delete it, unify all the unspec > enums into a single definition. > (fls): Fix predicates and printing. > (seti): Likewise. Looks good, Tha

Re: [PATCH 3/5] [ARC] Update movhi and movdi patterns.

2018-04-27 Thread Andrew Burgess
* Claudiu Zissulescu [2018-04-06 11:00:12 +0200]: > From: Claudiu Zissulescu > > Allow signed 6-bit short immediates into st[d] instructions. > > 2017-10-19 Claudiu Zissulescu > > * config/arc/arc.c (arc_split_move): Allow signed 6-bit constants > as source of std instructions

Re: [PATCH 4/5] [ARC] Cleanup sdata handling.

2018-04-27 Thread Andrew Burgess
* Claudiu Zissulescu [2018-04-06 11:00:13 +0200]: > From: Claudiu Zissulescu > > Clean up how we handle small data load/store operations. This patch clears > -flto-fat-lto-object LTO related errors. > > gcc/ > 2018-01-18 Claudiu Zissulescu > > * config/arc/arc-protos.h (prepare_ext

Re: [PATCH 5/5] [ARC] Clear the instruction cache using syscalls.

2018-04-27 Thread Andrew Burgess
* Claudiu Zissulescu [2018-04-06 11:00:14 +0200]: > Clear the instruction cache from `beg' to `end'. This makes an inline > system call to SYS_cacheflush. > > gcc/ > 2017-03-28 Claudiu Zissulescu > > * config/arc/linux.h (CLEAR_INSN_CACHE): Define. Looks good. Thanks, Andrew > --

Re: [PATCH] ARC: Add multilib support for linux targets

2018-05-16 Thread Andrew Burgess
* Alexey Brodkin [2018-05-16 22:42:36 +0300]: > We used to build baremetal (AKA Elf32) multilibbed toolchains for years > now but never made that for Linux targets since there were problems with > uClibc n multilib setup. Now with help of Crosstool-NG it is finally > possible to create uClibc-bas

Re: [PATCH] Delete temporary string within demangler even in failure cases.

2014-06-10 Thread Andrew Burgess
On 27/05/2014 2:47 PM, Ian Lance Taylor wrote: > On Tue, May 27, 2014 at 3:57 AM, Andrew Burgess wrote: >> >> libiberty/ChangeLog >> >> * cplus-dem.c (do_type): Call string_delete even if the call to >> demangle_template fails. > > This is O

PING: Re: [PATCH] demangler, only access valid fields for DEMANGLE_COMPONENT_FIXED_TYPE.

2014-06-10 Thread Andrew Burgess
Ping! Thanks for your time, Andrew On 29/05/2014 1:02 AM, Andrew Burgess wrote: > On 28/05/2014 11:56 PM, Pedro Alves wrote: >> On 05/28/2014 09:38 PM, Andrew Burgess wrote: >>> >>> diff --git a/libiberty/testsuite/demangle-expected >>> b/libiberty/testsuite

[PATCH] DW_AT_APPLE_* DWARF extensions.

2014-11-13 Thread Andrew Burgess
Some time ago there was an attempt to add the DWARF DW_AT_APPLE_* extensions to the file include/dwarf2.def. The original patch email is here: https://sourceware.org/ml/binutils/2012-09/msg00282.html the "patch committed" mail is here: https://sourceware.org/ml/binutils/2012-10/msg00424.html w

Re: [PATCH] DW_AT_APPLE_* DWARF extensions.

2014-11-13 Thread Andrew Burgess
* Jakub Jelinek [2014-11-13 10:55:34 +0100]: > On Thu, Nov 13, 2014 at 10:35:28AM +0100, Andrew Burgess wrote: > > Am I correct to think that the include/* files are owned by GCC, and > > so the mistake here was not propagating the change to the GCC > > repository? >

Re: [PATCH] DW_AT_APPLE_* DWARF extensions.

2014-11-14 Thread Andrew Burgess
* Jakub Jelinek [2014-11-13 14:13:42 +0100]: > On Thu, Nov 13, 2014 at 01:21:21PM +0100, Andrew Burgess wrote: > > I had a look around and couldn't find anything helpful. The best I > > can offer would be the current path within the llvm source code where > > these a

Re: [PATCH] libiberty/argv.c: Use freeargv() instead of free() to avoid memory leak.

2015-01-28 Thread Andrew Burgess
* Chen Gang S [2015-01-28 19:34:38 +0800]: > libiberty/argv.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/libiberty/argv.c b/libiberty/argv.c > index f2727e8..9fdd55b 100644 > --- a/libiberty/argv.c > +++ b/libiberty/argv.c > @@ -454,7 +454,7 @@ expandargv (int *arg

  1   2   3   >