Re: Rename DECL_IS_BUILTIN to DECL_IS_UNDECLARED_BUILTIN

2020-11-06 Thread Andreas Schwab
../../gcc/ada/gcc-interface/misc.c: In function 'const char* gnat_printable_name(tree, int)': ../../gcc/ada/gcc-interface/misc.c:562:47: error: 'DECL_IS_BUILTIN' was not declared in this scope if (verbosity == 2 && !DECL_IS_BUILTIN (decl)) Andreas. -- Andreas Schwab, sch...@linux-m68k.org

Re: [PATCH V2] arm: [testcase] Better narrow some bfloat16 testcase

2020-11-06 Thread Christophe Lyon via Gcc-patches
On Fri, 6 Nov 2020 at 15:06, Andrea Corallo wrote: > > Christophe Lyon writes: > > > On Thu, 5 Nov 2020 at 15:30, Andrea Corallo wrote: > >> > >> Christophe Lyon writes: > >> > >> > On Thu, 5 Nov 2020 at 12:11, Andrea Corallo > >> > wrote: > >> >> > >> >> Christophe Lyon writes: > >> >> >

Re: Add fnspec to C++ new and delete

2020-11-06 Thread Christophe Lyon via Gcc-patches
Hi, On Tue, 27 Oct 2020 at 11:22, Jan Hubicka wrote: > > Hi, > this patch makes C++ new and delete operators to be handled as > malloc/free for fnspec. > > I still do not understand why free is ".co " and not ".cO ". > I do not think we need to invalidate memory referenced to by blockbeing >

Re: PowerPC: Update long double IEEE 128-bit tests.

2020-11-06 Thread Michael Meissner via Gcc-patches
On Mon, Nov 02, 2020 at 07:00:15PM -0600, Segher Boessenkool wrote: > Hi! > > On Thu, Oct 22, 2020 at 06:07:14PM -0400, Michael Meissner wrote: > > This patch fixes 3 tests in the testsuite that fail if long double is set > > to IEEE 128-bit. > > > * c-c++-common/dfp/convert-bfp-11.c: If

Re: [PATCH] c++: Don't purge the satisfaction caches

2020-11-06 Thread Jason Merrill via Gcc-patches
On 11/2/20 9:03 AM, Patrick Palka wrote: On Tue, 27 Oct 2020, Patrick Palka wrote: The adoption of P2104 means we can memoize the result of satisfaction indefinitely and no longer have to clear the satisfaction caches on various events that would affect satisfaction. To that end, this patch

Re: Fwd: libstdc++: Attempt to resolve PR83562

2020-11-06 Thread Jonathan Yong via Gcc-patches
On 11/6/20 8:34 AM, Martin Storsjö wrote: On Fri, 6 Nov 2020, Liu Hao via Gcc-patches wrote: 在 2020/10/29 下午3:56, Liu Hao 写道: I forward it here for comments. Basing on the behavior of both GCC and Clang, `__cxa_thread_atexit` is used to register the destructor of thread_local objects

Re: [PATCH] "used" attribute saves decl from linker garbage collection

2020-11-06 Thread H.J. Lu via Gcc-patches
On Fri, Nov 6, 2020 at 4:17 PM Jeff Law wrote: > > > On 11/6/20 5:13 PM, H.J. Lu wrote: > > On Fri, Nov 6, 2020 at 4:01 PM Jeff Law wrote: > >> > >> On 11/6/20 4:45 PM, H.J. Lu wrote: > >>> On Fri, Nov 6, 2020 at 3:37 PM Jeff Law wrote: > On 11/6/20 4:29 PM, H.J. Lu wrote: > > On Fri,

Re: [PATCH] "used" attribute saves decl from linker garbage collection

2020-11-06 Thread Jeff Law via Gcc-patches
On 11/6/20 5:13 PM, H.J. Lu wrote: > On Fri, Nov 6, 2020 at 4:01 PM Jeff Law wrote: >> >> On 11/6/20 4:45 PM, H.J. Lu wrote: >>> On Fri, Nov 6, 2020 at 3:37 PM Jeff Law wrote: On 11/6/20 4:29 PM, H.J. Lu wrote: > On Fri, Nov 6, 2020 at 3:22 PM Jeff Law wrote: >> On 11/5/20 7:34

Re: [PATCH] "used" attribute saves decl from linker garbage collection

2020-11-06 Thread H.J. Lu via Gcc-patches
On Fri, Nov 6, 2020 at 4:01 PM Jeff Law wrote: > > > On 11/6/20 4:45 PM, H.J. Lu wrote: > > On Fri, Nov 6, 2020 at 3:37 PM Jeff Law wrote: > >> > >> On 11/6/20 4:29 PM, H.J. Lu wrote: > >>> On Fri, Nov 6, 2020 at 3:22 PM Jeff Law wrote: > On 11/5/20 7:34 AM, H.J. Lu via Gcc-patches wrote:

Re: [PATCH] "used" attribute saves decl from linker garbage collection

2020-11-06 Thread Jeff Law via Gcc-patches
On 11/6/20 4:45 PM, H.J. Lu wrote: > On Fri, Nov 6, 2020 at 3:37 PM Jeff Law wrote: >> >> On 11/6/20 4:29 PM, H.J. Lu wrote: >>> On Fri, Nov 6, 2020 at 3:22 PM Jeff Law wrote: On 11/5/20 7:34 AM, H.J. Lu via Gcc-patches wrote: > On Thu, Nov 5, 2020 at 3:37 AM Jozef Lawrynowicz >

[PATCH] rs6000: Don't use operands[] for temporaries in define_expand

2020-11-06 Thread Segher Boessenkool
In ac001f5ce604 Alan fixed my thinko using operands that do not refer to anything mentioned in the RTL pattern. Instead, it just uses fresh new local rtxes for those. This patch takes that a tiny bit further: it uses local rtx for all temporaries used in the expanders. As a bonus that

