Re: [PATCH] Use _GLIBCXX_ASSERTIONS as _GLIBCXX_DEBUG light

2021-08-22 Thread François Dumont via Gcc-patches
Any feedback ? Thanks On 08/08/21 9:34 pm, François Dumont wrote: After further testing here a fixed version which imply less changes. Moreover I already commit the fixes unrelated with this patch.     libstdc++: [_GLIBCXX_ASSERTIONS] Activate basic debug checks     libstdc++-v3/ChangeLog:

Re: [PATCH] Use _GLIBCXX_ASSERTIONS as _GLIBCXX_DEBUG light

2021-08-08 Thread François Dumont via Gcc-patches
After further testing here a fixed version which imply less changes. Moreover I already commit the fixes unrelated with this patch.     libstdc++: [_GLIBCXX_ASSERTIONS] Activate basic debug checks     libstdc++-v3/ChangeLog:     * include/bits/stl_algobase.h (equal): Use runtime-only

Re: [PATCH] Use _GLIBCXX_ASSERTIONS as _GLIBCXX_DEBUG light

2021-08-06 Thread François Dumont via Gcc-patches
On 07/06/21 6:25 am, François Dumont wrote: On 03/06/21 2:31 pm, Jonathan Wakely wrote: +  } +   /* Checks that [first, last) is a valid range, and then returns    * __first. This routine is useful when we can't use a separate    * assertion statement because, e.g., we are in a constructor.

Re: [PATCH] Use _GLIBCXX_ASSERTIONS as _GLIBCXX_DEBUG light

2021-06-06 Thread François Dumont via Gcc-patches
On 03/06/21 2:31 pm, Jonathan Wakely wrote: On 27/05/21 19:37 +0200, François Dumont via Libstdc++ wrote: We have been talking for a long time of a debug mode with less impact on performances. We already have it, that's what _GLIBCXX_ASSERTIONS already is :-) I propose to simply use the

Re: [PATCH] Use _GLIBCXX_ASSERTIONS as _GLIBCXX_DEBUG light

2021-06-03 Thread Jonathan Wakely via Gcc-patches
On 27/05/21 19:37 +0200, François Dumont via Libstdc++ wrote: We have been talking for a long time of a debug mode with less impact on performances. We already have it, that's what _GLIBCXX_ASSERTIONS already is :-) I propose to simply use the existing _GLIBCXX_ASSERTIONS macro.    

Re: [PATCH] Use _GLIBCXX_ASSERTIONS as _GLIBCXX_DEBUG light

2021-05-31 Thread François Dumont via Gcc-patches
Completing the tests revealed that this patch was missing a small change in include/bits/stl_iterator.h. Here is the updated patch.     libstdc++: [_GLIBCXX_ASSERTIONS] Activate basic debug checks     Use _GLIBCXX_ASSERTIONS as a _GLIBCXX_DEBUG light mode. When defined it activates     all

[PATCH] Use _GLIBCXX_ASSERTIONS as _GLIBCXX_DEBUG light

2021-05-27 Thread François Dumont via Gcc-patches
We have been talking for a long time of a debug mode with less impact on performances. I propose to simply use the existing _GLIBCXX_ASSERTIONS macro.     libstdc++: [_GLIBCXX_ASSERTIONS] Activate basic debug checks     Use _GLIBCXX_ASSERTIONS as a _GLIBCXX_DEBUG light mode. When defined it