Re: ZynqMP APU RAM Start

2024-05-14 Thread Sebastian Huber
On 14.05.24 08:03, Sebastian Huber wrote: Hello, the ZynqMP APU RAM start addresses are far away from 0x0: Sorry, I selected the wrong mailing list. -- embedded brains GmbH & Co. KG Herr Sebastian HUBER Dornierstr. 4 82178 Puchheim Germany email: sebastian.hu...@embedded-brains.de phone:

ZynqMP APU RAM Start

2024-05-14 Thread Sebastian Huber
it be possible to add a NULL pointer protection page? -- embedded brains GmbH & Co. KG Herr Sebastian HUBER Dornierstr. 4 82178 Puchheim Germany email: sebastian.hu...@embedded-brains.de phone: +49-89-18 94 741 - 16 fax: +49-89-18 94 741 - 08 Registergericht: Amtsgericht München Registernummer:

Re: [V2][PATCH] gcc-14/changes.html: Deprecate a GCC C extension on flexible array members.

2024-05-07 Thread Sebastian Huber
to the end, how would I add a configuration defined number of elements at the end? -- embedded brains GmbH & Co. KG Herr Sebastian HUBER Dornierstr. 4 82178 Puchheim Germany email: sebastian.hu...@embedded-brains.de phone: +49-89-18 94 741 - 16 fax: +49-89-18 94 741 - 08 Registerger

Re: [V2][PATCH] gcc-14/changes.html: Deprecate a GCC C extension on flexible array members.

2024-05-07 Thread Sebastian Huber
control \ name##_Objects[ _Objects_Maximum_per_allocation( max ) ]; \ static RTEMS_SECTION( ".noinit.rtems.content.objects." #name ) \ I would like no know which consequences the deprecation this GCC extension has. -- embedded brains GmbH & Co. KG Herr Sebastian HUBER Dornierstr. 4 82178 Puchheim Germany ema

Re: [V2][PATCH] gcc-14/changes.html: Deprecate a GCC C extension on flexible array members.

2024-05-06 Thread Sebastian Huber
On 06.05.24 09:08, Richard Biener wrote: On Sat, 4 May 2024, Sebastian Huber wrote: On 07.08.23 16:22, Qing Zhao via Gcc-patches wrote: Hi, This is the 2nd version of the patch. Comparing to the 1st version, the only change is to address Richard's comment on refering a warning option

Re: [V2][PATCH] gcc-14/changes.html: Deprecate a GCC C extension on flexible array members.

2024-05-04 Thread Sebastian Huber
ever the support for static initialization is quite important from my point of view especially for applications for embedded systems. Here, dynamic allocations may not be allowed or feasible. -- embedded brains GmbH & Co. KG Herr Sebastian HUBER Dornierstr. 4 82178 Puchheim Germany email: seba

Re: [PATCH v2] gcc-14: Mention that some warnings are now errors

2024-05-03 Thread Sebastian Huber
of the C section. How about this? OK for wwwdocs? This is fine for me as well, thanks. -- embedded brains GmbH & Co. KG Herr Sebastian HUBER Dornierstr. 4 82178 Puchheim Germany email: sebastian.hu...@embedded-brains.de phone: +49-89-18 94 741 - 16 fax: +49-89-18 94 741 - 08 Registerger

Re: [PATCH v2] gcc-14: Mention that some warnings are now errors

2024-05-02 Thread Sebastian Huber
On 24.04.24 14:28, Sebastian Huber wrote: - Am 15. Apr 2024 um 8:05 schrieb Sebastian Huber sebastian.hu...@embedded-brains.de: --- v2: Remove listing of options. htdocs/gcc-14/changes.html | 2 ++ 1 file changed, 2 insertions(+) diff --git a/htdocs/gcc-14/changes.html b/htdocs/gcc-14

Re: [PATCH v2] gcc-14: Mention that some warnings are now errors

2024-04-24 Thread Sebastian Huber
- Am 15. Apr 2024 um 8:05 schrieb Sebastian Huber sebastian.hu...@embedded-brains.de: > --- > v2: Remove listing of options. > > htdocs/gcc-14/changes.html | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/htdocs/gcc-14/changes.html b/htdocs/gcc-14/changes.

[gcc r13-8615] RTEMS: Fix powerpc configuration

2024-04-17 Thread Sebastian Huber via Gcc-cvs
https://gcc.gnu.org/g:3bfe591d62551605a9d2c6422257dc67b0fa9a2f commit r13-8615-g3bfe591d62551605a9d2c6422257dc67b0fa9a2f Author: Sebastian Huber Date: Mon Feb 12 14:53:21 2024 +0100 RTEMS: Fix powerpc configuration gcc/ChangeLog: * config/rs6000/rtems.h

[gcc r13-8614] RTEMS: Add multilib configuration for aarch64

2024-04-17 Thread Sebastian Huber via Gcc-cvs
https://gcc.gnu.org/g:b68995dcafa3412bcf0bfd19947758d807527dd2 commit r13-8614-gb68995dcafa3412bcf0bfd19947758d807527dd2 Author: Sebastian Huber Date: Mon Mar 25 08:00:02 2024 +0100 RTEMS: Add multilib configuration for aarch64 Add a multilib with workarounds for Cortex-A53

[PATCH v2] gcc-14: Mention that some warnings are now errors

2024-04-15 Thread Sebastian Huber
--- v2: Remove listing of options. htdocs/gcc-14/changes.html | 2 ++ 1 file changed, 2 insertions(+) diff --git a/htdocs/gcc-14/changes.html b/htdocs/gcc-14/changes.html index 8ac08e9a..665d050a 100644 --- a/htdocs/gcc-14/changes.html +++ b/htdocs/gcc-14/changes.html @@ -231,6 +231,8 @@ a

[wwwdocs] gcc-14: Mention that some warnings are now errors