Re: [PATCH] libiberty/pex-win32.c: Initialize orig_err

2020-11-06 Thread Jeff Law via Gcc-patches
On 9/18/20 6:33 AM, Christophe Lyon wrote: > On Thu, 17 Sep 2020 at 23:33, Jeff Law wrote: >> >> On 9/14/20 3:29 AM, Christophe Lyon via Gcc-patches wrote: >>> Initializing orig_err avoids a warning: "may be used uninitialized". >>> >>> 2020-09-14 Torbjörn SVENSSON >>> Christophe Lyon

Re: [PATCH] "used" attribute saves decl from linker garbage collection

2020-11-06 Thread H.J. Lu via Gcc-patches
On Fri, Nov 6, 2020 at 3:37 PM Jeff Law wrote: > > > On 11/6/20 4:29 PM, H.J. Lu wrote: > > On Fri, Nov 6, 2020 at 3:22 PM Jeff Law wrote: > >> > >> On 11/5/20 7:34 AM, H.J. Lu via Gcc-patches wrote: > >>> On Thu, Nov 5, 2020 at 3:37 AM Jozef Lawrynowicz > >>> wrote: > On Thu, Nov 05, 2020

Re: [PATCH] "used" attribute saves decl from linker garbage collection

2020-11-06 Thread Jeff Law via Gcc-patches
On 11/6/20 4:29 PM, H.J. Lu wrote: > On Fri, Nov 6, 2020 at 3:22 PM Jeff Law wrote: >> >> On 11/5/20 7:34 AM, H.J. Lu via Gcc-patches wrote: >>> On Thu, Nov 5, 2020 at 3:37 AM Jozef Lawrynowicz >>> wrote: On Thu, Nov 05, 2020 at 06:21:21AM -0500, Hans-Peter Nilsson wrote: > On Wed, 4

Re: [PING] [PATCH] S/390: Do not turn maybe-uninitialized warnings into errors

2020-11-06 Thread Jeff Law via Gcc-patches
On 10/30/20 4:08 AM, Stefan Schulze Frielinghaus wrote: > On Wed, Oct 28, 2020 at 11:34:53AM -0600, Jeff Law wrote: >> On 10/28/20 11:29 AM, Stefan Schulze Frielinghaus wrote: >>> On Wed, Oct 28, 2020 at 08:39:41AM -0600, Jeff Law wrote: On 10/28/20 3:38 AM, Stefan Schulze Frielinghaus via

Re: [PATCH] "used" attribute saves decl from linker garbage collection

2020-11-06 Thread H.J. Lu via Gcc-patches
On Fri, Nov 6, 2020 at 3:22 PM Jeff Law wrote: > > > On 11/5/20 7:34 AM, H.J. Lu via Gcc-patches wrote: > > On Thu, Nov 5, 2020 at 3:37 AM Jozef Lawrynowicz > > wrote: > >> On Thu, Nov 05, 2020 at 06:21:21AM -0500, Hans-Peter Nilsson wrote: > >>> On Wed, 4 Nov 2020, H.J. Lu wrote: > .retain

Re: [PATCH] "used" attribute saves decl from linker garbage collection

2020-11-06 Thread Jeff Law via Gcc-patches
On 11/5/20 7:34 AM, H.J. Lu via Gcc-patches wrote: > On Thu, Nov 5, 2020 at 3:37 AM Jozef Lawrynowicz > wrote: >> On Thu, Nov 05, 2020 at 06:21:21AM -0500, Hans-Peter Nilsson wrote: >>> On Wed, 4 Nov 2020, H.J. Lu wrote: .retain is ill-defined. For example, [hjl@gnu-cfl-2 gcc]$

Re: [PATCH] "used" attribute saves decl from linker garbage collection

2020-11-06 Thread Jeff Law via Gcc-patches
On 11/5/20 4:00 AM, Jozef Lawrynowicz wrote: > On Wed, Nov 04, 2020 at 03:58:56PM -0800, H.J. Lu wrote: >> On Wed, Nov 4, 2020 at 3:00 PM Hans-Peter Nilsson wrote: >>> On Wed, 4 Nov 2020, H.J. Lu wrote: On Wed, Nov 4, 2020 at 1:56 PM Hans-Peter Nilsson wrote: > On Wed, 4 Nov

Re: [RFC] Add support for the "retain" attribute utilizing SHF_GNU_RETAIN

2020-11-06 Thread Jeff Law via Gcc-patches
On 10/26/20 2:43 PM, Jozef Lawrynowicz wrote: > On Mon, Oct 26, 2020 at 07:08:06PM +, Pedro Alves via Gcc-patches wrote: >> On 10/6/20 12:10 PM, Jozef Lawrynowicz wrote: >> >>> Should "used" apply SHF_GNU_RETAIN? >>> === >>> Another talking point is whether

Re: [PATCH] rs6000: Fix default alignment ABI break caused by MMA base support

2020-11-06 Thread Peter Bergner via Gcc-patches
On 11/6/20 4:51 PM, Segher Boessenkool wrote: > The ABI break is all GCC's faultt, but it is exposed by that glibc code, > sure :-) Right, and I'm thankful it was caught (fairly) early enough! >> This passes bootstrap and regtesting on powerpc64le-linux with no >> regressions. >> Ok for

Re: [PATCH] rs6000: Fix default alignment ABI break caused by MMA base support

2020-11-06 Thread Segher Boessenkool
Hi! On Fri, Nov 06, 2020 at 04:18:00PM -0600, Peter Bergner wrote: > As part of the MMA base support, we incremented BIGGEST_ALIGNMENT in > order to align the __vector_pair and __vector_quad types to 256 and 512 > bits respectively. This had the unintended effect of changing the > default

