Re: [PATCH] Correct -fdump-go-spec's handling of incomplete types

2020-12-15 Thread Nikhil Benesch via Gcc-patches
On 12/14/20 10:48 PM, Nikhil Benesch wrote: On 12/14/20 10:34 PM, Ian Lance Taylor wrote: On Mon, Dec 14, 2020 at 7:14 PM Nikhil Benesch wrote: Also godump now emits a dummy `type _u?pad128_t struct {}` entry, so we just suppress that and conditionally add it back. I don't understand this bi

[PATCH 1/2] Remove Report keyword for options

2020-12-15 Thread Martin Liška
There's a small clean up that removes Report as it's not used any longer by -fverbose-asm. Patch can bootstrap on x86_64-linux-gnu and survives regression tests. Ready to be installed? Thanks, Martin >From 13c7f39e59862a639823d0f47bbe1f4d9aacd736 Mon Sep 17 00:00:00 2001 From: Martin Liska Date

[PATCH 2/2] opts: Remove all usages of Report keyword.

2020-12-15 Thread Martin Liška
I've tested this mechanical change for all cross compilers. Ready to be installed? Thanks, Martin

Re: [PATCH 3/3] RISC-V: Support version controling for ISA standard extensions

2020-12-15 Thread Gerald Pfeifer
On Fri, 13 Nov 2020, Kito Cheng wrote: > - New option -misa-spec support: -misa-spec=[2.2|20190608|20191213] and >corresponding configuration option --with-isa-spec. I noticed https://gcc.gnu.org/gcc-11/changes.html is currently empty. Are you planning to add updates for the GCC 11 release?

[committed] wwwdocs: gcc-11/changes: Use https for wg21.link

2020-12-15 Thread Gerald Pfeifer
All other links to wg21.link we have use https, so make this consistent. Gerald --- htdocs/gcc-11/changes.html | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/htdocs/gcc-11/changes.html b/htdocs/gcc-11/changes.html index 5c3519ba..ea9fe93c 100644 --- a/htdocs/gcc-11/changes.ht