2024-04-13 Thread Sebastian Huber
--- htdocs/gcc-14/changes.html | 11 +++ 1 file changed, 11 insertions(+) diff --git a/htdocs/gcc-14/changes.html b/htdocs/gcc-14/changes.html index 8ac08e9a..a183fad8 100644 --- a/htdocs/gcc-14/changes.html +++ b/htdocs/gcc-14/changes.html @@ -231,6 +231,17 @@ a work-in-progress.

[gcc r14-9871] RTEMS: Fix powerpc configuration

2024-04-09 Thread Sebastian Huber via Gcc-cvs
https://gcc.gnu.org/g:ce3c743d8cfbadc0aaa80267bc002fd05267ffac commit r14-9871-gce3c743d8cfbadc0aaa80267bc002fd05267ffac Author: Sebastian Huber Date: Mon Feb 12 14:53:21 2024 +0100 RTEMS: Fix powerpc configuration gcc/ChangeLog: * config/rs6000/rtems.h

Re: [PATCH v3 06/11] c: Turn -Wimplicit-function-declaration into a permerror

2024-04-09 Thread Sebastian Huber
On 09.04.24 14:10, Sam James wrote: Sebastian Huber writes: On 20.11.23 10:56, Florian Weimer wrote: In the future, it may make sense to avoid cascading errors from the implicit declaration, especially its assumed int return type. This change here only changes the kind of the diagnostic

Re: [PATCH v3 06/11] c: Turn -Wimplicit-function-declaration into a permerror

2024-04-09 Thread Sebastian Huber
to the GCC 14 release notes. -- embedded brains GmbH & Co. KG Herr Sebastian HUBER Dornierstr. 4 82178 Puchheim Germany email: sebastian.hu...@embedded-brains.de phone: +49-89-18 94 741 - 16 fax: +49-89-18 94 741 - 08 Registergericht: Amtsgericht München Registernummer: HRB 15

[gcc r14-9854] RTEMS: Add multilib configuration for aarch64

2024-04-09 Thread Sebastian Huber via Gcc-cvs
https://gcc.gnu.org/g:ddee4376d15ddde9280c9a6725ddd76bf33f2871 commit r14-9854-gddee4376d15ddde9280c9a6725ddd76bf33f2871 Author: Sebastian Huber Date: Mon Mar 25 08:00:02 2024 +0100 RTEMS: Add multilib configuration for aarch64 Add a multilib with workarounds for Cortex-A53

[PATCH] RTEMS: Add multilib configuration for aarch64

2024-03-27 Thread Sebastian Huber
Add a multilib with workarounds for Cortex-A53 errata. gcc/ChangeLog: * config.gcc (aarch64-*-rtems*): Add target makefile fragment t-aarch64-rtems. * config/aarch64/t-aarch64-rtems: New file. --- gcc/config.gcc | 1 +

[PATCH v2] RTEMS: Add multilib configuration for aarch64

2024-03-26 Thread Sebastian Huber
Add a multilib with workarounds for Cortex-A53 errata. gcc/ChangeLog: * config.gcc (aarch64-*-rtems*): Add target makefile fragment t-aarch64-rtems. * config/aarch64/t-aarch64-rtems: New file. --- gcc/config.gcc | 1 +

[GCC] RTEMS: Add multilib configuration for aarch64

2024-03-25 Thread Sebastian Huber
gcc/ChangeLog: * config.gcc (aarch64-*-rtems*): Add target makefile fragment t-aarch64-rtems. * config/aarch64/t-aarch64-rtems: New file. --- gcc/config.gcc | 1 + gcc/config/aarch64/t-aarch64-rtems | 41 ++ 2 files

Re: [PATCH v2] rs6000: Rework option -mpowerpc64 handling [PR106680]

2024-02-05 Thread Sebastian Huber
4-bit). It seems you removed the -m32 handling, so I am not sure how to approach this issue. I added a test case to the PR: https://gcc.gnu.org/bugzilla/show_bug.cgi?id=106680 -- embedded brains GmbH & Co. KG Herr Sebastian HUBER Dornierstr. 4 82178 Puchheim Germany email: sebastian.hu...@

Re: [PATCH] libgomp: Fix build for -fshort-enums

2024-01-19 Thread Sebastian Huber
On 11.09.23 14:57, Sebastian Huber wrote: On 04.07.23 08:20, Sebastian Huber wrote: On 22.05.23 14:51, Sebastian Huber wrote: Make sure that the API enums have at least the size of int. Otherwise the following build error may occur: In file included from gcc/libgomp/env.c:34: ./libgomp_f.h

[PATCH] gcov: Fix use of __LIBGCC_HAVE_LIBATOMIC

2023-12-01 Thread Sebastian Huber
libgcc/ChangeLog: PR target/112777 * libgcov.h (GCOV_SUPPORTS_ATOMIC): Honor that __LIBGCC_HAVE_LIBATOMIC is always defined as either 0 or 1. --- libgcc/libgcov.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libgcc/libgcov.h b/libgcc/libgcov.h

[PATCH] gcov: Fix __LIBGCC_HAVE_LIBATOMIC definition