Re: [PATCH] mixing of labels and code in C2X

2020-11-06 Thread Joseph Myers
On Fri, 6 Nov 2020, Uecker, Martin wrote: > Am Freitag, den 06.11.2020, 22:07 + schrieb Joseph Myers: > > On Fri, 6 Nov 2020, Uecker, Martin wrote: > > > > > Hi Joseph, > > > > > > here is the revised patch. I remove the 'fallthrough' > > > code as suggested, so everything becomes even

Re: [PATCH] Fix stack pointer handling in ms_hook_prologue functions for i386 target.

2020-11-06 Thread Jeff Law via Gcc-patches
On 2/10/20 9:22 AM, Paul Gofman wrote: > ChangeLog: > PR target/91489 > * config/i386/i386.md (simple_return): Also check > for ms_hook_prologue function attribute. > * config/i386/i386.c (ix86_can_use_return_insn_p): > Also check for ms_hook_prologue function attribute.

Re: [PATCH] mixing of labels and code in C2X

2020-11-06 Thread Uecker, Martin
Am Freitag, den 06.11.2020, 22:07 + schrieb Joseph Myers: > On Fri, 6 Nov 2020, Uecker, Martin wrote: > > > Hi Joseph, > > > > here is the revised patch. I remove the 'fallthrough' > > code as suggested, so everything becomes even simpler. > > Some tests had to be changed then, but it seems

[PATCH] rs6000: Fix default alignment ABI break caused by MMA base support

2020-11-06 Thread Peter Bergner via Gcc-patches
As part of the MMA base support, we incremented BIGGEST_ALIGNMENT in order to align the __vector_pair and __vector_quad types to 256 and 512 bits respectively. This had the unintended effect of changing the default alignment used by __attribute__ ((__aligned__)) which causes an ABI break because

[PATCH] rs6000: Fix TARGET_POWERPC64 vs. TARGET_64BIT confusion

2020-11-06 Thread Segher Boessenkool
I gave Ke Wen bad advice, luckily David corrected me: it is true that we cannot use TARGET_POWERPC64 on many 32-bit OSes, since either the kernel or userland does not save the top half of the 64-bit integer registers, but we do not have to care about that in separate patterns or related code. The

Re: [PATCH] mixing of labels and code in C2X

2020-11-06 Thread Joseph Myers
On Fri, 6 Nov 2020, Uecker, Martin wrote: > Hi Joseph, > > here is the revised patch. I remove the 'fallthrough' > code as suggested, so everything becomes even simpler. > Some tests had to be changed then, but it seems Ok to > me. This patch is missing the new tests. > + *

Re: [PATCH] Objective-C/C++ (C-family) : Add missing 'atomic' property attribute.

2020-11-06 Thread Joseph Myers
On Fri, 6 Nov 2020, Iain Sandoe wrote: > Hi > > Arguably, this is actually a bug fix since the ‘atomic’ attribute is > paired with the ‘nonatomic’ one. However it is the default and was > omitted when the @property implementation was added. > > ‘atomic’ in Objective-C terms is not specified in

Re: [PATCH] Objective-C : Implement NSObject attribute.

2020-11-06 Thread Joseph Myers
On Fri, 6 Nov 2020, Iain Sandoe wrote: > Hi > > Originally, I had this as a Darwin-only patch, however GNUStep > also makes use of NSObject and similar constructs, so this really > needs to be available to linux-gnu as well. > > tested across several Darwin versions and on x86_64-linux-gnu. >

Re: [PATCH] generalized range_query class for multiple contexts

2020-11-06 Thread Andrew MacLeod via Gcc-patches
On 11/6/20 11:13 AM, Martin Sebor wrote: On 11/5/20 8:08 PM, Andrew MacLeod wrote: On 11/5/20 7:50 PM, Martin Sebor wrote: On 11/5/20 5:02 PM, Andrew MacLeod wrote: On 11/5/20 4:02 PM, Martin Sebor wrote: On 11/5/20 12:29 PM, Martin Sebor wrote: On 10/1/20 11:25 AM, Martin Sebor wrote: On

Re: ping x2 [PATCH 0/5] MSP430: Implement macros to describe relative costs of operations

2020-11-06 Thread Jeff Law via Gcc-patches
On 11/6/20 2:23 PM, Jozef Lawrynowicz wrote: > On Fri, Nov 06, 2020 at 01:53:19PM -0700, Jeff Law via Gcc-patches wrote: >> On 9/15/20 2:30 PM, Jozef Lawrynowicz wrote: >>> Ping x2 for below. >>> >>> On Fri, Aug 07, 2020 at 12:02:59PM +0100, Jozef Lawrynowicz wrote: Pinging for this series

Re: ping x2 [PATCH 0/5] MSP430: Implement macros to describe relative costs of operations

2020-11-06 Thread Jozef Lawrynowicz
On Fri, Nov 06, 2020 at 01:53:19PM -0700, Jeff Law via Gcc-patches wrote: > > On 9/15/20 2:30 PM, Jozef Lawrynowicz wrote: > > Ping x2 for below. > > > > On Fri, Aug 07, 2020 at 12:02:59PM +0100, Jozef Lawrynowicz wrote: > >> Pinging for this series of patches. > >> Attached all patches to this

Re: [PATCH GCC11]Improve uninitialized warning with value range info

2020-11-06 Thread Jeff Law via Gcc-patches
On 1/14/20 1:23 AM, Richard Biener wrote: > On Mon, Jan 13, 2020 at 3:15 AM bin.cheng wrote: >> -- >> Sender:Richard Biener >> Sent At:2020 Jan. 9 (Thu.) 20:01 >> Recipient:Bin.Cheng >> Cc:bin.cheng ; GCC Patches >> >>

Re: [04/32] cpp lexer