Re: [PATCH 29/31] PDP11: Use `const_double_zero' to express double zero constant

2020-12-15 Thread Martin Liška
On 11/20/20 4:36 AM, Maciej W. Rozycki wrote: We do not define a comparison operation between floating-point and integer data, including integer zero constant. Consequently the RTL instruction stream presented to the post-reload comparison elimination pass will include, where applicable, floatin

[committed] libstdc++: Update link to Unicode-HOWTO

2020-12-15 Thread Gerald Pfeifer
https is the new lack. Gerald libstdc++-v3/ChangeLog: 2020-12-15 Gerald Pfeifer * doc/xml/manual/codecvt.xml: Update link to Unicode-HOWTO. * doc/html/manual/facets.html: Regenerate. --- libstdc++-v3/doc/html/manual/facets.html | 2 +- libstdc++-v3/doc/xml/manual/codecvt.xm

[PATCH] lto: set nthreads_var to 1 if it is zero

2020-12-15 Thread Martin Liška
This prevents using -j0 on target where we are unable to detect number of core. Ready to be installed? Thanks, Martin gcc/ChangeLog: PR lto/98275 * lto-wrapper.c: Do not use -j0 when we are unable to detect number of cores. --- gcc/lto-wrapper.c | 2 ++ 1 file changed,

[PATCH] i386: Fix up -march=x86-64-v[234] vs. target attribute [PR98274]

2020-12-15 Thread Jakub Jelinek via Gcc-patches
Hi! The following testcase fails to compile. The problem is that when ix86_option_override_internal is called the first time for command line, it sees -mtune= wasn't present on the command line and so as fallback sets ix86_tune_string to ix86_arch_string value ("x86-64-v2"), but ix86_tune_specifi

[PATCH] match.pd: Optimize X / bool_range_Y to X [PR96094]

2020-12-15 Thread Jakub Jelinek via Gcc-patches
Hi! When the divisor is bool or has [0, 1] range, as division by 0 is UB, the only remaining option in valid programs is division by 1, so we can optimize X / bool_range_Y into X. Bootstrapped/regtested on x86_64-linux and i686-linux, ok for trunk? 2020-12-15 Jakub Jelinek PR tree-op

Re: [PATCH] lto: set nthreads_var to 1 if it is zero

2020-12-15 Thread Jakub Jelinek via Gcc-patches
On Tue, Dec 15, 2020 at 10:00:35AM +0100, Martin Liška wrote: > This prevents using -j0 on target where we are unable > to detect number of core. > > Ready to be installed? > Thanks, > Martin > > gcc/ChangeLog: > > PR lto/98275 > * lto-wrapper.c: Do not use -j0 when we are unable to

[committed] wwwdocs: readings: Update R32C/M32C/M16C and SuperH references

2020-12-15 Thread Gerald Pfeifer
renesas.com is an interesting site: I got "403 Forbidden" for these two links, when really it's about them having moved once again, URLs of 100+ characters, and blocking depending on user-agent. Oh, and try regularly accessing robots.txt -- good luck. If any of you has contacts there it would be

Re: [PATCH] i386: Fix up -march=x86-64-v[234] vs. target attribute [PR98274]

2020-12-15 Thread Uros Bizjak via Gcc-patches
On Tue, Dec 15, 2020 at 10:03 AM Jakub Jelinek wrote: > > Hi! > > The following testcase fails to compile. The problem is that > when ix86_option_override_internal is called the first time for command > line, it sees -mtune= wasn't present on the command line and so as fallback > sets ix86_tune_s

Re: [gcc r11-4816] Fix Ada build failure for the SuSE PowerPC64/Linux compiler

2020-12-15 Thread Eric Botcazou
> Are you saying that ADA doesn't support multilibs and you can only > have/use the default lib? It's Ada (the first name of Lady Lovelace), not ADA (an acronym for various organizations), see e.g https://en.wikipedia.org/wiki/Ada_Lovelace > A --target=powerpc64-linux compiler can either default

[PATCH] bswap: Handle vector CONSTRUCTORs [PR96239]

2020-12-15 Thread Jakub Jelinek via Gcc-patches
Hi! The following patch teaches the bswap pass to handle for small (2/4/8 byte long) vectors a CONSTRUCTOR by determining if the bytes of the constructor come from non-vector sources and are either nop or bswap and changing the CONSTRUCTOR in that case to VIEW_CONVERT_EXPR from scalar integer to t

Re: [gcc r11-4816] Fix Ada build failure for the SuSE PowerPC64/Linux compiler

2020-12-15 Thread Andreas Schwab
On Dez 15 2020, Eric Botcazou wrote: > The two assertions cannot possibly be true at the time. But they are. Andreas. -- Andreas Schwab, sch...@linux-m68k.org GPG Key fingerprint = 7578 EB47 D4E5 4D69 2510 2552 DF73 E780 A9DA AEC1 "And now for something completely different."

Re: [gcc r11-4816] Fix Ada build failure for the SuSE PowerPC64/Linux compiler

2020-12-15 Thread Jakub Jelinek via Gcc-patches
On Tue, Dec 15, 2020 at 10:54:47AM +0100, Andreas Schwab wrote: > On Dez 15 2020, Eric Botcazou wrote: > > > The two assertions cannot possibly be true at the time. > > But they are. Yeah, it is the same thing for Fedora/RHEL when we were supporting -m32 for powerpc. The bi-arch compiler with 32

Re: [PATCH] ira.c: Fix ICE in ira-color [PR97092]

2020-12-15 Thread Andrea Corallo via Gcc-patches
Vladimir Makarov writes: > On 2020-12-14 11:23 a.m., Andrea Corallo wrote: >> Vladimir Makarov writes: >> >>> On 2020-12-10 10:12 a.m., Andrea Corallo wrote: Hi all, following discussion on PR97092 I'd like to submit the following patch with a fix plus associated testcase. >>

[PATCH] [X86] Fold more shuffle builtins to VEC_PERM_EXPR.

2020-12-15 Thread Hongtao Liu via Gcc-patches
Hi: As indicated in PR98167, this patch is a follow-up to [1]. Bootstrapped and regtested on x86_64-linux-gnu. Ok for trunk? gcc/ PR target/98167 * config/i386/i386.c (ix86_gimple_fold_builtin): Handle IX86_BUILTIN_SHUFPD512, IX86_BUILTIN_SHUFPS512, IX86_BUILT

Re: [gcc r11-4816] Fix Ada build failure for the SuSE PowerPC64/Linux compiler

2020-12-15 Thread Eric Botcazou
> Yeah, it is the same thing for Fedora/RHEL when we were supporting -m32 > for powerpc. > The bi-arch compiler with 32-bit binaries that defaults to -m32, but also > supports -m64 was configured > --build=powerpc64-redhat-linux --target=powerpc64-redhat-linux > --with-cpu=default32 while the bi-ar

[PATCH] libstdc++: Fix the test for rvalue stream extraction

2020-12-15 Thread Ville Voutilainen via Gcc-patches
libstdc++-v3/ChangeLog: * testsuite/27_io/rvalue_streams.cc: Run the extraction to a char* for C++17 and lower only. diff --git a/libstdc++-v3/testsuite/27_io/rvalue_streams.cc b/libstdc++-v3/testsuite/27_io/rvalue_streams.cc index ad4d11c7cf3..487aa4deedd 100644 --- a/

Re: [patch] Fix PR tree-optimization/96344

2020-12-15 Thread Eric Botcazou
> OK. Thanks. Unfortunately I overlooked the comment of the function: MINVAL is the lowest case value of in the case nodes, and RANGE is highest value minus MINVAL. MINVAL and RANGE are not guaranteed to be of the same type as INDEX_EXPR (the gimplifier doesn't change the type o

Re: [gcc r11-4816] Fix Ada build failure for the SuSE PowerPC64/Linux compiler

2020-12-15 Thread Jakub Jelinek via Gcc-patches
On Tue, Dec 15, 2020 at 11:10:17AM +0100, Eric Botcazou wrote: > > Similarly for sparc, > > --build=sparc64-redhat-linux --target=sparc64-redhat-linux --with-cpu=v7 > > was the bi-arch compiler with 32-bit binaries that defaults to -m32, but > > also supports -m64, while > > --build=sparc64-redhat-

Re: [PING] [PATCH] testsuite/libstdc++: Fix required locales of a testcase

2020-12-15 Thread Jonathan Wakely via Gcc-patches
On 14/12/20 11:19 +0100, Marius Hillenbrand wrote: I'd like to ping the small fixup in https://gcc.gnu.org/pipermail/gcc-patches/2020-November/560100.html Please review and merge, if OK. Patches for libstdc++ need to be CC'd to the libstdc++ list, or I won't see them (this is mentioned at http

[PATCH] options: fix integer overflow

2020-12-15 Thread Martin Liška
The patch reports an error when a non-HOST_WIDE_INT options is used and its value is bigger than INT_MAX. Patch can bootstrap on x86_64-linux-gnu and survives regression tests. Ready to be installed? Thanks, Martin gcc/ChangeLog: PR rtl-optimization/98271 PR rtl-optimization/98

Re: [gcc r11-4816] Fix Ada build failure for the SuSE PowerPC64/Linux compiler

2020-12-15 Thread Andreas Schwab
https://build.opensuse.org/package/live_build_log/openSUSE:Factory:PowerPC/gcc10/standard/ppc https://build.opensuse.org/package/live_build_log/openSUSE:Factory:PowerPC/gcc10/standard/ppc64 Andreas. -- Andreas Schwab, sch...@linux-m68k.org GPG Key fingerprint = 7578 EB47 D4E5 4D69 2510 2552 DF7

Re: [PATCH] libstdc++: Fix the test for rvalue stream extraction

2020-12-15 Thread Jonathan Wakely via Gcc-patches
On 15/12/20 12:13 +0200, Ville Voutilainen via Libstdc++ wrote: libstdc++-v3/ChangeLog: * testsuite/27_io/rvalue_streams.cc: Run the extraction to a char* for C++17 and lower only. OK, thanks. diff --git a/libstdc++-v3/testsuite/27_io/rvalue_streams.cc b/libstdc++-

Re: [PATCH] [X86] Fold more shuffle builtins to VEC_PERM_EXPR.

2020-12-15 Thread Jakub Jelinek via Gcc-patches
On Tue, Dec 15, 2020 at 06:10:57PM +0800, Hongtao Liu via Gcc-patches wrote: > --- a/gcc/config/i386/i386.c > +++ b/gcc/config/i386/i386.c > @@ -18187,21 +18187,67 @@ ix86_gimple_fold_builtin (gimple_stmt_iterator *gsi) > } >break; > > +case IX86_BUILTIN_SHUFPD512: > +case I

Re: [committed][wwwdocs] gcc-11/changes: "used" attribute saves decls from linker garbage collection

2020-12-15 Thread Jozef Lawrynowicz
On Tue, Dec 15, 2020 at 12:48:47AM +0100, Gerald Pfeifer wrote: > On Mon, 7 Dec 2020, Jozef Lawrynowicz wrote: > > Committed as obvious. > > Thank you! > > + > +For ELF targets that support the GNU or FreeBSD OSABIs, the > +used attribute will now save the symbol declaration it is > +

[Ada] Refactor repeated code for Value attributes

2020-12-15 Thread Pierre-Marie de Rodat
Check related to restriction No_Enumeration_Maps and marking enumeration literals as referenced were done to attribute Value in both compilation and semantic checking mode (-gnatc), but to attributes Wide_Value and Wide_Wide_Value only in compilation (because those attributes are rewritten into att

[Ada] Fix possible infinite recursion in directory iterator

2020-12-15 Thread Pierre-Marie de Rodat
When the directory iterator Find is called we need to ensure that symbolic links are skipped to avoid possible circularities or exploring unrelated directories. Tested on x86_64-pc-linux-gnu, committed on trunk gcc/ada/ * libgnat/g-diopit.adb (Find): Fix possible infinite recursion

[Ada] Crash on unnesting unnamed blocks

2020-12-15 Thread Pierre-Marie de Rodat
As seen on e.g. ACATS ca5006a when front-end unnesting is enabled. Tested on x86_64-pc-linux-gnu, committed on trunk gcc/ada/ * exp_ch7.adb (Reset_Scopes_To_Block_Elab_Proc): Do not crash on a block with no Identifier. Code cleanups.diff --git a/gcc/ada/exp_ch7.adb b/gcc/ada/exp_

[Ada] Remove redundant calls to UI_To_Int in relational operands

2020-12-15 Thread Pierre-Marie de Rodat
Relational operations on Uint are overloaded to accept Int as one of the operands. There is no need to explicitly convert such operands with UI_To_Int. Cleanup only; semantics is unaffected. Tested on x86_64-pc-linux-gnu, committed on trunk gcc/ada/ * exp_disp.adb (Make_Tags): Remove cal

[Ada] Transform_Function_Array issues

2020-12-15 Thread Pierre-Marie de Rodat
As shown by ACATS c74209a, there are remaining cases where the Transform_Function_Array does not trigger properly, related to private types on one hand, and to freezing of function with no separate spec on the other hand. Tested on x86_64-pc-linux-gnu, committed on trunk gcc/ada/ * exp_c

[Ada] Mark generic body outside of SPARK

2020-12-15 Thread Pierre-Marie de Rodat
New body for Ada.Text_IO.Float_IO makes explicit use of Long_Long_Float which is not supported by GNATprove for now. Exclude that generic body from SPARK explicitly so that the unit can be instantiated from SPARK code. Tested on x86_64-pc-linux-gnu, committed on trunk gcc/ada/ * libgnat/

[Ada] Small adjustment to System.Val_Real

2020-12-15 Thread Pierre-Marie de Rodat
This recovers a few bits of precision by always using the Extra digit. Tested on x86_64-pc-linux-gnu, committed on trunk gcc/ada/ * libgnat/s-valrea.adb (Integer_to_Real): Always use Extra.diff --git a/gcc/ada/libgnat/s-valrea.adb b/gcc/ada/libgnat/s-valrea.adb --- a/gcc/ada/libgnat/s-va

[Ada] Small adjustment to parameterization of System.Value_R

2020-12-15 Thread Pierre-Marie de Rodat
This changes Precision_Limit from being computed based on the unsigned type to being explicitly specified in the instantiation, and restores its value to the precision of the mantissa for floating-pointt types. Tested on x86_64-pc-linux-gnu, committed on trunk gcc/ada/ * doc/gnat_ugn/gna

[Ada] Implement tiered support for floating-point input operations

2020-12-15 Thread Pierre-Marie de Rodat
This changes the implementation of input operations for floating-point types from using Long_Long_Float for all floating-point types to using a base type tailored to the type being operated on. This comprises adjusting Ada.Text_IO.Float_IO and Ada.Text_IO.Complex_IO to the new approach, as well as

[Ada] Complain about missing -gnat2020 switch on iterator_filter

2020-12-15 Thread Pierre-Marie de Rodat
Reuse Error_Msg_Ada_2020_Feature to complain about a missing -gnat2020 switch when compiling iterator_filter that would be legal in Ada 2020, but the compiler is expecting an earlier version of the language. A small enhancement; opportunity spotted while adding support for iterator filters in GNAT

[Ada] Refactor repeated complains about missing -gnat2005 switch

2020-12-15 Thread Pierre-Marie de Rodat
Refactor repeated code in the parser for complaining about a missing -gnat2005 switch. Fix singular-vs-plural phrasing. Tested on x86_64-pc-linux-gnu, committed on trunk gcc/ada/ * errout.ads (Error_Msg_Ada_2005_Extension): New routine (spec). * errout.adb (Error_Msg_Ada_2005_Ext

[Ada] Postcondition checks performed before finalization

2020-12-15 Thread Pierre-Marie de Rodat
This patch fixes an issue in the compiler whereby postconditions were incorrectly checked prior to the execution of finalization. Tested on x86_64-pc-linux-gnu, committed on trunk gcc/ada/ * contracts.adb, contracts.ads (Build_Postconditions_Procedure): Add declarations for Postc

[Ada] Fix serial port baud rate setting on GNU/Linux

2020-12-15 Thread Pierre-Marie de Rodat
This fixes an issue when setting the baud rate. The baud rate is set using the cfsetospeed and cfsetispeed system calls. The code is using speed_t for clarity. The non-blocking status is only reset when Block is True. And serial blocking mode is now properly set according to termios manual. Add do

[committed] libstdc++: Use init_priority attribute for Init object [PR 98108]

2020-12-15 Thread Jonathan Wakely via Gcc-patches
This causes the global objects that run the initialization code to be constructed earlier, which avoids some bugs in user code due to incorrectly relying on static initialization order. libstdc++-v3/ChangeLog: PR libstdc++/98108 * include/std/iostream (__ioinit): Add init_priorit

Re: Problem building libstdc++ for the avr target

2020-12-15 Thread Jonathan Wakely via Gcc-patches
On 10/12/20 18:39 +0100, Vladimir V via Libstdc++ wrote: Hello. Could you please have a look at my trivial patch. It works as intended with avr-libc and doesn't seem to introduce regressions for x86_64 hosts. I've pushed this to master now, thanks for the patch. What would be your suggestion

[committed] libstdc++: Remove workaround for PR c++/78173

2020-12-15 Thread Jonathan Wakely via Gcc-patches
Now that the G++ bug is fixed we no longer need to protect this partial specialization from complaining about subtracting void pointers. libstdc++-v3/ChangeLog: * include/bits/iterator_concepts.h (incrementable_traits): Remove workaround for PR c++/78173. Tested powerpc64le-linux

[PATCH v6] cacheflush.2: Document __builtin___clear_cache() as a more portable alternative

2020-12-15 Thread Alejandro Colomar via Gcc-patches
Reported-by: Heinrich Schuchardt Signed-off-by: Alejandro Colomar Cc: Martin Sebor Cc: Dave Martin --- v6: - GCC has always exposed 'void *', as Martin Sebor noted. It's Clang (and maybe others) that (following GCC's docs) exposed 'char *'. man2/cacheflush.2 | 24

Re: [PATCH] nvptx: Cache stacks block for OpenMP kernel launch

2020-12-15 Thread Julian Brown
On Tue, 8 Dec 2020 20:11:38 +0300 Alexander Monakov wrote: > On Tue, 8 Dec 2020, Julian Brown wrote: > > > Ping? > > This has addressed my concerns, thanks. Jakub, Tom -- just to confirm, is this OK for trunk now? I noticed a slight bugfix myself in the no-stacks/out-of-memory case -- i.e.

Re: [PATCH] nvptx: Cache stacks block for OpenMP kernel launch

2020-12-15 Thread Jakub Jelinek via Gcc-patches
On Tue, Dec 15, 2020 at 01:39:13PM +, Julian Brown wrote: > @@ -1922,7 +1997,9 @@ GOMP_OFFLOAD_run (int ord, void *tgt_fn, void > *tgt_vars, void **args) >nvptx_adjust_launch_bounds (tgt_fn, ptx_dev, &teams, &threads); > >size_t stack_size = nvptx_stacks_size (); > - void *stacks =

Re: [PATCH 29/31] PDP11: Use `const_double_zero' to express double zero constant