2023-11-30 Thread Sebastian Huber
In libgcov we use defined (__LIBGCC_HAVE_LIBATOMIC), so we must define it only if needed (vs. #if __LIBGCC_HAVE_LIBATOMIC). gcc/c-family/ChangeLog: PR target/112777 * c-cppbuiltin.cc (c_cpp_builtins): Define __LIBGCC_HAVE_LIBATOMIC only if targetm.have_libatomic is

Re: [r14-5666 Regression] FAIL: gcc.dg/tree-prof/time-profiler-3.c scan-ipa-dump-times profile "Read tp_first_run: 2" 1 on Linux/x86_64

2023-11-27 Thread Sebastian Huber
On 26.11.23 12:18, haochen.jiang wrote: On Linux/x86_64, 41aacdea55c5d795a7aa195357d966645845d00e is the first bad commit commit 41aacdea55c5d795a7aa195357d966645845d00e Author: Sebastian Huber Date: Mon Nov 20 15:26:38 2023 +0100 gcov: Fix integer types in gen_counter_update() caused

Re: [r14-5579 Regression] FAIL: gcc.dg/tree-ssa/ssa-lim-11.c scan-tree-dump-times lim2 "Executing store motion of __gcov0.access_buf\\[[12]\\] from loop 1" 2 on Linux/x86_64

2023-11-26 Thread Sebastian Huber
On 26.11.23 12:18, haochen.jiang wrote: On Linux/x86_64, 20a3c74c347429c109bc7002285b735be83f6a0b is the first bad commit commit 20a3c74c347429c109bc7002285b735be83f6a0b Author: Sebastian Huber Date: Tue Nov 14 21:36:51 2023 +0100 gcov: Improve -fprofile-update=atomic caused FAIL

Re: [r14-5578 Regression] FAIL: gfortran.dg/gomp/pr27573.f90 -O (test for excess errors) on Linux/x86_64

2023-11-26 Thread Sebastian Huber
On 26.11.23 12:18, haochen.jiang wrote: On Linux/x86_64, a350a74d6113e3a84943266eb691275951c109d9 is the first bad commit commit a350a74d6113e3a84943266eb691275951c109d9 Author: Sebastian Huber Date: Sat Oct 21 15:52:15 2023 +0200 gcov: Add gen_counter_update() caused FAIL: gcc.dg

Re: [PATCH] Update GMP/MPFR/MPC/ISL/gettext to latest release

2023-11-26 Thread Sebastian Huber
On 26.11.23 11:15, Iain Sandoe wrote: On 26 Nov 2023, at 10:05, Sebastian Huber wrote: On 26.11.23 01:35, Iain Sandoe wrote: On 25 Nov 2023, at 21:44, Sebastian Huber wrote: On 25.11.23 14:59, Richard Biener wrote: On Sat, Nov 25, 2023 at 12:26 PM Sebastian Huber wrote: contrib

Re: [PATCH] Update GMP/MPFR/MPC/ISL/gettext to latest release

2023-11-26 Thread Sebastian Huber
On 26.11.23 01:35, Iain Sandoe wrote: On 25 Nov 2023, at 21:44, Sebastian Huber wrote: On 25.11.23 14:59, Richard Biener wrote: On Sat, Nov 25, 2023 at 12:26 PM Sebastian Huber wrote: contrib/ChangeLog Did you verify an in-tree build with these works and the testsuite is clean? I

Re: [PATCH] Update GMP/MPFR/MPC/ISL/gettext to latest release

2023-11-25 Thread Sebastian Huber
On 25.11.23 14:59, Richard Biener wrote: On Sat, Nov 25, 2023 at 12:26 PM Sebastian Huber wrote: contrib/ChangeLog Did you verify an in-tree build with these works and the testsuite is clean? I was able to build a native GCC: /tmp/sh/i-native/bin/gcc --version --verbose Using built

[PATCH] Update GMP/MPFR/MPC/ISL/gettext to latest release

2023-11-25 Thread Sebastian Huber
contrib/ChangeLog * download_prerequisites: Update to gmp-6.3.0, mpfr-4.2.1, mpc-1.3.1, isl-0.26, and gettext-0.22.4. * prerequisites.md5: Update hash. * prerequisites.sha512: Likewise. --- contrib/download_prerequisites | 10 +- contrib/prerequisites.md5

Re: [PATCH v2] gcov: Fix integer types in gen_counter_update()

2023-11-24 Thread Sebastian Huber
On 23.11.23 18:29, Sebastian Huber wrote: On 23.11.23 09:20, Sebastian Huber wrote: On 23.11.23 09:11, Jiang, Haochen wrote: -Original Message- From: Sebastian Huber Sent: Wednesday, November 22, 2023 10:24 PM To: Christophe Lyon Cc: Jakub Jelinek;gcc-patches@gcc.gnu.org Subject: Re

Re: [PATCH v2] gcov: Fix integer types in gen_counter_update()

2023-11-23 Thread Sebastian Huber
On 23.11.23 09:20, Sebastian Huber wrote: On 23.11.23 09:11, Jiang, Haochen wrote: -Original Message- From: Sebastian Huber Sent: Wednesday, November 22, 2023 10:24 PM To: Christophe Lyon Cc: Jakub Jelinek;gcc-patches@gcc.gnu.org Subject: Re: [PATCH v2] gcov: Fix integer types

Re: [PATCH] gcov: No atomic ops for -fprofile-update=single

2023-11-23 Thread Sebastian Huber
On 23.11.23 15:19, Richard Biener wrote: On Thu, Nov 23, 2023 at 2:47 PM Sebastian Huber wrote: gcc/ChangeLog: PR tree-optimization/112678 * tree-profile.cc (tree_profiling): Do not use atomic operations for -fprofile-update=single. --- gcc/tree-profile.cc | 8

[PATCH] gcov: No atomic ops for -fprofile-update=single

2023-11-23 Thread Sebastian Huber
gcc/ChangeLog: PR tree-optimization/112678 * tree-profile.cc (tree_profiling): Do not use atomic operations for -fprofile-update=single. --- gcc/tree-profile.cc | 8 +--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/gcc/tree-profile.cc

Re: [PATCH v2] gcov: Fix integer types in gen_counter_update()

2023-11-23 Thread Sebastian Huber
On 23.11.23 09:11, Jiang, Haochen wrote: -Original Message- From: Sebastian Huber Sent: Wednesday, November 22, 2023 10:24 PM To: Christophe Lyon Cc: Jakub Jelinek;gcc-patches@gcc.gnu.org Subject: Re: [PATCH v2] gcov: Fix integer types in gen_counter_update() On 22.11.23 15:22

Re: [PATCH v2] gcov: Fix integer types in gen_counter_update()

2023-11-22 Thread Sebastian Huber
On 22.11.23 15:22, Christophe Lyon wrote: On Tue, 21 Nov 2023 at 12:22, Sebastian Huber wrote: On 21.11.23 11:46, Jakub Jelinek wrote: On Tue, Nov 21, 2023 at 11:42:06AM +0100, Sebastian Huber wrote: On 21.11.23 11:34, Jakub Jelinek wrote: --- a/gcc/tree-profile.cc +++ b/gcc/tree

Re: [PATCH v2] gcov: Fix integer types in gen_counter_update()

2023-11-21 Thread Sebastian Huber
On 21.11.23 11:46, Jakub Jelinek wrote: On Tue, Nov 21, 2023 at 11:42:06AM +0100, Sebastian Huber wrote: On 21.11.23 11:34, Jakub Jelinek wrote: --- a/gcc/tree-profile.cc +++ b/gcc/tree-profile.cc @@ -281,10 +281,13 @@ gen_assign_counter_update (gimple_stmt_iterator *gsi, gcall *call, tree

Re: [PATCH v2] gcov: Fix integer types in gen_counter_update()

2023-11-21 Thread Sebastian Huber
make_ssa_name() would be the first use in this file. -- embedded brains GmbH Herr Sebastian HUBER Dornierstr. 4 82178 Puchheim Germany email: sebastian.hu...@embedded-brains.de phone: +49-89-18 94 741 - 16 fax: +49-89-18 94 741 - 08 Registergericht: Amtsgericht München Registernummer: HRB 157899

[PATCH v2] gcov: Fix integer types in gen_counter_update()

2023-11-21 Thread Sebastian Huber
This change fixes issues like this: gcc.dg/gomp/pr27573.c: In function ‘main._omp_fn.0’: gcc.dg/gomp/pr27573.c:19:1: error: non-trivial conversion in ‘ssa_name’ 19 | } | ^ long int long unsigned int # .MEM_19 = VDEF <.MEM_18> __gcov7.main._omp_fn.0[0] =

Re: [PATCH 2/2] gcov: Fix integer types in gen_counter_update()

2023-11-21 Thread Sebastian Huber
On 20.11.23 15:56, Jakub Jelinek wrote: On Mon, Nov 20, 2023 at 03:33:31PM +0100, Sebastian Huber wrote: This change fixes issues like this: gcc.dg/gomp/pr27573.c: In function ‘main._omp_fn.0’: gcc.dg/gomp/pr27573.c:19:1: error: non-trivial conversion in ‘ssa_name’ 19

[PATCH 1/2] gcov: Use unshare_expr() in gen_counter_update()

2023-11-20 Thread Sebastian Huber
This fixes issues like this: gcc/testsuite/gcc.dg/no_profile_instrument_function-attr-1.c: In function 'main': gcc/testsuite/gcc.dg/no_profile_instrument_function-attr-1.c:19:1: error: incorrect sharing of tree nodes __gcov0.main[0] # .MEM_12 = VDEF <.MEM_9> __gcov0.main[0] =

[PATCH 2/2] gcov: Fix integer types in gen_counter_update()

2023-11-20 Thread Sebastian Huber
This change fixes issues like this: gcc.dg/gomp/pr27573.c: In function ‘main._omp_fn.0’: gcc.dg/gomp/pr27573.c:19:1: error: non-trivial conversion in ‘ssa_name’ 19 | } | ^ long int long unsigned int # .MEM_19 = VDEF <.MEM_18> __gcov7.main._omp_fn.0[0] =