2020-11-06 Thread Nathan Sidwell
On 11/6/20 3:23 PM, Jeff Law wrote: 04-cpp-lexer.diff diff --git c/libcpp/include/cpplib.h w/libcpp/include/cpplib.h index 8e398863cf6..81be6457951 100644 --- c/libcpp/include/cpplib.h +++ w/libcpp/include/cpplib.h @@ -888,9 +915,9 @@ struct GTY(()) cpp_hashnode { unsigned int

Re: Patch for 96948

2020-11-06 Thread Martin Storsjö
On Fri, 6 Nov 2020, Jeff Law wrote: On 9/8/20 9:34 AM, Martin Storsjö wrote: Hi, On Tue, 8 Sep 2020, Kirill Müller wrote: Thanks for the heads up. The coincidence is funny -- a file that hasn't been touched for years. I think we both may originally be triggered from the same guy asking

Re: ping x2 [PATCH 0/5] MSP430: Implement macros to describe relative costs of operations

2020-11-06 Thread Jeff Law via Gcc-patches
On 9/15/20 2:30 PM, Jozef Lawrynowicz wrote: > Ping x2 for below. > > On Fri, Aug 07, 2020 at 12:02:59PM +0100, Jozef Lawrynowicz wrote: >> Pinging for this series of patches. >> Attached all patches to this mail with the ammended patch 4 thanks to >> Segher's review. >> >> Thanks, >> Jozef >>

ping x4 [PATCH 0/5] MSP430: Implement macros to describe relative costs of operations

2020-11-06 Thread Jozef Lawrynowicz
4th ping for below. Patches are attached, OP linked here: https://gcc.gnu.org/pipermail/gcc-patches/2020-July/550542.html Thanks, Jozef On Wed, Oct 14, 2020 at 04:31:30PM +0100, Jozef Lawrynowicz wrote: > 3rd ping for below. > > On Tue, Sep 15, 2020 at 09:30:22PM +0100, Jozef Lawrynowicz wrote:

Re: [PATCH 4/4 v2] c++: Consider only relevant template arguments in sat_hasher

2020-11-06 Thread Jason Merrill via Gcc-patches
On 11/5/20 8:40 PM, Patrick Palka wrote: A large source of cache misses in satisfy_atom is caused by the identity of an (atom,args) pair within the satisfaction cache being determined by the entire set of supplied template arguments rather than by the subset of template arguments that the atom

Re: [PATCH] pch: Specify reason of -Winvalid-pch warning [PR86674]

2020-11-06 Thread Nicholas Guriev
Hello! On Wed, 2020-03-25 at 16:46 -0600, Jeff Law wrote: > On Mon, 2020-03-09 at 11:55 +0300, Nicholas Guriev wrote: > > gcc/c-family/ChangeLog: > > > > PR pch/86674 > > * c-pch.c (c_common_valid_pch): Use cpp_warning with CPP_W_INVALID_PCH > > reason to fix -Werror=invalid-pch and

Re: [PATCH] openmp: Retire nest-var ICV

2020-11-06 Thread Tobias Burnus
Hello Kwok, hi Jakub, On 06.11.20 21:13, Kwok Cheung Yeung wrote: In addition to deprecating the omp_(get|set)_nested() functions and OMP_NESTED environment variable, OpenMP 5.0 also removes the nest-var ICV altogether, defining it in terms of the max-active-levels-var ICV instead. [...]

Re: [30/32] test harness

2020-11-06 Thread Jeff Law via Gcc-patches
On 11/3/20 2:18 PM, Nathan Sidwell wrote: > Here's the test harness change, adding a few more prune cases. > > > 30-test-harness.diff > OK jeff

Re: [19/32] global trees

2020-11-06 Thread Jeff Law via Gcc-patches
On 11/3/20 2:16 PM, Nathan Sidwell wrote: > We directly reference global trees, and expect them to be immutable. > This reorders the global tree arrays, moving the mutable ones after a > High Water Mark.  Those after the HWM are not directly accessed in the > module machinery (we'll reference by

Re: [14/32] new keywords

2020-11-06 Thread Jeff Law via Gcc-patches
On 11/3/20 2:15 PM, Nathan Sidwell wrote: > We have 3 new keywords.  As I mentioned in the preprocessor lexing, > the keywords are context-sensitive, and we create internal ones.  > These internal ones have an 'invisible' space at the end of them.  > This has the advantage of making them return

Re: [PATCH 3/4 v2] c++: Use two levels of caching in satisfy_atom

2020-11-06 Thread Jason Merrill via Gcc-patches
On 11/5/20 8:40 PM, Patrick Palka wrote: This improves the effectiveness of caching in satisfy_atom by querying the cache again after we've instantiated the atom's parameter mapping. Before instantiating its mapping, the identity of an (atom,args) pair within the satisfaction cache is

Re: [11/32] instrumentation

2020-11-06 Thread Jeff Law via Gcc-patches
On 11/3/20 2:14 PM, Nathan Sidwell wrote: > I add one new parameter -- the number of concurrently open module > files, and 3 instrumentation timers. > > > > 11-core-parmtime.diff > OK jeff

Re: [PATCH] c++: Reuse identical ATOMIC_CONSTRs during normalization

2020-11-06 Thread Jason Merrill via Gcc-patches
On 11/5/20 8:31 PM, Patrick Palka wrote: On Thu, 5 Nov 2020, Patrick Palka wrote: On Thu, 5 Nov 2020, Jason Merrill wrote: On 11/3/20 3:43 PM, Patrick Palka wrote: Profiling revealed that sat_hasher::equal accounts for nearly 40% of compile time in some cmcstl2 tests. This patch eliminates

[PATCH] Objective-C/C++ (C-family) : Add missing 'atomic' property attribute.

2020-11-06 Thread Iain Sandoe
Hi Arguably, this is actually a bug fix since the ‘atomic’ attribute is paired with the ‘nonatomic’ one. However it is the default and was omitted when the @property implementation was added. ‘atomic’ in Objective-C terms is not specified in relation to _Atomic or std::atomic (the _Atomic

Re: [04/32] cpp lexer

2020-11-06 Thread Jeff Law via Gcc-patches
On 11/3/20 2:13 PM, Nathan Sidwell wrote: > c++ modules creates 2 new kinds of preprocessor lines > [export] module ... > [export] import ... > > To all intents and purposes these are cppdirectives spelt without a > leading '#'.  module and import are context-sensitive keywords.  Thus >

[PATCH] Objective-C : Implement NSObject attribute.

2020-11-06 Thread Iain Sandoe
Hi Originally, I had this as a Darwin-only patch, however GNUStep also makes use of NSObject and similar constructs, so this really needs to be available to linux-gnu as well. tested across several Darwin versions and on x86_64-linux-gnu. OK for the c-family changes? thanks Iain = This

c++: Consistently expose singleton overloads

2020-11-06 Thread Nathan Sidwell
This is a patch from my name-lookup overhaul. I noticed the parser and one path in name-lookup looked through an overload of a single known decl. It seems more consistent to do that in both paths through name-lookup, and not in the parser itself. gcc/cp/ * name-lookup.c

Re: [committed 1/2] libstdc++: Export basic_stringbuf constructor [PR 97729]

2020-11-06 Thread Jonathan Wakely via Gcc-patches
On 06/11/20 11:56 +0100, Rainer Orth wrote: Hi Jonathan, libstdc++-v3/ChangeLog: PR libstdc++/97729 * config/abi/pre/gnu.ver (GLIBCXX_3.4.29): Add exports. * src/c++20/sstream-inst.cc (basic_stringbuf): Instantiate private constructor taking __xfer_bufptrs.

[PATCH] openmp: Retire nest-var ICV

2020-11-06 Thread Kwok Cheung Yeung
Hello In addition to deprecating the omp_(get|set)_nested() functions and OMP_NESTED environment variable, OpenMP 5.0 also removes the nest-var ICV altogether, defining it in terms of the max-active-levels-var ICV instead. This patch removes the ICV, and implements the handling of

Re: [03/32] cpp-callbacks

2020-11-06 Thread Jeff Law via Gcc-patches
On 11/3/20 2:13 PM, Nathan Sidwell wrote: > Here are the callbacks in the preprocessor itself. > > a) one to handle deferred macros > > b) one to handle include translation.  For every '#include ', > there is the possibility of replacing that with 'import '.  > This hook determines if that

Re: [02/32] linemaps

2020-11-06 Thread Jeff Law via Gcc-patches
On 11/3/20 2:13 PM, Nathan Sidwell wrote: > Location handling needs to add 2 things > > 1) a new kind of inclusion -- namely a module.  We add LC_MODULE as a > map kind, > > 2) the ability to allocate blocks of line-maps for both ordinary > locations and macro locations, that are then filled in

