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
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 in
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
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
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-in
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
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
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
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
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
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 true.
tresults/2012-09/msg02507.html
--
Sebastian Huber, embedded brains GmbH
Address : Obere Lagerstr. 30, D-82178 Puchheim, Germany
Phone : +49 89 18 90 80 79-6
Fax : +49 89 18 90 80 79-9
E-Mail : sebastian.hu...@embedded-brains.de
PGP : Public key available on request.
Diese Nachricht ist
On 10/09/2012 11:08 AM, Sebastian Huber wrote:
Hello,
this is an updated patch for the GCC 4.8. It renames the target
"arm-rtemseabi" to "arm-rtems" to bring the ARM tool chain back to the standard
RTEMS target pattern "$ARCH-rtems".
GCC test suite re
Updated patch.
--
Sebastian Huber, embedded brains GmbH
Address : Obere Lagerstr. 30, D-82178 Puchheim, Germany
Phone : +49 89 18 90 80 79-6
Fax : +49 89 18 90 80 79-9
E-Mail : sebastian.hu...@embedded-brains.de
PGP : Public key available on request.
Diese Nachricht ist keine
--
Sebastian Huber, embedded brains GmbH
Address : Obere Lagerstr. 30, D-82178 Puchheim, Germany
Phone : +49 89 18 90 80 79-6
Fax : +49 89 18 90 80 79-9
E-Mail : sebastian.hu...@embedded-brains.de
PGP : Public key available on request.
Diese Nachricht ist keine geschäftliche
On 03/09/2012 10:01 AM, Sebastian Huber wrote:
Hi,
please have a look at the attached patch. Test suite results for GCC 4.7
http://gcc.gnu.org/ml/gcc-testresults/2012-03/msg00986.html
This patch is still pending for GCC 4.7 and 4.8. Can someone please review and
commit it. Without this
Hello Alan,
thanks for looking at this issue.
On 10/24/2012 04:50 PM, Alan Modra wrote:
On Tue, Oct 23, 2012 at 06:25:43PM +0200, Sebastian Huber wrote:
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=55033
This patch fixes my problem, but I am absolutely not sure if this is the
right way
Hello,
here is a test case for PR55033.
--
Sebastian Huber, embedded brains GmbH
Address : Obere Lagerstr. 30, D-82178 Puchheim, Germany
Phone : +49 89 18 90 80 79-6
Fax : +49 89 18 90 80 79-9
E-Mail : sebastian.hu...@embedded-brains.de
PGP : Public key available on request.
Diese
On 10/26/2012 02:22 PM, Sebastian Huber wrote:
Hello,
here is a test case for PR55033.
Is there something wrong with this test case? It compiles well with Alan's
patch.
--
Sebastian Huber, embedded brains GmbH
Address : Obere Lagerstr. 30, D-82178 Puchheim, Germany
Phone : +49 89
Hello,
what needs to be done to get this committed?
--
Sebastian Huber, embedded brains GmbH
Address : Obere Lagerstr. 30, D-82178 Puchheim, Germany
Phone : +49 89 18 90 80 79-6
Fax : +49 89 18 90 80 79-9
E-Mail : sebastian.hu...@embedded-brains.de
PGP : Public key available on
want to
look at the remaining 104 test cases which are marked as UNSUPPORTED for RTEMS
some time in the future. Please have a look at the attached patches.
On 10/31/2012 01:12 AM, Alan Modra wrote:
On Tue, Oct 30, 2012 at 02:45:40PM +0100, Sebastian Huber wrote:
On 10/26/2012 02:22 PM
Hello Jakub,
with your patch and 4.8.0 20121109 the problem in PR55137 vanishes and I am
able to run the RTEMS testsuite on PowerPC again. Thanks.
--
Sebastian Huber, embedded brains GmbH
Address : Obere Lagerstr. 30, D-82178 Puchheim, Germany
Phone : +49 89 18 90 80 79-6
Fax : +49 89
Recently support for LEON3 specific instructions were added to GCC.
Make this support available for RTEMS.
This patch should be committed to GCC 4.9.
gcc/ChangeLog
2013-08-29 Sebastian Huber
* config/sparc/t-rtems: Add leon3 multilibs.
---
gcc/config/sparc/t-rtems |4 ++--
1
ror-always still
>>does not build.
>>
>>As before, Sandra and me will serve as nios2 port maintainers.
>>
>>Attached is the patch for the compiler-proper.
>>
>>Thanks,
>>Chung-Lin
--
Sebastian Huber, embedded brains GmbH
Address : Dornierstr. 4,
=leon3 in RTEMS.
To my knowledge, GCC up to 4.8.1 doesn't support -mcpu=leon3 at all, but I
don't know about the status on current gcc-4_8-branch.
Ralf
I don't expect that this will be back ported to GCC 4.8. You also need
Binutils 2.24 for this.
--
Sebastian Huber, embedded brains
PARC-specific,
but perhaps the RMs are of a different opinion here.
A back port would be quite nice for us, since we work currently on SMP support
for LEON3/4 and C11 atomic operations would be very handy for this.
--
Sebastian Huber, embedded brains GmbH
Address : Dornierstr. 4, D-82178 Puch
on't know how the compiler works.
I run the GCC test suite on the GDB SIS with RTEMS, but its hard for me to
interpret the results. I think there are no new test failures due to the back
ports.
--
Sebastian Huber, embedded brains GmbH
Address : Dornierstr. 4, D-82178 Puchheim, Germany
Ph
On 05/17/2013 01:27 AM, David Edelsohn wrote:
On Fri, May 10, 2013 at 11:43 PM, Chung-Ju Wu wrote:
2013/5/10 Sebastian Huber :
v2: Format changes
gcc/testsuite/ChangeLog
2013-05-10 Sebastian Huber
PR target/55033
* gcc.target/powerpc/pr55033.c: New.
The testcase is
Tested on powerpc64-unknown-linux-gnu with:
make -k check RUNTESTFLAGS="--target_board=unix'{-m64,-m32,-m32/-mpowerpc64}'"
gcc/testsuite/ChangeLog
2013-06-17 Sebastian Huber
PR target/55033
* gcc.target/powerpc/pr55033.c: Fix options.
---
gcc/testsuit
Hello Chung-Ju,
On 06/18/2013 05:12 AM, Chung-Ju Wu wrote:
2013/6/18 David Edelsohn :
gcc/testsuite/ChangeLog
2013-06-17 Sebastian Huber
PR target/55033
* gcc.target/powerpc/pr55033.c: Fix options.
Okay.
Thanks, David
P.S. Please explicitly copy me on patches.
Hi, Sebastian,
Since
-boun...@rtems.org [rtems-devel-boun...@rtems.org] on behalf
of Sebastian Huber [sebastian.hu...@embedded-brains.de]
Sent: Tuesday, June 18, 2013 4:58 AM
To: rtems-de...@rtems.org
Subject: [PATCH] RTEMS: Use strict DWARF-2 on ARM, PowerPC, SPARC
Some debuggers do not cope with the new DWARF3/4 debug
e. The __dso_handle is referenced by application
code. In case this code uses the small data section, the __dso_handle
must be there.
This patch should be committed to GCC 4.8 and 4.9.
Test results:
http://gcc.gnu.org/ml/gcc-testresults/2013-07/msg00671.html
gcc/ChangeLog
2013-07
powerpc-*-eabi* pattern, e.g. powerpc-*-rtems*.
--
Sebastian Huber, embedded brains GmbH
Address : Dornierstr. 4, D-82178 Puchheim, Germany
Phone : +49 89 189 47 41-16
Fax : +49 89 189 47 41-09
E-Mail : sebastian.hu...@embedded-brains.de
PGP : Public key available on request.
Diese Nach
Ping.
On 2013-07-08 15:29, Sebastian Huber wrote:
The __cxa_atexit support is a reqirement for destructor registration of
thread-local objects.
For *-*-elf it is already enabled by default. See comment line 810 in
"gcc/config.gcc".
Define TARGET_LIBGCC_SDATA_SECTION on PowerPC fo
2014-01-06 Sebastian Huber
* config.gcc (*-*-rtems*): Add t-rtems to tmake_file.
(arm*-*-eabi* | arm*-*-symbianelf* | arm*-*-rtems*): Do not
override an existing tmake_file.
(arm*-*-rtems*): Use t-rtems from existing tmake_file.
(avr-*-rtems*): Likewise
Hello Richard,
would you mind having a look at the ARM specific part of this patch:
On 01/06/2014 07:27 PM, Sebastian Huber wrote:
2014-01-06 Sebastian Huber
* config.gcc (*-*-rtems*): Add t-rtems to tmake_file.
(arm*-*-eabi* | arm*-*-symbianelf* | arm*-*-rtems*): Do not
v2: Do not override an existing tmake_file in (arm*-*-uclinux*eabi*).
The ARM changes are already approved:
http://gcc.gnu.org/ml/gcc-patches/2014-01/msg00254.html
The rest is RTEMS specific.
gcc/ChangeLog
2014-01-07 Sebastian Huber
* config.gcc (*-*-rtems*): Add t-rtems to
f (decl && !DECL_P (decl))
decl = NULL_TREE;
before calling get_section(). The answer is that get_section() is not
prepared to handle !DECL_P trees when reporting errors. Arguably it
should be modified to do that.
--
Sebastian Huber, embedded brains GmbH
Address : Dornierstr. 4,
than two options.
gcc/ChangeLog
2013-05-08 Sebastian Huber
* config/arm/t-rtems-eabi: Remove mthumb/march=armv7 multilib.
Add mthumb/march=armv7-a multilib.
Add mthumb/march=armv7-r multilib.
Add mthumb/march=armv7-a/mfpu=neon/mfloat-abi=hard multilib.
---
gcc/c
here.
This patch is ok for mainline and 4.8.1.
Thanks. Committed revision 198762 and 198763.
Thanks a lot, and sorry for being so obtrusive.
--
Sebastian Huber, embedded brains GmbH
Address : Dornierstr. 4, D-82178 Puchheim, Germany
Phone : +49 89 189 47 41-16
Fax : +49 89 189 47 41
gcc/testsuite/ChangeLog
2013-05-10 Sebastian Huber
PR target/55033
* gcc.target/powerpc/pr55033.c: New.
---
gcc/testsuite/gcc.target/powerpc/pr55033.c | 21 +
1 files changed, 21 insertions(+), 0 deletions(-)
create mode 100644 gcc/testsuite/gcc.target
v2: Format changes
gcc/testsuite/ChangeLog
2013-05-10 Sebastian Huber
PR target/55033
* gcc.target/powerpc/pr55033.c: New.
---
gcc/testsuite/gcc.target/powerpc/pr55033.c | 23 +++
1 files changed, 23 insertions(+), 0 deletions(-)
create mode 100644 gcc
A lot of things use the -pthread GCC option if they want to use the
Pthreads. Examples are various configure scripts and some GCC test
cases. RTEMS supports Pthreads. It is convenient to simply add a
-pthread option and ignore it for RTEMS.
2013-01-29 Sebastian Huber
* config
2013-01-18 Sebastian Huber
* lib/target-supports.exp
(check_effective_target_powerpc_eabi_ok): New.
* gcc.target/powerpc/ppc-eabi.c: Use require effective target
powerpc_eabi_ok.
* gcc.target/powerpc/ppc-sdata-1.c: Likewise.
* gcc.target/powerpc
2013-01-29 Sebastian Huber
PR target/55033
* gcc.target/powerpc/pr55033.c: New.
---
gcc/testsuite/gcc.target/powerpc/pr55033.c | 21 +
1 files changed, 21 insertions(+), 0 deletions(-)
create mode 100644 gcc/testsuite/gcc.target/powerpc/pr55033.c
diff
Hello,
is it possible to integrate this patch into the GCC 4.8 release?
On 10/24/2012 04:50 PM, Alan Modra wrote:
On Tue, Oct 23, 2012 at 06:25:43PM +0200, Sebastian Huber wrote:
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=55033
This patch fixes my problem, but I am absolutely not sure if
Test results with these patches are here:
http://gcc.gnu.org/ml/gcc-testresults/2013-01/msg03064.html
--
Sebastian Huber, embedded brains GmbH
Address : Dornierstr. 4, D-82178 Puchheim, Germany
Phone : +49 89 189 47 41-16
Fax : +49 89 189 47 41-09
E-Mail : sebastian.hu...@embedded
On 01/29/2013 03:40 PM, Sebastian Huber wrote:
Hello,
is it possible to integrate this patch into the GCC 4.8 release?
With this patch we have:
LAST_UPDATED: Mon Jan 28 07:50:05 UTC 2013 (revision
5c2b636:7d84cac:9fc8b0bfcfe944067fc4c75d8b3f97eab67cecf3)
Native configuration is x86_64
Would someone mind having a look at this.
On 01/29/2013 03:38 PM, Sebastian Huber wrote:
2013-01-18 Sebastian Huber
* lib/target-supports.exp
(check_effective_target_powerpc_eabi_ok): New.
* gcc.target/powerpc/ppc-eabi.c: Use require effective target
s are okay.
Thanks a lot for your reviews.
Are you able to check them in?
I don't have commit access, but my company and I have signed the copyright
assignments regarding GCC with the FSF.
--
Sebastian Huber, embedded brains GmbH
Address : Dornierstr. 4, D-82178 Puchheim, Germany
Phone
Test results with this patch:
http://gcc.gnu.org/ml/gcc-testresults/2013-02/msg01351.html
2013-02-12 Sebastian Huber
PR target/55033
* gcc.target/powerpc/pr55033.c: New.
---
gcc/testsuite/gcc.target/powerpc/pr55033.c | 21 +
1 files changed, 21
This patch from Alan Modra fixes a section type conflict error. See also
http://gcc.gnu.org/ml/gcc-patches/2012-10/msg02172.html
Test results on PowerPC without this patch:
http://gcc.gnu.org/ml/gcc-testresults/2013-02/msg01351.html
Test results on PowerPC with this patch:
http://gcc.gnu.org/
2013-02-22 Sebastian Huber
* config.gcc (arm*-*-rtemself*): New.
(arm*-*-rtems*): Removed.
(arm*-*-eabi* | arm*-*-symbianelf*): Add (arm*-*-rtems*).
* config/arm/rtems-eabi.h: New file.
* config/arm/t-rtems-eabi: New file.
---
gcc/config.gcc
Testsuite results:
http://gcc.gnu.org/ml/gcc-testresults/2013-02/msg02508.html
2013-02-22 Sebastian Huber
* config.gcc (arm*-*-rtemself*): New.
(arm*-*-rtems*): Removed.
(arm*-*-eabi* | arm*-*-symbianelf* | arm*-*-rtemseabi*): Rename
"arm*-*-rtemseabi*&quo
?
On 02/24/2013 02:04 PM, Dave Korn wrote:
On 12/02/2013 16:11, Sebastian Huber wrote:
This patch from Alan Modra fixes a section type conflict error. See also
http://gcc.gnu.org/ml/gcc-patches/2012-10/msg02172.html
The Windows part is OK. I ran the g++ testsuite (gcc/, not libstdc) with no
This patch should be applied to GCC 4.8 and 4.9.
Test results for powerpc-rtems4.11 and arm-rtems4.11:
http://gcc.gnu.org/ml/gcc-testresults/2013-03/msg02184.html
http://gcc.gnu.org/ml/gcc-testresults/2013-03/msg02185.html
gcc/testsuite/ChangeLog
2013-03-21 Sebastian Huber
PR
This patch should be applied to GCC 4.8 and 4.9.
Test results for powerpc-rtems4.11 and arm-rtems4.11:
http://gcc.gnu.org/ml/gcc-testresults/2013-03/msg02184.html
http://gcc.gnu.org/ml/gcc-testresults/2013-03/msg02185.html
v2: Keep alphabetic order
gcc/testsuite/ChangeLog
2013-03-21 Sebastian
gcc/testsuite/ChangeLog
2013-03-22 Sebastian Huber
* gcc.c-torture/execute/builtins/builtins.exp: Sort targets
alphabetically.
---
gcc/testsuite/gcc.c-torture/execute/builtins/builtins.exp |4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/gcc/testsuite
without maintainer my
guess is that it is still a popular GNU/Linux embedded platform.
On 02/27/2013 03:40 PM, Sebastian Huber wrote:
Hello,
so now we have the approval of the Windows specific part. David Edelsohn said
that the rs6000 specific part is all right:
http://gcc.gnu.org/ml/gcc/2013-02
Ping.
On 01/29/2013 02:43 PM, Sebastian Huber wrote:
A lot of things use the -pthread GCC option if they want to use the
Pthreads. Examples are various configure scripts and some GCC test
cases. RTEMS supports Pthreads. It is convenient to simply add a
-pthread option and ignore it for RTEMS
This patch is for GCC 4.8 and 4.9.
libcpp/ChangeLog
2013-04-02 Sebastian Huber
* configure.ac: Require 64-bit int for arm*-*-rtems*.
* configure: Regenerate.
---
libcpp/configure|1 +
libcpp/configure.ac |1 +
2 files changed, 2 insertions(+), 0 deletions(-)
diff
This patch is for GCC 4.6 and 4.7.
libcpp/ChangeLog
2013-04-02 Sebastian Huber
PR target/56771
* configure.ac: Require 64-bit int for arm*-*-rtems[.0-9]*.
* configure: Regenerate.
---
libcpp/configure|1 +
libcpp/configure.ac |1 +
2 files changed, 2
This patch is for GCC 4.8 and 4.9.
v2: Fix ChangeLog.
libcpp/ChangeLog
2013-04-02 Sebastian Huber
PR target/56771
* configure.ac: Require 64-bit int for arm*-*-rtems*.
* configure: Regenerate.
---
libcpp/configure|1 +
libcpp/configure.ac |1 +
2 files
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 (omp
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
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 is
s 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
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
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
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
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 the
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 i
:
__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:
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
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): Ne
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
synchronization.
Is th
The gcov_profile_merge() already had code to deal with profile information
which had no counterpart to merge with. For profile information from files
with no associated counterpart, the profile information is simply used as is
with the weighting transformation applied. Make sure that gcov_profile
Hello Martin,
On 23/03/2022 13:19, Martin Liška wrote:
On 3/23/22 10:34, Sebastian Huber wrote:
Hello.
Thanks for the patch. Note we're in stage4, so the patch can eventually go
in in the next stage1.
ok, good.
The gcov_profile_merge() already had code to deal with profile
inform
ess of the value of the size argument.
I would not give an advice on using the nonnull attribute here. This
attribute could have pretty dangerous effects in the function definition
(removal of null pointer checks).
--
embedded brains GmbH
Herr Sebastian HUBER
Dornierstr. 4
82178 Puchheim
Ger
The gcov_profile_merge() already had code to deal with profile information
which had no counterpart to merge with. For profile information from files
with no associated counterpart, the profile information is simply used as is
with the weighting transformation applied. Make sure that gcov_profile
On 24/03/2022 11:29, Martin Liška wrote:
On 3/23/22 15:50, Sebastian Huber wrote:
The attached script reads the log file and creates the *.gcda files
using gcov-tool. Initially, the target files do not exist.
Now I've got your use-case and I like it. It's cool one can utilize
On 24/03/2022 13:03, Martin Liška wrote:
On 3/24/22 11:51, Sebastian Huber wrote:
Maybe we could add the file path into the gcov information stream
using a new tag:
#define GCOV_TAG_GCDA_FILE_NAME ((gcov_unsigned_t)0xa500)
Then the complete gcov information can be dumped using a single
header : int32:filename-magic int32:version string
#define GCOV_FILENAME_MAGIC ((gcov_unsigned_t)0x6763666e) /* "gcfn" */
--
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
On 30/03/2022 15:30, Sebastian Huber wrote:
On 30/03/2022 13:56, Martin Liška wrote:
Example:
base64 -d log.txt | gcov-tool merge-stream
The gcov-tool uses a new tag which contains the filename of the
associated gcov info file:
gcov-dump b-xilinx_zynq_a9_qemu/init.gcda
b
gcc/
* gcov-io.cc (gcov_read_string): Reword documentation comment.
---
gcc/gcov-io.cc | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/gcc/gcov-io.cc b/gcc/gcov-io.cc
index c2e9e2b6d64..72c40f8eaa0 100644
--- a/gcc/gcov-io.cc
+++ b/gcc/gcov-io.cc
@@ -473,9 +473,9
ode
-w, --weight Set weights (float point values)
Example:
base64 -d log.txt | gcov-tool merge-stream
The patch set does not change the format of gcda files.
TODO:
* Documentation
* Tests
Sebastian Huber (12):
gcov-tool: Allow merging of empty profile lists
gcov: Add m
The gcov_profile_merge() already had code to deal with profile information
which had no counterpart to merge with. For profile information from files
with no associated counterpart, the profile information is simply used as is
with the weighting transformation applied. Make sure that gcov_profile
gcc/
* gcov-io.cc (gcov_open): Always use the mode parameter.
* gcov-io.h (gcov_open): Declare it unconditionally.
libgcc/
* libgcov-driver-system.c (gcov_exit_open_gcda_file): Open file for
reading and writing.
* libgcov-util.c (read_gcda_file): Open file
gcc/
* gcov-tool.cc (gcov_do_dump): Add mode parameter.
(gcov_output_files): Open files for reading and writing.
libgcc/
* libgcov-driver-system.c (gcov_exit_open_gcda_file): Add mode
parameter. Pass mode to gcov_open() calls.
* libgcov-driver.c (dump_one
gcc/
* gcov-io.cc (GCOV_MODE_STDIN): Define.
(gcov_position): For gcov-tool, return calculated position if file is
stdin.
(gcov_open): For gcov-tool, use stdin if filename is NULL.
(gcov_close): For gcov-tool, do not close stdin.
(gcov_read_bytes):
This function is only used by gcov_write_length() in the gcov-io.cc file.
gcc/
* gcov-io.cc (gcov_seek): Make it static.
* gcov-io.h (struct gcov_summary): Do not mention gcov_seek().
libgcc/
* libgcov.h (gcov_seek): Remove define and declaration.
---
gcc/gcov-io.cc |
Move duplication of filename to caller and use xstrdup() instead of custom
code. This helps to reuse read_gcda_file() for other purposes.
libgcc/
* libgcov-util.c (read_gcda_file): Do not duplicate filename.
(ftw_read_file): Duplicate filename for read_gcda_file().
---
libgcc/li
This helps to reuse read_gcda_file().
libgcc/
* libgcov-util.c (read_gcda_file): Prepend new info object to global
list.
(ftw_read_file): Remove list append here.
---
libgcc/libgcov-util.c | 11 ---
1 file changed, 4 insertions(+), 7 deletions(-)
diff --git a/lib
This allows to reuse read_gcda_file() to read multiple objects from a single
file.
libgcc/
* libgcov-util.c (read_gcda_file): Do not open file.
(ftw_read_file): Open file here.
---
libgcc/libgcov-util.c | 16 +++-
1 file changed, 7 insertions(+), 9 deletions(-)
diff
gcc/
* gcov-io.h (GCOV_FILENAME_MAGIC): Define and document.
libgcc/
* gcov.h (__gcov_info_to_gcda): Mention __gcov_filename_to_gcfn().
(__gcov_filename_to_gcfn): Declare and document.
* libgcov-driver.c (dump_string): New.
(__gcov_filename_to_gcfn): Likew
Use an enum for file error codes.
gcc/
* gcov-io.cc (gcov_file_error): New enum.
(gcov_var): Use gcov_file_error enum for the error member.
(gcov_open): Use GCOV_FILE_NO_ERROR.
(gcov_close): Use GCOV_FILE_WRITE_ERROR.
(gcov_write): Likewise.
(gcov_w
gcc/
* gcov-tool.cc (gcov_profile_merge_stream): Declare.
(print_merge_stream_usage_message): New.
(merge_stream_usage): Likewise.
(do_merge_stream): Likewise.
(print_usage): Call print_merge_stream_usage_message().
(main): Call do_merge_stream() to
libgcc/
* libgcov-util.c (ftw_read_file): Use size_t for strlen() variables.
---
libgcc/libgcov-util.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/libgcc/libgcov-util.c b/libgcc/libgcov-util.c
index 03902ed10b1..622d5a9dc71 100644
--- a/libgcc/libgcov-util.c
++
1: 19:x = 1;
-: 20:else
2: 21:x = 2;
3: 22:}
I have some trouble to understand the output. Would 8/8 mean that we
have 100% MC/DC coverage? What does "not covered (false)" or "not
covered (true)" mean?
--
embedded b
Hello Segher,
On 15/03/2022 23:29, Segher Boessenkool wrote:
On Tue, Mar 15, 2022 at 03:29:23PM +0100, Sebastian Huber wrote:
now that the PR104829 is fixed could I back port
Segher Boessenkool (2):
rs6000: Improve .machine
rs6000: Do not use rs6000_cpu for .machine ppc and ppc64
301 - 400 of 488 matches
Mail list logo