Re: [PATCH 3/4] gcov: Add gen_counter_update()

2023-11-19 Thread Sebastian Huber
Hello Dimitar, On 19.11.23 10:00, Dimitar Dimitrov wrote: On Tue, Nov 14, 2023 at 11:08:24PM +0100, Sebastian Huber wrote: Move the counter update to the new gen_counter_update() helper function. Use it in gimple_gen_edge_profiler() and gimple_gen_time_profiler(). The resulting gimple

Re: [PATCH 4/4] gcov: Improve -fprofile-update=atomic

2023-11-14 Thread Sebastian Huber
Hello,   sorry, in the patch I should use targetm.have_atomic instead of TARGET_HAVE_LIBATOMIC.   --- embedded brains GmbH & Co. KG Herr Sebastian HUBER Dornierstr. 4 82178 Puchheim Germany email: sebastian.hu...@embedded-brains.de phone: +49-89-18 94 741 - 16 fax: +49-89-18 94 741

[PATCH 4/4] gcov: Improve -fprofile-update=atomic

2023-11-14 Thread Sebastian Huber
The code coverage support uses counters to determine which edges in the control flow graph were executed. If a counter overflows, then the code coverage information is invalid. Therefore the counter type should be a 64-bit integer. In multi-threaded applications, it is important that the counter

[PATCH 1/4] gcov: Remove TARGET_GCOV_TYPE_SIZE target hook

2023-11-14 Thread Sebastian Huber
This reverts commit 8cdcea51c0fd753e6a652c9b236e91b3a6e0911c. gcc/c-family/ChangeLog: * c-cppbuiltin.cc (c_cpp_builtins): Do not define __LIBGCC_GCOV_TYPE_SIZE. gcc/ChangeLog: * config/sparc/rtemself.h (SPARC_GCOV_TYPE_SIZE): Remove. * config/sparc/sparc.cc

[PATCH 3/4] gcov: Add gen_counter_update()

2023-11-14 Thread Sebastian Huber
Move the counter update to the new gen_counter_update() helper function. Use it in gimple_gen_edge_profiler() and gimple_gen_time_profiler(). The resulting gimple instructions should be identical with the exception of the removed unshare_expr() call. The unshare_expr() call was used in

[PATCH 0/4] gcov: Improve -fprofile-update=atomic