Re: [PATCH] c++: DR 1914 - Allow duplicate standard attributes.

2020-11-06 Thread Jason Merrill via Gcc-patches
On 11/6/20 2:34 PM, Marek Polacek wrote: On Fri, Nov 06, 2020 at 02:23:10PM -0500, Jason Merrill via Gcc-patches wrote: On 11/6/20 2:06 PM, Marek Polacek wrote: Following Joseph's change for C to allow duplicate C2x standard attributes

Re: [PATCH 1/4] c++: Fix ICE with variadic concepts and aliases [PR93907]

2020-11-06 Thread Jason Merrill via Gcc-patches
On 11/5/20 8:40 PM, Patrick Palka wrote: This patch (naively) extends the PR93907 fix to also apply to variadic concepts invoked with a type argument pack. Without this, we ICE on the below testcase (a variadic version of concepts-using2.C) in the same manner as we used to on concepts-using2.C

[pushed] Objective-C/C++ : Allow visibility prefix attributes on interfaces.

2020-11-06 Thread Iain Sandoe
Hi, Some system headers apply visibility attributes to Objective-C @interface declarations. Those are “default”, but still need to be accepted. tested across the Darwin patch and on x86_64-linux-gnu, pushed to master, thanks Iain - This passes visibility through without warning (so that,

Re: [01/32] langhooks

2020-11-06 Thread Jeff Law via Gcc-patches
On 11/3/20 2:13 PM, Nathan Sidwell wrote: > I needed a set of hook interfacing the preprocessor to the language. > they get called from pieces in c-family. > > preprocess_main_file:  we need to know when any forced headers have > been parsed in order to deal with linemaps and macro visibility >

[pushed] Objective-C/C++ (parsers) : Update @property attribute parsing.

2020-11-06 Thread Iain Sandoe
Hi This is preparatory work for bringing at least one aspect of the Objective-C implementation up to date. Tested on a number of Darwin revisions, and x86_64-linux-gnu pushed to master, thanks Iain --- At present, we are missing parsing and checking for around half of the property

[pushed] Darwin: Darwin 20 is to be macOS 11 (Big Sur).

2020-11-06 Thread Iain Sandoe
Hi, It looks like the next macOS release is imminent. Tested across the Darwin patch, and on x86_64-linux-gnu, pushed to master thanks Iain -- As per Nigel Tufnel's assertion "... this one goes to 11". The various parts of the code that deal with mapping Darwin versions to macOS (X)

Re: [PATCH] c++: DR 1914 - Allow duplicate standard attributes.

2020-11-06 Thread Marek Polacek via Gcc-patches
On Fri, Nov 06, 2020 at 02:23:10PM -0500, Jason Merrill via Gcc-patches wrote: > On 11/6/20 2:06 PM, Marek Polacek wrote: > > Following Joseph's change for C to allow duplicate C2x standard attributes > > , > > this patch does a

[PATCH] Combine new calculated ranges with existing range.

2020-11-06 Thread Andrew MacLeod via Gcc-patches
This patch fixed PR 97737 and 97741. basically, when we re-calculate a range, intersect it with whatever we knew before so we dont lose any accumulated information.  PR97741 details the specific case, but I was considering doing this anyway as it will allo us to retain any externally

Re: [PATCH] c++: Small tweak to can_convert_eh [PR81660]

2020-11-06 Thread Jason Merrill via Gcc-patches
On 11/6/20 1:15 AM, Marek Polacek wrote: While messing with check_handlers_1, I spotted this bug report which complains that we don't warn about the case when we have two duplicated handlers of type int. can_convert_eh implements [except.handle] and that says: A handler is a match for an

Re: [PATCH] c++: Propagate attributes to clones in duplicate_decls [PR67453]

2020-11-06 Thread Jason Merrill via Gcc-patches
On 11/6/20 4:07 AM, Jakub Jelinek wrote: Hi! On the following testcase where the cdtor attributes aren't on the in-class declaration but on an out-of-class definition, the cdtors have their clones created from the in-class declaration, and later on duplicate_decls updates attributes on the

Re: [PATCH] c++: DR 1914 - Allow duplicate standard attributes.

2020-11-06 Thread Jason Merrill via Gcc-patches
On 11/6/20 2:06 PM, Marek Polacek wrote: Following Joseph's change for C to allow duplicate C2x standard attributes , this patch does a similar thing for C++. This is DR 1914, to be resolved by , which is not part of the

[PATCH] c++: DR 1914 - Allow duplicate standard attributes.

2020-11-06 Thread Marek Polacek via Gcc-patches
Following Joseph's change for C to allow duplicate C2x standard attributes , this patch does a similar thing for C++. This is DR 1914, to be resolved by , which is not part of the standard yet, but has a wide support so look like

Re: [PATCH] RX add control register PC

2020-11-06 Thread Jeff Law via Gcc-patches
On 8/26/20 4:24 AM, Darius Galis wrote: > Hello, > Thank you for adjusting the patch. > I don't have commit privileges, so if you could please commit it, that would > be great. Sorry it took so long.  I got pulled away for most of the last few months. It's committed to the trunk. jeff

[r11-4770 Regression] FAIL: gcc.dg/ipa/modref-2.c scan-ipa-dump modref "Parm 1 param offset:0 offset:0 size:-1 max_size:64" on Linux/x86_64

2020-11-06 Thread sunil.k.pandey via Gcc-patches
On Linux/x86_64, 6cef01c32817b3d08af2cadcdb0e23c72ceed426 is the first bad commit commit 6cef01c32817b3d08af2cadcdb0e23c72ceed426 Author: Jan Hubicka Date: Fri Nov 6 10:23:58 2020 +0100 Add fnspec handling to ipa mode of ipa-modef. caused FAIL: gcc.dg/ipa/modref-2.c scan-ipa-dump modref

Re: [10/32] config

2020-11-06 Thread Nathan Sidwell
On 11/6/20 12:56 PM, Jeff Law wrote: On 11/4/20 12:24 PM, Nathan Sidwell wrote: I managed to flub sending this yesterday. This is the gcc/configure.ac changes (rebuild configure and config.h.in after applying).  Generally just checking for network-related functionality.  If it's not

Re: [patch] Add dg-require-effective-target fpic to an aarch64 specific test in gcc.dg

2020-11-06 Thread Olivier Hainque
> On 6 Nov 2020, at 19:03, Richard Sandiford wrote: >>> OK, thanks. Also OK for any other current or future aarch64 test that >>> has -fpic or -fPIC in the options and forgets to do this. >> For the avoidance of doubt, that would hold for arm >> tests as well, right ? > > Yeah, it does. >

Re: [patch] Add dg-require-effective-target fpic to an aarch64 specific test in gcc.dg

2020-11-06 Thread Richard Sandiford via Gcc-patches
Olivier Hainque writes: >> On 4 Nov 2020, at 20:16, Richard Sandiford wrote: >> >> Olivier Hainque writes: >>> Hello, >>> >>> This patch adds dg-require-effective-target fpic >>> to an aarch64 specific gcc.dg test using -fPIC, >>> which helps circumvent a failure we observed while >>> testing

Re: [10/32] config

2020-11-06 Thread Jeff Law via Gcc-patches
On 11/4/20 12:24 PM, Nathan Sidwell wrote: > I managed to flub sending this yesterday. > > This is the gcc/configure.ac changes (rebuild configure and > config.h.in after applying).  Generally just checking for > network-related functionality.  If it's not available, those features > of the

Re: [PATCH, rs6000] Update instruction attributes for Power10

2020-11-06 Thread will schmidt via Gcc-patches
On Fri, 2020-11-06 at 10:46 -0600, Pat Haugen wrote: > On 11/5/20 4:32 PM, will schmidt wrote: > > On Wed, 2020-11-04 at 14:42 -0600, Pat Haugen via Gcc-patches > > wrote: > > > * config/rs6000/rs6000.c (rs6000_final_prescan_insn): Only add > > > 'p' for > > > PREFIXED_YES. > > > > The code

Re: [PATCH][AArch64] Use intrinsics for upper saturating shift right

2020-11-06 Thread Richard Sandiford via Gcc-patches
David Candler writes: > Hi Richard, > > Thanks for the feedback. > > Richard Sandiford writes: >> > diff --git a/gcc/config/aarch64/aarch64-builtins.c >> > b/gcc/config/aarch64/aarch64-builtins.c >> > index 4f33dd936c7..f93f4e29c89 100644 >> > --- a/gcc/config/aarch64/aarch64-builtins.c >> >

Re: float.h: C2x decimal signaling NaN macros

2020-11-06 Thread Jeff Law via Gcc-patches
On 11/5/20 4:21 PM, Joseph Myers wrote: > C2x adds macros for decimal floating-point signaling NaNs to > . Add these macros to GCC's implementation. > > Note that the current C2x draft has these under incorrect names > D32_SNAN, D64_SNAN, D128_SNAN. The intent was to change the naming >

Re: builtins: Add DFP signaling NaN built-in functions

2020-11-06 Thread Jeff Law via Gcc-patches
On 11/4/20 5:35 PM, Joseph Myers wrote: > Add built-in functions __builtin_nansd32, __builtin_nansd64 and > __builtin_nansd128 to return signaling NaNs of decimal floating-point > types, analogous to the functions already present for binary > floating-point types. > > This patch, independent of

Re: [Patch] testsuite: Avoid TCL errors when rootme or ASAN/TSAN/UBSAN is not available (was: Re: [Patch] testsuite: Avoid TCL errors when ASAN/TSAN/UBSAN is not available)

2020-11-06 Thread Jeff Law via Gcc-patches
On 10/19/20 10:03 AM, Tobias Burnus wrote: > Thomas Schwinge and Joseph convinced me that 'rootme' only makes sense > for in-tree testing and, hence, does not need (or: should not) be set in > site.exp. > > Thus, if it is not set, we have to check its existence before using it – > to avoid

Re: [Patch, fortran] PR83118 - [8/9/10/11 Regression] Bad intrinsic assignment of class(*) array component of derived type

2020-11-06 Thread Tobias Burnus
Hi Paul, sorry for the belated attempt to review your patch. Attempt as both via @gcc.gnu.org as in the direct email, I did not see the attached patch. (Matches what Andre mentioned to you today at IRC #gfortran.) I only see: * Content-Type: multipart/alternative Content-Type: text/plain;

Re: [PATCH] c-family: Fix regression in location-overflow-test-1.c [PR97117]

2020-11-06 Thread Jeff Law via Gcc-patches
On 10/14/20 9:56 AM, Patrick Palka via Gcc-patches wrote: > The r11-3266 patch that added macro support to -Wmisleading-indentation > accidentally suppressed the column-tracking diagnostic in > get_visual_column in some cases, e.g. in the location-overflow-test-1.c > testcase. > > More

Re: [PATCH] libcpp: Update cpp_wcwidth() to Unicode 13.0.0

2020-11-06 Thread Jeff Law via Gcc-patches
On 10/23/20 9:01 AM, Lewis Hyatt via Gcc-patches wrote: > Hello- > > The attached patch updates cpp_wcwidth() (for computation of display > widths needed to calculate column numbers in diagnostics) from Unicode 12 > to Unicode 13. The patch was purely mechanical, following the directions > in

Re: Rename DECL_IS_BUILTIN to DECL_IS_UNDECLARED_BUILTIN

2020-11-06 Thread Jeff Law via Gcc-patches
On 10/23/20 6:48 AM, Nathan Sidwell wrote: > Patch affects C++, C, GO, common-core > > In cleaning up C++'s handling of hidden decls, I renamed its > DECL_BUILTIN_P, which checks for loc == BUILTINS_LOCATION to > DECL_UNDECLARED_BUILTIN_P, because the location gets updated, if user > source

Re: [PATCH] Optimize macro: make it more predictable

2020-11-06 Thread Jeff Law via Gcc-patches
On 10/23/20 5:47 AM, Martin Liška wrote: > Hey. > > This is a follow-up of the discussion that happened in thread about > no_stack_protector > attribute: https://gcc.gnu.org/pipermail/gcc-patches/2020-May/545916.html > > The current optimize attribute works in the following way: > - 1) we take

Re: [PATCH] arc: Improve/add instruction patterns to better use MAC instructions.

2020-11-06 Thread Jeff Law via Gcc-patches
On 10/9/20 8:24 AM, Claudiu Zissulescu wrote: > From: Claudiu Zissulescu > > ARC MYP7+ instructions add MAC instructions for vector and scalar data > types. This patch adds a madd pattern for 16it datum that is using the > 32bit MAC instruction, and dot_prod patterns for v4hi vector > types.

Re: [PATCH] Support the new ("v0") mangling scheme in rust-demangle.

2020-11-06 Thread Jeff Law via Gcc-patches
On 11/1/20 10:26 AM, Nikhil Benesch via Gcc-patches wrote: > > > On 11/1/20 6:57 AM, Eduard-Mihai Burtescu wrote: >> Reading the diff patch, the v0 changes look great. I wouldn't be too >> worried >> about the "printable character" aspect, there are similar >> Unicode-related >> issues

Re: [21/32] miscelaneous

2020-11-06 Thread Nathan Sidwell
On 11/5/20 8:30 AM, Richard Biener wrote: On Tue, Nov 3, 2020 at 10:16 PM Nathan Sidwell wrote: These are changes to gcc/tree.h adding some raw accessors to nodes, which seemed preferable to direct field access. I also needed access to the integral constant cache can you please document

[PATCH] rework PRE PHI translation cache

2020-11-06 Thread Richard Biener
Turns out its size and time requirements can be stripped down dramatically. Bootstrap & regtest running on x86_64-unknown-linux-gnu. 2020-11-06 Richard Biener * tree-ssa-pre.c (expr_pred_trans_d): Modify so elements are embedded rather than allocated. Remove hashval member,

libcpp: Provide date routine

2020-11-06 Thread Nathan Sidwell
Joseph pointed me at cb_get_source_date_epoch, which allows repeatable builds and solves a FIXME I had on the modules branch. Unfortunately it's used exclusively to generate __DATE__ and __TIME__ values, which fallback to using a time(2) call. It'd be nicer if the preprocessor made whatever

[PATCH] aarch64: Support permutes on unpacked SVE vectors

2020-11-06 Thread Richard Sandiford via Gcc-patches
This patch adds support for permuting unpacked SVE vectors using: - DUP - EXT - REV[BHW] - REV - TRN[12] - UZP[12] - ZIP[12] This involves rewriting the REV[BHW] permute code so that the inputs and outputs of the insn pattern have the same mode as the vectors being permuted. This is different

Re: [PATCH, rs6000] Update instruction attributes for Power10

2020-11-06 Thread Pat Haugen via Gcc-patches
On 11/5/20 4:32 PM, will schmidt wrote: > On Wed, 2020-11-04 at 14:42 -0600, Pat Haugen via Gcc-patches wrote: >> * config/rs6000/rs6000.c (rs6000_final_prescan_insn): Only add 'p' for >> PREFIXED_YES. > > The code change reads as roughly > - next_insn_prefixed_p != PREFIXED_NO > > +

Re: Patch for 96948

2020-11-06 Thread Jeff Law via Gcc-patches
On 9/8/20 9:34 AM, Martin Storsjö wrote: > Hi, > > On Tue, 8 Sep 2020, Kirill Müller wrote: > >> Thanks for the heads up. The coincidence is funny -- a file that >> hasn't been touched for years. > > I think we both may originally be triggered from the same guy asking > around in different

Re: [PATCH] issue -Wstring-compare in more case (PR 95673)

2020-11-06 Thread Jeff Law via Gcc-patches
On 9/30/20 6:14 PM, Martin Sebor via Gcc-patches wrote: > -Wstring-compare triggers under the same strict conditions as > the strcmp/strncmp call is folded into a constant: only when > all the uses of the result are [in]equality expressions with > zero.  However, even when the call cannot be

Re: [PATCH] use get_size_range to get allocated size (PR 92942)

2020-11-06 Thread Jeff Law via Gcc-patches
On 8/28/20 11:12 AM, Martin Sebor via Gcc-patches wrote: > The gimple_call_alloc_size() function that determines the range > of sizes of allocated objects and constrains the bounds in calls > to functions like memcpy calls get_range() instead of > get_size_range() to obtain its result.  The

Re: [PATCH] generalized range_query class for multiple contexts

2020-11-06 Thread Martin Sebor via Gcc-patches
On 11/5/20 8:08 PM, Andrew MacLeod wrote: On 11/5/20 7:50 PM, Martin Sebor wrote: On 11/5/20 5:02 PM, Andrew MacLeod wrote: On 11/5/20 4:02 PM, Martin Sebor wrote: On 11/5/20 12:29 PM, Martin Sebor wrote: On 10/1/20 11:25 AM, Martin Sebor wrote: On 10/1/20 9:34 AM, Aldy Hernandez wrote:

Re: Fix uninitialized memory use in ipa-modref

2020-11-06 Thread Martin Liška
On 11/5/20 6:37 PM, Jan Hubicka wrote: We can't because our vec does not accept non-pods and this needs to be GGC safe since it points to trees. Ah, that's new to me! Thanks, Martin

Re: Fix uninitialized memory use in ipa-modref

2020-11-06 Thread Martin Liška
On 11/5/20 6:54 PM, Jan Hubicka wrote: On 11/5/20 3:27 PM, Jan Hubicka wrote: poly_int64 offset; struct modref_parm_map parm_map; + parm_map.parm_offset_known = false; + parm_map.parm_offset = 0; + I'm curious, can't we use a proper C++ class construction. The IPA pass is new and

Re: [PATCH] arm: Implement vceqq_p64, vceqz_p64 and vceqzq_p64 intrinsics

2020-11-06 Thread Christophe Lyon via Gcc-patches
On Thu, 5 Nov 2020 at 12:55, Christophe Lyon wrote: > > On Thu, 5 Nov 2020 at 10:36, Kyrylo Tkachov wrote: > > > > H, Christophe, > > > > > -Original Message- > > > From: Gcc-patches On Behalf Of > > > Christophe Lyon via Gcc-patches > > > Sent: 15 October 2020 18:23 > > > To:

Re: [PATCH]ira: recompute regstat as max_regno changes [PR97705]

2020-11-06 Thread Vladimir Makarov via Gcc-patches
On 2020-11-06 1:15 a.m., Kewen.Lin wrote: Hi, As PR97705 shows, my commit r11-4637 caused some dumping comparison difference error on pass ira. It exposed one issue about the newly introduced function remove_scratches, which can increase the largest pseudo reg number if it succeeds, later

[PATCH] make PRE constant value IDs negative

2020-11-06 Thread Richard Biener
This separates constant and non-constant value-ids to allow for a more efficient constant_value_id_p and for more efficient bit-packing inside the bitmap sets which never contain any constant values. There's further optimization opportunities but at this stage I'll do small refactorings.

[PATCH 3/6] Add Field Reordering

2020-11-06 Thread Erick Ochoa
From 72e6ea57b04ca2bf223faef262b478dc407cdca7 Mon Sep 17 00:00:00 2001 From: Erick Ochoa Date: Sun, 9 Aug 2020 10:22:49 +0200 Subject: [PATCH 3/6] Add Field Reordering Field reordering of structs at link-time 2020-11-04 Erick Ochoa * gcc/Makefile.in: add new file to list of sources

[PATCH 6/6] Add heuristic to take into account void* pattern.

2020-11-06 Thread Erick Ochoa
From ccd82a7e484d9e4562c23f1b9cbebf3f47e2a822 Mon Sep 17 00:00:00 2001 From: Erick Ochoa Date: Fri, 16 Oct 2020 08:49:08 +0200 Subject: [PATCH 6/6] Add heuristic to take into account void* pattern. We add a heuristic in order to be able to transform functions which receive void* arguments as a

  1   2   >