2020-12-15 Thread Maciej W. Rozycki
On Tue, 15 Dec 2020, Martin Liška wrote: > If I see correctly, starting from this revision I can't compile a cross > compiler of x86_64-linux-gnu: > > ../configure --target=pdp11-aout --disable-bootstrap --enable-languages=c,c++ > --disable-multilib --enable-obsolete Thanks for the heads-up an

Re: AVR maintainership (was: [patch, avr, 1/3] Support 64-bit (long) double: The gcc part.)

2020-12-15 Thread Segher Boessenkool
On Tue, Dec 15, 2020 at 08:40:08AM +0100, Gerald Pfeifer wrote: > On Mon, 6 Jan 2020, Jeff Law wrote: > >> p.s. You know anything about Denis Chertykov? He used to reply to avr > >> patches within hours, but my latest patches (after a long period where I > >> didn't propose any patches) where all

Re: [PATCH] rs6000: Use subreg for QI/HI vector init

2020-12-15 Thread Segher Boessenkool
Hi Ke Wen, On Tue, Dec 15, 2020 at 03:53:29PM +0800, Kewen.Lin wrote: > on 2020/12/15 上午2:51, Segher Boessenkool wrote: > > On Wed, Dec 02, 2020 at 05:44:24PM +0800, Kewen.Lin wrote: > >> --- a/gcc/config/rs6000/rs6000.c > >> +++ b/gcc/config/rs6000/rs6000.c > >> @@ -6793,17 +6793,8 @@ rs6000_expa

commit checker

2020-12-15 Thread Nathan Sidwell
Add toplevel dirs contrib/ * gcc-changelog/git_commit.py (changelog_locations): Add libcody, c++tools. pushed to trunk -- Nathan Sidwell diff --git c/contrib/gcc-changelog/git_commit.py w/contrib/gcc-changelog/git_commit.py index 0c438165516..01f49874652 1007

Re: [PATCH] Fix _GLIBCXX_DEBUG tests

2020-12-15 Thread Jonathan Wakely via Gcc-patches
On 14/12/20 22:36 +0100, François Dumont wrote: On 14/12/20 11:08 am, Jonathan Wakely wrote: On Mon, 14 Dec 2020, 06:51 François Dumont via Libstdc++, mailto:libstdc%2b...@gcc.gnu.org>> wrote: On 13/12/20 11:17 pm, Jonathan Wakely wrote: > On 13/12/20 15:52 +0100, François Dumont via

Re: [PATCH] Fix _GLIBCXX_DEBUG tests

2020-12-15 Thread Jonathan Wakely via Gcc-patches
On 15/12/20 15:20 +, Jonathan Wakely wrote: On 14/12/20 22:36 +0100, François Dumont wrote: On 14/12/20 11:08 am, Jonathan Wakely wrote: On Mon, 14 Dec 2020, 06:51 François Dumont via Libstdc++, mailto:libstdc%2b...@gcc.gnu.org>> wrote: On 13/12/20 11:17 pm, Jonathan Wakely wrote:

[committed] libstdc++: Increase timeout for tests in debug mode

2020-12-15 Thread Jonathan Wakely via Gcc-patches
These tests FAIL when testing debug mode with a small tool_timeout value. Give them a longer relative timeout. libstdc++-v3/ChangeLog: * testsuite/25_algorithms/lexicographical_compare/deque_iterators/1.cc: Add dg-timeout-factor directive. * testsuite/ext/pb_ds/regression/

c++: module mapper

2020-12-15 Thread Nathan Sidwell
To avoid always requiring an active mapper to connect to, we provide a default in-process mapper with similar functionality to the sample server. This is that code. Also included is the client-side connection mechanism, which determines what server to use and how to connect to it.

doc: Document C++ 20 modules

2020-12-15 Thread Nathan Sidwell
And here is the user-facing documentation. gcc/ * doc/cppopts.texi: Document new cpp opt. * doc/invoke.texi: Add C++20 module option & documentation. -- Nathan Sidwell diff --git c/gcc/doc/cppopts.texi w/gcc/doc/cppopts.texi index 7f1849d841f..e5ece92487b 1006

C++ 20 modules

2020-12-15 Thread Nathan Sidwell
I've completed merging modules to trunk modulo the testsuite. I included a few smoke tests, but nothing more. I'll add the rest when the smoke clears. There will undoubtedly be issues related to configs that I've not built. As I mentioned I did what I could. Further, it'll undoubtedly bre

Re: [patch, fortran] Optionally improve debugging of auxiliary variables

2020-12-15 Thread Thomas Koenig via Gcc-patches
Am 13.12.20 um 20:59 schrieb Iain Sandoe via Fortran: For bonus points you can prefix with _GFC_ which puts the symbols in the implementation’s namespace (at least for C-Family purposes) and therefore means that a clash with a user’s symbols is the user’s problem … I don't think this is necessa

Re: [PATCH] nvptx: Cache stacks block for OpenMP kernel launch

2020-12-15 Thread Julian Brown
On Tue, 15 Dec 2020 14:49:40 +0100 Jakub Jelinek wrote: > On Tue, Dec 15, 2020 at 01:39:13PM +, Julian Brown wrote: > > @@ -1922,7 +1997,9 @@ GOMP_OFFLOAD_run (int ord, void *tgt_fn, void > > *tgt_vars, void **args) nvptx_adjust_launch_bounds (tgt_fn, > > ptx_dev, &teams, &threads); > >s

Re: [PATCH] nvptx: Cache stacks block for OpenMP kernel launch

2020-12-15 Thread Jakub Jelinek via Gcc-patches
On Tue, Dec 15, 2020 at 04:49:38PM +, Julian Brown wrote: > > Do you need to hold the omp_stacks.lock across the entire offloading? > > Doesn't that serialize all offloading kernels to the same device? > > I mean, can't the lock be taken just shortly at the start to either > > acquire the cache

[PATCH] maintainer-scripts: Prefer temp dir /sourceware/snapshot-tmp/gcc for update_version_git

2020-12-15 Thread Jakub Jelinek via Gcc-patches
On Mon, Dec 14, 2020 at 11:58:05PM +, Joseph Myers wrote: > > Thanks for heads up. I'm aware of it and I don't see reason why (running the > > update script in dry mode works). > > https://gcc.gnu.org/pipermail/gccadmin/2020q4/017037.html > > OSError: [Errno 28] No space left on device: > '/

PING^1 [PATCH 0/3] Enable PGO/LTO build for binutils+gdb

2020-12-15 Thread H.J. Lu via Gcc-patches
On Thu, Oct 29, 2020 at 12:11 PM H.J. Lu wrote: > > Add the --enable-pgo-build[=lto] configure option. When binutils+gdb > is not built together with GCC, --enable-pgo-build enables the PGO build: > > 0. Pass --plugin to AR and RANLIB. > 1. First build with -fprofile-generate. > 2. Use "make mayb

V4 [PATCH 0/3] Switch to a new section if the SECTION_RETAIN bit doesn't match

2020-12-15 Thread H.J. Lu via Gcc-patches
When SECTION_RETAIN is used, definitions marked with used attribute and unmarked definitions are placed in a section with the same name. Instead of issue an error: [hjl@gnu-cfl-2 gcc]$ /usr/gcc-11.0.0-x32/bin/gcc -S c.c -fdiagnostics-plain-output c.c:2:49: error: ‘foo1’ causes a section type con

V4 [PATCH 3/3] Require .init_array/.fini_array support for SHF_GNU_RETAIN

2020-12-15 Thread H.J. Lu via Gcc-patches
Since SHF_GNU_RETAIN support doesn't work for crtstuff.c which switches the output section directly with asm statement: --- static void __attribute__((used)) __do_global_dtors_aux (void) { static _Bool completed; if (__builtin_expect (completed, 0)) return; completed = 1; } static void

V4 [PATCH 2/3] Warn used and not used symbols in section with the same name

2020-12-15 Thread H.J. Lu via Gcc-patches
When SECTION_RETAIN is used, issue a warning when a symbol without used attribute and a symbol with used attribute are placed in the section with the same name, like int __attribute__((used,section(".data.foo"))) foo2 = 2; int __attribute__((section(".data.foo"))) foo1 = 1; since assembler will p

V4 [PATCH 1/3] Switch to a new section if the SECTION_RETAIN bit doesn't match

2020-12-15 Thread H.J. Lu via Gcc-patches
When definitions marked with used attribute and unmarked definitions are placed in the section with the same name, switch to a new section if the SECTION_RETAIN bit doesn't match. gcc/ PR target/98146 * output.h (switch_to_section): Add a tree argument, default to nullptr.

Re: [PATCH 29/31] PDP11: Use `const_double_zero' to express double zero constant

2020-12-15 Thread Paul Koning via Gcc-patches
> On Dec 15, 2020, at 9:06 AM, Maciej W. Rozycki wrote: > > On Tue, 15 Dec 2020, Martin Liška wrote: > >> If I see correctly, starting from this revision I can't compile a cross >> compiler of x86_64-linux-gnu: >> >> ../configure --target=pdp11-aout --disable-bootstrap --enable-languages=c,c

Re: [PATCH 29/31] PDP11: Use `const_double_zero' to express double zero constant

2020-12-15 Thread Maciej W. Rozycki
On Tue, 15 Dec 2020, Paul Koning wrote: > > I'm fairly sure this is due to the difference in TARGET_SUPPORTS_WIDE_INT > > with the VAX backend vs the PDP-11 one. I have an idea how this should be > > addressed and will be implementing it shortly. > > What's the difference? pdp11 does support w

Re: [PATCH 1/2] libstdc++: Add --enable-stdio=stdio_pure option [v2]

2020-12-15 Thread Jonathan Wakely via Gcc-patches
On 10/12/20 12:56 -0800, Keith Packard via Libstdc++ wrote: Jonathan Wakely writes: I'll do a bit more testing and push it next week. That's awesome news. Thanks so much for you help; I'm looking forward to having real C++ support for my embedded customers! Pushed to master. commit 75aee

Re: [committed] libstdc++: Use init_priority attribute for Init object [PR 98108]

2020-12-15 Thread Jonathan Wakely via Gcc-patches
On 15/12/20 11:47 +, Jonathan Wakely wrote: This causes the global objects that run the initialization code to be constructed earlier, which avoids some bugs in user code due to incorrectly relying on static initialization order. libstdc++-v3/ChangeLog: PR libstdc++/98108 *

Re: [PATCH] match.pd: Optimize X / bool_range_Y to X [PR96094]

2020-12-15 Thread Jeff Law via Gcc-patches
On 12/15/20 2:05 AM, Jakub Jelinek wrote: > Hi! > > When the divisor is bool or has [0, 1] range, as division by > 0 is UB, the only remaining option in valid programs is division by 1, > so we can optimize X / bool_range_Y into X. > > Bootstrapped/regtested on x86_64-linux and i686-linux, ok fo

libcody: Fix for dash

2020-12-15 Thread Nathan Sidwell
Apparently 'var+=...' is not a dash thing. Fixed thusly. * config.m4: Avoid non-dash idiom * configure: Rebuilt. pushed (2 patches, because I didn't look carefully enough the first time) -- Nathan Sidwell diff --git i/libcody/config.m4 w/libcody/config.m4 index 364195a9f0b..0ae

Re: [PATCH 1/2] Remove Report keyword for options

2020-12-15 Thread Jeff Law via Gcc-patches
On 12/15/20 1:01 AM, Martin Liška wrote: > There's a small clean up that removes Report as it's not used > any longer by -fverbose-asm. > > Patch can bootstrap on x86_64-linux-gnu and survives regression tests. > > Ready to be installed? > Thanks, > Martin > > 0001-Remove-Report-keyword-for-opti

Re: [PATCH 2/2] opts: Remove all usages of Report keyword.

2020-12-15 Thread Jeff Law via Gcc-patches
On 12/15/20 1:02 AM, Martin Liška wrote: > I've tested this mechanical change for all cross compilers. > > Ready to be installed? ENOPATCH, but same as 1/1 of this series. OK by me given I can guess what it looks like, but give Jakub and Richi time to chime in. jeff

Re: [PATCH]AArch64: Add NEON, SVE and SVE2 RTL patterns for Complex Addition, Multiply and FMA.

2020-12-15 Thread Tamar Christina via Gcc-patches
Hi Richard, Here's the split off complex add. Bootstrapped Regtested on aarch64-none-linux-gnu and no issues. Checked with armv8-a+sve2+fp16 and no issues. Note that due to a mid-end limitation SLP for SVE currently fails for some permutes. The tests have these marked as XFAIL. Matching tests

[r11-6072 Regression] FAIL: g++.dg/warn/Wstringop-overflow-6.C -std=gnu++2a (test for excess errors) on Linux/x86_64

2020-12-15 Thread sunil.k.pandey via Gcc-patches
On Linux/x86_64, cf4ed3b41594b6935a337fe0aaf8149eadf88751 is the first bad commit commit cf4ed3b41594b6935a337fe0aaf8149eadf88751 Author: Jonathan Wakely Date: Tue Dec 15 11:40:06 2020 + libstdc++: Use init_priority attribute for Init object [PR 98108] caused FAIL: g++.dg/pch/system-

[PATCH] genemit: Handle `const_double_zero' rtx

2020-12-15 Thread Maciej W. Rozycki
Complement commit 20ab43b5cad6 ("RTL: Add `const_double_zero' syntactic rtx") and remove a commit c60d0736dff7 ("PDP11: Use `const_double_zero' to express double zero constant") build regression observed with the `pdp11-aout' target: genemit: Internal error: abort in gen_exp, at genemit.c:202 m

Re: AVR maintainership (was: [patch,avr, 1/3] Support 64-bit (long) double: The gcc part.)

2020-12-15 Thread Jeff Law via Gcc-patches
On 12/15/20 7:30 AM, Segher Boessenkool wrote: > On Tue, Dec 15, 2020 at 08:40:08AM +0100, Gerald Pfeifer wrote: >> On Mon, 6 Jan 2020, Jeff Law wrote: p.s. You know anything about Denis Chertykov? He used to reply to avr patches within hours, but my latest patches (after a long perio

[PATCH] varasm: Fix up __patchable_function_entries handling

2020-12-15 Thread Jakub Jelinek via Gcc-patches
On Wed, Dec 02, 2020 at 05:15:21AM -0800, H.J. Lu via Gcc-patches wrote: > gcc/ > > PR middle-end/93195 > PR middle-end/93197 > * configure.ac (HAVE_GAS_SECTION_LINK_ORDER): New. Define 1 if > the assembler supports the section flag 'o' for specifying > section with

Go patch committed: Better error for unexpected digit

2020-12-15 Thread Ian Lance Taylor via Gcc-patches
This patch to the Go frontend generates a better error for an unexpected digit. A digit character is not invalid in general, but it's not permitted at the start of an identifier. Report a better error message. The test case is issue11359.go in the source repo. Bootstrapped and ran Go testsuite o

Go patch committed: Discard global sinks with static initializers

2020-12-15 Thread Ian Lance Taylor via Gcc-patches
This patch to the Go frontend discards global sink variables with static initializers. This is specifically for the test issue23781.go, which builds a large static array. The code does compile and work without this change, but it takes a long time and generates a large object file. Discarding th

[PATCH] PR fortran/98284 - ICE in get_array_index

2020-12-15 Thread Harald Anlauf via Gcc-patches
Dear all, ICE-on-invalid: testcase by Arseny, draft patch by Steve, slightly polished and regtested by me on x86_64-pc-linux-gnu. OK for master? Thanks, Harald PR fortran/98284 - ICE in get_array_index Reject DATA elements with the ALLOCATABLE attribute also when they are components of a deri

Go patch committed: Avoid knock-on errors from invalid interfaces

2020-12-15 Thread Ian Lance Taylor via Gcc-patches
This patch to the Go frontend avoids knock-on errors from invalid interfaces. The test case for this is issue11614.go in the source repo. Bootstrapped and ran Go testsuite on x86_64-pc-linux-gnu. Committed to mainline. Ian 788d204885c187d5604e3960d7c78e1523f04861 diff --git a/gcc/go/gofrontend/M

Go patch committed: Correct grammar in error message

2020-12-15 Thread Ian Lance Taylor via Gcc-patches
This patch to the Go frontend corrects the grammar in an error message. Noticed while looking at https://golang.org/issue/43200. Bootstrapped and ran Go testsuite on x86_64-pc-linux-gnu. Committed to mainline. Ian 5aba02667113fbe43d74c37724f29868f098dd1c diff --git a/gcc/go/gofrontend/MERGE b/gc

Re: Go patch committed: Avoid knock-on errors from invalid interfaces

2020-12-15 Thread Ian Lance Taylor via Gcc-patches
On Tue, Dec 15, 2020 at 1:01 PM Ian Lance Taylor wrote: > > This patch to the Go frontend avoids knock-on errors from invalid > interfaces. The test case for this is issue11614.go in the source > repo. Bootstrapped and ran Go testsuite on x86_64-pc-linux-gnu. > Committed to mainline. Sorry, I f

[committed] d: Fix ICE in gimplify_expr, at gimplify.c (PR98277)

2020-12-15 Thread Iain Buclaw via Gcc-patches
Hi, This patch fixes an ICE during gimplify_expr for code generated by the D front-end. The DMD front-end shouldn't, but can sometimes leak manifest constants in the AST passed to the code generator. To prevent this being an issue, the setting of DECL_INITIAL has been moved to the point where th

libcody: Work with older gccs

2020-12-15 Thread Nathan Sidwell
Older GCCs don't have all the exciting options we have now. let's just turn them off. Makefile.in: Disable some flags -- Nathan Sidwell >From 98b960b40ba4934bdbd8d87d9432475e148d1022 Mon Sep 17 00:00:00 2001 From: Nathan Sidwell Date: Tue, 15 Dec 2020 13:34:26 -0800 Subject: [PATC

gcc_update update

2020-12-15 Thread Nathan Sidwell
I missed adding these two configure files. gcc_update: Add c++tools & libcody thanks David for pointing at the fix. nathan -- Nathan Sidwell >From c6d406f5039072938d308fe3a7fc4cd2a0bac952 Mon Sep 17 00:00:00 2001 From: Nathan Sidwell Date: Tue, 15 Dec 2020 13:42:23 -0800 Subject:

Re: [PATCH] libstdc++: Avoid zero-probability events in discrete_distribution [PR61369]

2020-12-15 Thread Lewis Hyatt via Gcc-patches
On Thu, Nov 19, 2020 at 07:16:52PM +, Jonathan Wakely wrote: > On 19/11/20 12:57 -0500, Lewis Hyatt via Libstdc++ wrote: > > Hello- > > > > PR61369 (https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61369) points out > > that std::discrete_distribution can return an event even if it has 0 > > proba

Re: Optimize combination of comparisons to dec+compare

2020-12-15 Thread Eugene Rozenfeld via Gcc-patches
Richard, > Do we already handle x < y || x <= CST to x <= y - CST? That is an invalid transformation: e.g., consider x=3, y=4, CST=2. Can you please clarify? Thanks, Eugene -Original Message- From: Richard Biener Sent: Thursday, December 10, 2020 12:21 AM To: Eugene Rozenfeld Cc: gc

[pushed] cody: Remove unused variable.

2020-12-15 Thread Marek Polacek via Gcc-patches
Tested x86_64-pc-linux-gnu, applying to trunk. libcody/ChangeLog: * buffer.cc (MessageBuffer::Lex): Remove unused variable. --- libcody/buffer.cc | 1 - 1 file changed, 1 deletion(-) diff --git a/libcody/buffer.cc b/libcody/buffer.cc index 52df3176c9a..3256c37399b 100644 --- a/libcody/b

Re: C++ 20 modules

2020-12-15 Thread Nathan Sidwell
as expected there are a bunch of configurey type errors. I am aware of the following: windows build (pr 98300), insufficient #ifing. A patch is being tested solaris 11, sys/socket.h, bcopy and poisoning. Asked jwakely to try a patch, he seems to have a build set up. AIX install. Testing

Re: C++ 20 modules

2020-12-15 Thread Rainer Orth
Hi Nathan, > as expected there are a bunch of configurey type errors. I am aware of the > following: [...] > solaris 11, sys/socket.h, bcopy and poisoning. Asked jwakely to try a > patch, he seems to have a build set up. two more Solaris issues (will file proper PRs tomorrow): * Solaris 11.4

Re: [Ada] Fix serial port baud rate setting on GNU/Linux

2020-12-15 Thread Rainer Orth
Hi Pierre-Marie, > This fixes an issue when setting the baud rate. The baud rate is set > using the cfsetospeed and cfsetispeed system calls. The code is using > speed_t for clarity. The non-blocking status is only reset when Block is > True. And serial blocking mode is now properly set according

Re: [PATCH] nvptx: Cache stacks block for OpenMP kernel launch

2020-12-15 Thread Julian Brown
On Tue, 15 Dec 2020 18:00:36 +0100 Jakub Jelinek wrote: > On Tue, Dec 15, 2020 at 04:49:38PM +, Julian Brown wrote: > > > Do you need to hold the omp_stacks.lock across the entire > > > offloading? Doesn't that serialize all offloading kernels to the > > > same device? I mean, can't the lock

Re: [PATCH] genemit: Handle `const_double_zero' rtx

2020-12-15 Thread Jeff Law via Gcc-patches
On 12/15/20 1:38 PM, Maciej W. Rozycki wrote: > Complement commit 20ab43b5cad6 ("RTL: Add `const_double_zero' syntactic > rtx") and remove a commit c60d0736dff7 ("PDP11: Use `const_double_zero' > to express double zero constant") build regression observed with the > `pdp11-aout' target: > > g

Re: [PATCH] options: fix integer overflow

2020-12-15 Thread Jeff Law via Gcc-patches
On 12/15/20 3:30 AM, Martin Liška wrote: > The patch reports an error when a non-HOST_WIDE_INT options is used > and its value is bigger than INT_MAX. > > Patch can bootstrap on x86_64-linux-gnu and survives regression tests. > > Ready to be installed? > Thanks, > Martin > > gcc/ChangeLog: > >

[pushed] c++: Remove dg-ice in constexpr-52830.C [PR52830]

2020-12-15 Thread Marek Polacek via Gcc-patches
It turned out that r11-5942 fixed this old PR, and it was detected by one of the few dg-ice tests -- exactly the point of them! Now the PR won't be opened until someone notices that it'd been fixed. The patch failed to remove the dg-ice though, so now it XPASSes. Fixing this now. Tested x86_64-p

Change to gcc/testsuite/go.test/test rejected by ChangeLog checker

2020-12-15 Thread Ian Lance Taylor via Gcc-patches
I'm trying to commit a change that adds a bunch of new files to gcc/testsuite/go.test/test. It's being rejected because the hook can't deduce a ChangeLog file. But this directory has no ChangeLog files. I've committed other changes to this directory without a ChangeLog entry. What is going wron

  1   2   >