2023-11-14 Thread Sebastian Huber
Sebastian Huber (4): gcov: Remove TARGET_GCOV_TYPE_SIZE target hook Add TARGET_HAVE_LIBATOMIC gcov: Add gen_counter_update() gcov: Improve -fprofile-update=atomic gcc/c-family/c-cppbuiltin.cc | 4 +- gcc/config/rtems.h | 2 + gcc/config/sparc/rtemself.h | 2 - gcc/config

[PATCH 2/4] Add TARGET_HAVE_LIBATOMIC

2023-11-14 Thread Sebastian Huber
Add target data to indicate if libatomic is available. gcc/ChangeLog: * config/rtems.h (TARGET_HAVE_LIBATOMIC): Define. * doc/tm.texi: Regenerate. * doc/tm.texi.in (TARGET_HAVE_LIBATOMIC): Add. * target.def (have_libatomic): New. --- gcc/config/rtems.h | 2 ++

License of libgcc/config/sparc/lb1spc.S?

2023-09-22 Thread Sebastian Huber
the desired calling convention, and also to optimize them for our purposes. */ -- embedded brains GmbH Herr Sebastian HUBER Dornierstr. 4 82178 Puchheim Germany email: sebastian.hu...@embedded-brains.de phone: +49-89-18 94 741 - 16 fax: +49-89-18 94 741 - 08 Registergericht: Amtsgericht München

Re: [PATCH] libatomic: Provide gthr.h default implementation

2023-09-11 Thread Sebastian Huber
On 31.05.23 09:27, Richard Biener wrote: On Wed, May 31, 2023 at 7:31 AM Sebastian Huber wrote: On 30.05.23 13:17, Richard Biener wrote: The alternative would be to provide the required subset of atomic library functions from libgcov.a and emit calls to that directly? The locked data isn't

Re: [PATCH] libgomp: Fix build for -fshort-enums

2023-09-11 Thread Sebastian Huber
On 04.07.23 08:20, Sebastian Huber wrote: On 22.05.23 14:51, Sebastian Huber wrote: Make sure that the API enums have at least the size of int.  Otherwise the following build error may occur: In file included from gcc/libgomp/env.c:34: ./libgomp_f.h: In function 'omp_check_defines

Re: [PATCH] libgomp: Fix build for -fshort-enums

2023-07-04 Thread Sebastian Huber
On 22.05.23 14:51, Sebastian Huber wrote: Make sure that the API enums have at least the size of int. Otherwise the following build error may occur: In file included from gcc/libgomp/env.c:34: ./libgomp_f.h: In function 'omp_check_defines': ./libgomp_f.h:77:8: error: size of array 'test

Re: [PATCH] libatomic: Provide gthr.h default implementation

2023-05-30 Thread Sebastian Huber
are not available in hardware, then I should emit calls to libgcov.a which would use gthr.h to implement them? I guess that I can to this, but it needs a bit of time. Should I add the libgcov functions to builtin_decl_explicit()? -- embedded brains GmbH Herr Sebastian HUBER Dornierstr. 4 82178

Re: [PATCH] libatomic: Provide gthr.h default implementation

2023-05-30 Thread Sebastian Huber
On 30.05.23 11:53, Richard Biener wrote: On Tue, May 23, 2023 at 11:28 AM Sebastian Huber wrote: On 10.01.23 16:38, Sebastian Huber wrote: On 19/12/2022 17:02, Sebastian Huber wrote: Build libatomic for all targets. Use gthr.h to provide a default implementation. If the thread model

Re: [PATCH] libatomic: Provide gthr.h default implementation

2023-05-23 Thread Sebastian Huber
On 10.01.23 16:38, Sebastian Huber wrote: On 19/12/2022 17:02, Sebastian Huber wrote: Build libatomic for all targets.  Use gthr.h to provide a default implementation.  If the thread model is "single", then this implementation will not work if for example atomic operations are used

[PATCH] libgomp: Fix build for -fshort-enums

2023-05-22 Thread Sebastian Huber
Make sure that the API enums have at least the size of int. Otherwise the following build error may occur: In file included from gcc/libgomp/env.c:34: ./libgomp_f.h: In function 'omp_check_defines': ./libgomp_f.h:77:8: error: size of array 'test' is negative 77 | char test[(28 != sizeof

Re: [wwwdocs] gcc-13: Mention new gcov feature

2023-05-04 Thread Sebastian Huber
On 26.04.23 08:10, Sebastian Huber wrote: --- htdocs/gcc-13/changes.html | 5 + 1 file changed, 5 insertions(+) diff --git a/htdocs/gcc-13/changes.html b/htdocs/gcc-13/changes.html index 4515a6af..bae65219 100644 --- a/htdocs/gcc-13/changes.html +++ b/htdocs/gcc-13/changes.html @@ -154,6

[wwwdocs] gcc-13: Mention new gcov feature

2023-04-26 Thread Sebastian Huber
--- htdocs/gcc-13/changes.html | 5 + 1 file changed, 5 insertions(+) diff --git a/htdocs/gcc-13/changes.html b/htdocs/gcc-13/changes.html index 4515a6af..bae65219 100644 --- a/htdocs/gcc-13/changes.html +++ b/htdocs/gcc-13/changes.html @@ -154,6 +154,11 @@ a work-in-progress.

Re: Third GC 13.1 Release Candidate available from gcc.gnu.org

2023-04-24 Thread Sebastian Huber
-- embedded brains GmbH Herr Sebastian HUBER Dornierstr. 4 82178 Puchheim Germany email: sebastian.hu...@embedded-brains.de phone: +49-89-18 94 741 - 16 fax: +49-89-18 94 741 - 08 Registergericht: Amtsgericht München Registernummer: HRB 157899 Vertretungsberechtigte Geschäftsführer: Peter

Re: [gcc] RTEMS: Tune multilib selection

2023-02-23 Thread Sebastian Huber
, but if there's something in RTEMS land that benefits from this being early then I think it's fine. RTEMS has its own release cycle, so I would back port this change to all GCC branches which may be used with RTEMS 6 and this is GCC 10 or later. -- embedded brains GmbH Herr Sebastian HUBER

[gcc] RTEMS: Tune multilib selection

2023-02-23 Thread Sebastian Huber
gcc/ChangeLog: * config/riscv/t-rtems: Keep only -mcmodel=medany 64-bit multilibs. Add non-compact 32-bit multilibs. --- gcc/config/riscv/t-rtems | 17 + 1 file changed, 9 insertions(+), 8 deletions(-) diff --git a/gcc/config/riscv/t-rtems

Re: [PATCH] libatomic: Provide gthr.h default implementation

2023-01-10 Thread Sebastian Huber
On 19/12/2022 17:02, Sebastian Huber wrote: Build libatomic for all targets. Use gthr.h to provide a default implementation. If the thread model is "single", then this implementation will not work if for example atomic operations are used for thread/interrupt synch

[PATCH] libatomic: Provide gthr.h default implementation

2022-12-19 Thread Sebastian Huber
Build libatomic for all targets. Use gthr.h to provide a default implementation. If the thread model is "single", then this implementation will not work if for example atomic operations are used for thread/interrupt synchronization. libatomic/ChangeLog: * Makefile.am (BUILT_SOURCES):

Re: Why is there no libatomic default implementation using gthr.h?

2022-12-19 Thread Sebastian Huber
is actually needed) and compile that locally in the libatomic build dir. That shouldn't need changes to libgcc or libstdc++, should it? Yes, this would also work. I can probably duplicate the gthr configure/Makefile stuff from libstdc++ for this. -- embedded brains GmbH Herr Sebastian HUBER Dornierstr

Why is there no libatomic default implementation using gthr.h?

2022-12-19 Thread Sebastian Huber
fault implementation of libatomic. -- embedded brains GmbH Herr Sebastian HUBER Dornierstr. 4 82178 Puchheim Germany email: sebastian.hu...@embedded-brains.de phone: +49-89-18 94 741 - 16 fax: +49-89-18 94 741 - 08 Registergericht: Amtsgericht München Registernummer: HRB 157899 Vertretungsberecht

Re: [PATCH] gcov: Fix -fprofile-update=atomic

2022-12-16 Thread Sebastian Huber
On 16.12.22 13:09, Richard Biener wrote: On Fri, Dec 16, 2022 at 11:39 AM Sebastian Huber wrote: On 16.12.22 10:47, Richard Biener wrote: No, if you select -fprofile-update=atomic, then the updates shall be atomic from my point of view. If a fallback is acceptable, then you can use

Re: [PATCH] gcov: Fix -fprofile-update=atomic

2022-12-16 Thread Sebastian Huber
: __gcov_inc_counter(counter) -> updated value This function could use a __gthread_mutex_t mutex for updates. This ends up probably with quite a bad performance. -- embedded brains GmbH Herr Sebastian HUBER Dornierstr. 4 82178 Puchheim Germany email: sebastian.hu...@embedded-brains.de phone:

[PATCH v2] gcov: Fix -fprofile-update=atomic

2022-12-16 Thread Sebastian Huber
The code coverage support uses counters to determine which edges in the control flow graph were executed. If a counter overflows, then the code coverage information is invalid. Therefore the counter type should be a 64-bit integer. In multithreaded applications, it is important that the counter

Re: [PATCH] gcov: Fix -fprofile-update=atomic

2022-12-15 Thread Sebastian Huber
On 13/12/2022 15:30, Richard Biener wrote: On Fri, Dec 9, 2022 at 2:56 PM Sebastian Huber wrote: The code coverage support uses counters to determine which edges in the control flow graph were executed. If a counter overflows, then the code coverage information is invalid. Therefore

[PATCH] gcov: Fix -fprofile-update=atomic

2022-12-09 Thread Sebastian Huber
The code coverage support uses counters to determine which edges in the control flow graph were executed. If a counter overflows, then the code coverage information is invalid. Therefore the counter type should be a 64-bit integer. In multithreaded applications, it is important that the counter

Re: [PATCH v2 1/2] Allow subtarget customization of CC1_SPEC

2022-12-08 Thread Sebastian Huber
=3e4b8dc477c12f303171ec7f0394c97494095545 -- embedded brains GmbH Herr Sebastian HUBER Dornierstr. 4 82178 Puchheim Germany email: sebastian.hu...@embedded-brains.de phone: +49-89-18 94 741 - 16 fax: +49-89-18 94 741 - 08 Registergericht: Amtsgericht München Registernummer: HRB 157899 Vertretungsberechtigte

Re: -fprofile-update=atomic vs. 32-bit architectures

2022-12-07 Thread Sebastian Huber
On 04.11.22 09:27, Sebastian Huber wrote: Hello, even recent 32-bit architectures such as RISC-V do not support 64-bit atomic operations.  Using -fprofile-update=atomic for the 32-bit RISC-V RV32GC ISA yields: warning: target does not support atomic profile update, single mode

Re: -fprofile-update=atomic vs. 32-bit architectures

2022-12-07 Thread Sebastian Huber
On 07.12.22 10:09, Richard Biener wrote: On Wed, Dec 7, 2022 at 9:51 AM Sebastian Huber wrote: On 06.12.22 17:08, Richard Biener wrote: Likely. I'd use the gimple_build () API from gimple-fold.h which builds the expression(s) to a gimple_seq creating necessary temporaries on-the-fly

Re: -fprofile-update=atomic vs. 32-bit architectures

2022-12-07 Thread Sebastian Huber
gcc/passes.cc:2145 Please submit a full bug report, with preprocessed source (by using -freport-bug). Please include the complete backtrace with any bug report. See <https://gcc.gnu.org/bugs/> for instructions. -- embedded brains GmbH Herr Sebastian HUBER Dornierstr. 4 82178 Puchheim Germ

Re: [PATCH v2 1/2] Allow subtarget customization of CC1_SPEC

2022-12-06 Thread Sebastian Huber
On 07.12.22 08:10, Thomas Schwinge wrote: Hi! On 2022-12-07T07:04:10+0100, Sebastian Huber wrote: On 06.12.22 22:06, Thomas Schwinge wrote: I suppose I just fail to see some detail here, but: On 2022-11-21T08:25:25+0100, Sebastian Huber wrote: gcc/ChangeLog: * gcc.cc

Re: [PATCH v2 1/2] Allow subtarget customization of CC1_SPEC

2022-12-06 Thread Sebastian Huber
On 06.12.22 22:06, Thomas Schwinge wrote: Hi! I suppose I just fail to see some detail here, but: On 2022-11-21T08:25:25+0100, Sebastian Huber wrote: gcc/ChangeLog: * gcc.cc (SUBTARGET_CC1_SPEC): Define if not defined. (cc1_spec): Append SUBTARGET_CC1_SPEC. --- v2: Append

Re: -fprofile-update=atomic vs. 32-bit architectures

2022-12-06 Thread Sebastian Huber
On 05/12/2022 08:44, Richard Biener wrote: On Mon, Dec 5, 2022 at 8:26 AM Sebastian Huber wrote: On 08/11/2022 11:25, Richard Biener wrote: It would be great to have a code example for the construction of the "if (f()) f();". I think for the function above we ne

Re: -fprofile-update=atomic vs. 32-bit architectures

2022-12-04 Thread Sebastian Huber
a5,a4,0(a0) addia5,a5,1 seqza5,a5 addia4,a0,4 amoadd.w zero,a5,0(a4) -- embedded brains GmbH Herr Sebastian HUBER Dornierstr. 4 82178 Puchheim Germany email: sebastian.hu...@embedded-brains.de phone: +49-89-18 94 741 - 16 fax: +49-89-18 94 741 - 08 Regis

Re: [PATCH 1/2] Allow subtarget customization of CC1_SPEC

2022-11-20 Thread Sebastian Huber
On 20/11/2022 17:57, Jeff Law wrote: On 10/26/22 03:34, Sebastian Huber wrote: On 04/10/2022 11:47, Sebastian Huber wrote: On 08/09/2022 07:33, Sebastian Huber wrote: On 04/08/2022 15:02, Sebastian Huber wrote: On 22/07/2022 15:02, Sebastian Huber wrote: gcc/ChangeLog: * gcc.cc

[PATCH v2 2/2] RTEMS: Use local-exec TLS model by default

2022-11-20 Thread Sebastian Huber
gcc/ChangeLog: * config/rtems.h (SUBTARGET_CC1_SPEC): Undef and define. --- gcc/config/rtems.h | 4 1 file changed, 4 insertions(+) diff --git a/gcc/config/rtems.h b/gcc/config/rtems.h index 95bcdc41b2f..4742b1f3722 100644 --- a/gcc/config/rtems.h +++ b/gcc/config/rtems.h @@ -56,3

[PATCH v2 1/2] Allow subtarget customization of CC1_SPEC

2022-11-20 Thread Sebastian Huber
gcc/ChangeLog: * gcc.cc (SUBTARGET_CC1_SPEC): Define if not defined. (cc1_spec): Append SUBTARGET_CC1_SPEC. --- v2: Append SUBTARGET_CC1_SPEC directly to cc1_spec and not through CC1_SPEC. This avoids having to modify all the CC1_SPEC definitions in the targets. gcc/gcc.cc |

Re: [PATCH 3/3] Add '--oslib=' option when default C library is picolibc

2022-11-20 Thread Sebastian Huber
tm_file="${tm_file} picolibc.h" ;; In this header place: #undef LIB_SPEC #define LIB_SPEC "--start-group %(libgcc) -lc %{-oslib=*:-l%*} --end-group" #endif This avoids having to modify all the ELF-specific files. -- embedded brains GmbH Herr Sebastian HUBER Dornierstr.

Re: -fprofile-update=atomic vs. 32-bit architectures

2022-11-08 Thread Sebastian Huber
then redirect edges to form a half-diamond. move_sese_in_condition has most of that CFG manipulation (but it performs sth different) Thanks, I will probably able to do this with a bit trial and error. -- embedded brains GmbH Herr Sebastian HUBER Dornierstr. 4 82178 Puchheim Germany email: seba

Re: -fprofile-update=atomic vs. 32-bit architectures

2022-11-07 Thread Sebastian Huber
On 05.11.22 12:18, Richard Biener wrote: On Fri, Nov 4, 2022 at 9:28 AM Sebastian Huber wrote: Hello, even recent 32-bit architectures such as RISC-V do not support 64-bit atomic operations. Using -fprofile-update=atomic for the 32-bit RISC-V RV32GC ISA yields: warning: target does

Re: -fprofile-update=atomic vs. 32-bit architectures

2022-11-04 Thread Sebastian Huber
, but this approach would also work on big-endian architectures. You just have to use other addresses. I guess the compiler knows for which endianess it generates code. -- embedded brains GmbH Herr Sebastian HUBER Dornierstr. 4 82178 Puchheim Germany email: sebastian.hu...@embedded-brains.de phone: +49-89-18 94

-fprofile-update=atomic vs. 32-bit architectures

2022-11-04 Thread Sebastian Huber
a library call (libatomic). -- embedded brains GmbH Herr Sebastian HUBER Dornierstr. 4 82178 Puchheim Germany email: sebastian.hu...@embedded-brains.de phone: +49-89-18 94 741 - 16 fax: +49-89-18 94 741 - 08 Registergericht: Amtsgericht München Registernummer: HRB 157899 Vertretungsberechtigte

Re: [PATCH] riscv/RTEMS: Add RISCV_GCOV_TYPE_SIZE

2022-10-27 Thread Sebastian Huber
On 28/10/2022 01:05, Palmer Dabbelt wrote: On Thu, 27 Oct 2022 15:56:17 PDT (-0700), gcc-patches@gcc.gnu.org wrote: On 10/26/22 01:49, Sebastian Huber wrote: The RV32A extension does not support 64-bit atomic operations.  For RTEMS, use a 32-bit gcov type for RV32. gcc/ChangeLog

Re: [PATCH 1/2] Allow subtarget customization of CC1_SPEC

2022-10-26 Thread Sebastian Huber
On 04/10/2022 11:47, Sebastian Huber wrote: On 08/09/2022 07:33, Sebastian Huber wrote: On 04/08/2022 15:02, Sebastian Huber wrote: On 22/07/2022 15:02, Sebastian Huber wrote: gcc/ChangeLog: * gcc.cc (SUBTARGET_CC1_SPEC): Define if not defined. (CC1_SPEC): Define

[PATCH] riscv/RTEMS: Add RISCV_GCOV_TYPE_SIZE

2022-10-26 Thread Sebastian Huber
The RV32A extension does not support 64-bit atomic operations. For RTEMS, use a 32-bit gcov type for RV32. gcc/ChangeLog: * config/riscv/riscv.cc (riscv_gcov_type_size): New. (TARGET_GCOV_TYPE_SIZE): Likewise. * config/riscv/rtems.h (RISCV_GCOV_TYPE_SIZE): New. ---

Re: [PATCH 1/2] Allow subtarget customization of CC1_SPEC

2022-10-04 Thread Sebastian Huber
On 08/09/2022 07:33, Sebastian Huber wrote: On 04/08/2022 15:02, Sebastian Huber wrote: On 22/07/2022 15:02, Sebastian Huber wrote: gcc/ChangeLog: * gcc.cc (SUBTARGET_CC1_SPEC): Define if not defined. (CC1_SPEC): Define to SUBTARGET_CC1_SPEC. * config/arm/arm.h (CC1_SPEC): Remove

Re: [PATCH 1/2] Allow subtarget customization of CC1_SPEC

2022-09-07 Thread Sebastian Huber
On 04/08/2022 15:02, Sebastian Huber wrote: On 22/07/2022 15:02, Sebastian Huber wrote: gcc/ChangeLog: * gcc.cc (SUBTARGET_CC1_SPEC): Define if not defined. (CC1_SPEC): Define to SUBTARGET_CC1_SPEC. * config/arm/arm.h (CC1_SPEC): Remove. * config/arc/arc.h (CC1_SPEC): Append

[PATCH/gcc] RTEMS: Add -mvrsave multilibs

2022-09-01 Thread Sebastian Huber
gcc/ChangeLog: * config/rs6000/rtems.h (CPP_OS_DEFAULT_SPEC): Define __PPC_VRSAVE__ if -mvrsave is present. * config/rs6000/t-rtems: Add -mvrsave multilib variants for -mcpu=e6500. --- gcc/config/rs6000/rtems.h | 3 ++- gcc/config/rs6000/t-rtems | 5 + 2 files

Re: [PATCH 1/2] Allow subtarget customization of CC1_SPEC

2022-08-04 Thread Sebastian Huber
On 22/07/2022 15:02, Sebastian Huber wrote: gcc/ChangeLog: * gcc.cc (SUBTARGET_CC1_SPEC): Define if not defined. (CC1_SPEC): Define to SUBTARGET_CC1_SPEC. * config/arm/arm.h (CC1_SPEC): Remove. * config/arc/arc.h (CC1_SPEC): Append SUBTARGET_CC1_SPEC

Re: [PATCH] Add condition coverage profiling

2022-08-04 Thread Sebastian Huber
GmbH Herr Sebastian HUBER Dornierstr. 4 82178 Puchheim Germany email: sebastian.hu...@embedded-brains.de phone: +49-89-18 94 741 - 16 fax: +49-89-18 94 741 - 08 Registergericht: Amtsgericht München Registernummer: HRB 157899 Vertretungsberechtigte Geschäftsführer: Peter Rasmussen, Thomas Dörf

Re: [PATCH 1/2] Allow subtarget customization of CC1_SPEC

2022-07-25 Thread Sebastian Huber
o a subset of a target... Iain Sandoe told me that for the GCC implementation the OSs are sub-targets of the architecture: https://gcc.gnu.org/pipermail/gcc/2022-July/239158.html -- embedded brains GmbH Herr Sebastian HUBER Dornierstr. 4 82178 Puchheim Germany email: sebastian.hu...@embedded-brains.de

[PATCH 1/2] Allow subtarget customization of CC1_SPEC

2022-07-22 Thread Sebastian Huber
gcc/ChangeLog: * gcc.cc (SUBTARGET_CC1_SPEC): Define if not defined. (CC1_SPEC): Define to SUBTARGET_CC1_SPEC. * config/arm/arm.h (CC1_SPEC): Remove. * config/arc/arc.h (CC1_SPEC): Append SUBTARGET_CC1_SPEC. * config/cris/cris.h (CC1_SPEC): Likewise.

[PATCH 2/2] RTEMS: Use local-exec TLS model by default

2022-07-22 Thread Sebastian Huber
gcc/ChangeLog: * config/rtems.h (SUBTARGET_CC1_SPEC): Undef and define. --- gcc/config/rtems.h | 4 1 file changed, 4 insertions(+) diff --git a/gcc/config/rtems.h b/gcc/config/rtems.h index 95bcdc41b2f..4742b1f3722 100644 --- a/gcc/config/rtems.h +++ b/gcc/config/rtems.h @@ -56,3

Re: Use -ftls-model=local-exec for RTEMS by default?

2022-07-22 Thread Sebastian Huber
): #define CC1_SPEC "%{G*}" For each target you would have to check if you have to provide some extra times for CC1_SPEC through copy and paste. -- embedded brains GmbH Herr Sebastian HUBER Dornierstr. 4 82178 Puchheim Germany email: sebastian.hu...@embedded-brains.de phone: +49-89-1

  1   2   3   4   5   6   7   >