Re: [stage1] [PATCH] Merge dg-options and dg-additional-options if len <= 120 chars.

2020-05-04 Thread Martin Liška
On 4/28/20 6:13 PM, Christophe Lyon wrote: Hi, On Thu, 16 Apr 2020 at 09:50, Martin Liška wrote: On 4/14/20 1:43 PM, Jakub Jelinek wrote: Roughly, yes. A few extra in testcases don't hurt necessarily, but say 160 chars or more is clearly too much. All right, I made a limit of 120

Re: [PATCH v4] c++: Member template function lookup failure [PR94799]

2020-05-04 Thread Jason Merrill via Gcc-patches
On 5/4/20 9:51 PM, Marek Polacek wrote: On Mon, May 04, 2020 at 05:41:37PM -0400, Jason Merrill via Gcc-patches wrote: On 5/4/20 4:37 PM, Marek Polacek wrote: On Fri, May 01, 2020 at 04:12:35PM -0400, Jason Merrill wrote: @@ -7754,9 +7755,22 @@ cp_parser_postfix_dot_deref_expression

Re: [PATCH v4] c++: Member template function lookup failure [PR94799]

2020-05-04 Thread Marek Polacek via Gcc-patches
On Mon, May 04, 2020 at 05:41:37PM -0400, Jason Merrill via Gcc-patches wrote: > On 5/4/20 4:37 PM, Marek Polacek wrote: > > On Fri, May 01, 2020 at 04:12:35PM -0400, Jason Merrill wrote: > > > > @@ -7754,9 +7755,22 @@ cp_parser_postfix_dot_deref_expression > > > > (cp_parser *parser, > > > >

Re: [PATCH] c++: ICE in value_dependent_expression_p in C++98 mode [PR94938]

2020-05-04 Thread Jason Merrill via Gcc-patches
On 5/4/20 7:32 PM, Marek Polacek wrote: Here we ICE with -std=c++98 since the newly added call to uses_template_parms (r10-6357): we hit 26530 gcc_assert (cxx_dialect >= cxx11 26531 || INTEGRAL_OR_ENUMERATION_TYPE_P (type)); and TYPE is a record type. The

[PATCH] c++: ICE in value_dependent_expression_p in C++98 mode [PR94938]

2020-05-04 Thread Marek Polacek via Gcc-patches
Here we ICE with -std=c++98 since the newly added call to uses_template_parms (r10-6357): we hit 26530 gcc_assert (cxx_dialect >= cxx11 26531 || INTEGRAL_OR_ENUMERATION_TYPE_P (type)); and TYPE is a record type. The problem is that the argument to

Re: [PATCH 2/4] x86: Add -mzero-caller-saved-regs=[skip|used-gpr|all-gpr|used|all]

2020-05-04 Thread Rodriguez Bahena, Victor via Gcc-patches
-Original Message- From: "H.J. Lu" Date: Monday, May 4, 2020 at 2:02 PM To: "gcc-patches@gcc.gnu.org" Cc: Uros Bizjak , Jeff Law , Richard Biener , Jakub Jelinek , Qing Zhao , "keesc...@chromium.org" , "Rodriguez Bahena, Victor" Subject: [PATCH 2/4] x86: Add

Re: [PATCH] libstdc++: Fix the return type of __cxa_finalize according to the Itanium C++ ABI

2020-05-04 Thread Jonathan Wakely via Gcc-patches
On 12/03/20 10:13 +, Jonathan Wakely via Libstdc++ wrote: Please CC libstd...@gcc.gnu.org for all libstdc++ patches, as per https://gcc.gnu.org/lists.html On 11/03/20 21:24 -0700, Fangrui Song wrote: Alternatively, we can delete it, because no user code should call it. It may be weird that

Re: [PATCH v3] [Stage1] Refactor tree-ssa-operands.c

2020-05-04 Thread Giuliano Belinassi via Gcc-patches
Hi, This patch Refactors tree-ssa-operands.c by wrapping the global variables into a class, and also removes unused code. The difference between this version and v2 is: * Disable the copy of operands_scanner * remove void from empty arguments functions. gcc/ChangeLog: 2020-05-04 Giuliano

[committed] libstdc++: Fix broken link to SGI STL FAQ

2020-05-04 Thread Jonathan Wakely via Gcc-patches
The previous URL to an entry in the wayback machine now redirects to a page saying "SGI.com Tech Archive Resources now retired" so use an older entry from the archive. * doc/xml/faq.xml: Use working link for SGI STL FAQ. * doc/html/*: Regenerate. Committed to master, I'll

[committed] libstdc++: Fix incorrect size calculation in PMR resource (PR 94906)

2020-05-04 Thread Jonathan Wakely via Gcc-patches
Calculating the size of a chunk being returned to the upstream allocator was done with a 32-bit type, so it wrapped if the chunk was 4GB or larger. I don't know how to test this without allocating 4GB, so there's no test in the testsuite. It has been tested manually with allocations sizes and

Re: [PATCH v3] c++: Member template function lookup failure [PR94799]

2020-05-04 Thread Jason Merrill via Gcc-patches
On 5/4/20 4:37 PM, Marek Polacek wrote: On Fri, May 01, 2020 at 04:12:35PM -0400, Jason Merrill wrote: @@ -7754,9 +7755,22 @@ cp_parser_postfix_dot_deref_expression (cp_parser *parser, } if (dependent_p) -/* Tell cp_parser_lookup_name that there was an object, even though it's

Re: [PATCH] c++: Avoid unnecessary copying in cp_fold [PR94038]

2020-05-04 Thread Jason Merrill via Gcc-patches
On 5/4/20 3:44 PM, Patrick Palka wrote: When folding a CALL_EXPR, we can avoid copying it until folding changes one of its arguments. And when folding a TREE_VEC, we can avoid using an intermediate releasing_vec by working with a copy of the TREE_VEC as soon as folding changes one of its

Re: [PATCH] PowerPC -mcpu=future Patch 4 of 7, Make sure an invalid instruction is not generated

2020-05-04 Thread Segher Boessenkool
Hi! On Mon, Apr 27, 2020 at 03:57:59PM -0400, Michael Meissner wrote: > +#ifndef SIZE > +#define SIZE 5 > +#endif > + > +struct foo { > + unsigned int field; > + char pad[SIZE]; > +}; > + > +struct foo *inc_load (struct foo *p, unsigned int *q) > +{ > + *q = (++p)->field; /* PLWZ, PADDI,

Re: [PATCH v3] c++: Member template function lookup failure [PR94799]

2020-05-04 Thread Marek Polacek via Gcc-patches
On Fri, May 01, 2020 at 04:12:35PM -0400, Jason Merrill wrote: > > @@ -7754,9 +7755,22 @@ cp_parser_postfix_dot_deref_expression (cp_parser > > *parser, > > } > > if (dependent_p) > > -/* Tell cp_parser_lookup_name that there was an object, even though > > it's > > -

Re: [PATCH] PowerPC -mcpu=future Patch 4 of 7, Make sure an invalid instruction is not generated

2020-05-04 Thread Segher Boessenkool
Hi! On Mon, Apr 27, 2020 at 05:09:15PM -0500, will schmidt wrote: > > * gcc.target/powerpc/prefix-no-premodify.c: Make sure we do not > > generate the non-existent PLWZU instruction if -mcpu=future. > > Should "premodify" instead be "no-update" or "no-update-forms" ? I > don't

[Patch] PR fortran/93366 - ICE on invalid, reject invalid use of NULL() as argument

2020-05-04 Thread Harald Anlauf
The attached patch fixes a couple of cases where an invalid NULL() argument provided to intrinsics produced either an ICE or was silently accepted. We now reject them. Steve Kargl provided an initial version, which I extended to catch a few more cases. Regtests with no new failures on

Re: [PATCH][AArch64] Use __getauxval instead of getauxval in LSE detection code in libgcc

2020-05-04 Thread Joseph Myers
On Mon, 4 May 2020, Szabolcs Nagy wrote: > i think __getauxval needs to be declared where it is used. Yes. > and libgcc should do a link test in configure becasuse a > libc may provide sys/auxv.h but not have __getauxval e.g. > musl libc does not export __getauxval, only getauxval. libgcc

Re: PING[STAGE 1][PATCH][x86][1/3]: Add -mzero-caller-saved-regs=[skip|used-gpr|all-gpr|used|all]

2020-05-04 Thread H.J. Lu via Gcc-patches
On Mon, May 4, 2020 at 12:48 PM Kees Cook wrote: > > On Mon, May 04, 2020 at 12:02:45PM -0700, H.J. Lu wrote: > > On Mon, May 4, 2020 at 11:21 AM Kees Cook wrote: > > > > > > On Mon, May 04, 2020 at 11:51:49AM -0500, Qing Zhao wrote: > > > > Hi, > > > > > > > > This is a PING for this patch for

[pushed] coroutines: Mark the gro as artificial.

2020-05-04 Thread Iain Sandoe
Hi This corrects an oversight; the coro.gro object is a a compiler-generated entity and should be marked as artificial and ignored. tested on x86_64-darwin16, applied to master as obvious, thanks Iain gcc/cp/ChangeLog: 2020-05-04 Iain Sandoe * coroutines.cc (morph_fn_to_coro): Mark

Re: PING[STAGE 1][PATCH][x86][1/3]: Add -mzero-caller-saved-regs=[skip|used-gpr|all-gpr|used|all]

2020-05-04 Thread Kees Cook via Gcc-patches
On Mon, May 04, 2020 at 12:02:45PM -0700, H.J. Lu wrote: > On Mon, May 4, 2020 at 11:21 AM Kees Cook wrote: > > > > On Mon, May 04, 2020 at 11:51:49AM -0500, Qing Zhao wrote: > > > Hi, > > > > > > This is a PING for this patch for gcc11 stage 1. > > > > > >

[PATCH] c++: Avoid unnecessary copying in cp_fold [PR94038]

2020-05-04 Thread Patrick Palka via Gcc-patches
When folding a CALL_EXPR, we can avoid copying it until folding changes one of its arguments. And when folding a TREE_VEC, we can avoid using an intermediate releasing_vec by working with a copy of the TREE_VEC as soon as folding changes one of its arguments, like we do in the CALL_EXPR case.

Re: PING[STAGE 1][PATCH][x86][1/3]: Add -mzero-caller-saved-regs=[skip|used-gpr|all-gpr|used|all]

2020-05-04 Thread H.J. Lu via Gcc-patches
On Mon, May 4, 2020 at 11:21 AM Kees Cook wrote: > > On Mon, May 04, 2020 at 11:51:49AM -0500, Qing Zhao wrote: > > Hi, > > > > This is a PING for this patch for gcc11 stage 1. > > > > https://gcc.gnu.org/pipermail/gcc-patches/2020-April/544058.html > >

[PATCH 1/4] matcher-1.m: Change return type to int

2020-05-04 Thread H.J. Lu via Gcc-patches
my_exception_matcher must return int. Otherwise, this test fails. PR testsuite/84324 * objc/execute/exceptions/matcher-1.m (my_exception_matcher): Change return type to int. --- gcc/testsuite/objc/execute/exceptions/matcher-1.m | 2 +- 1 file changed, 1 insertion(+), 1

[PATCH 4/4] Update gcc.target/i386/ret-thunk-2[234].c

2020-05-04 Thread H.J. Lu via Gcc-patches
--- gcc/testsuite/gcc.target/i386/ret-thunk-22.c | 2 +- gcc/testsuite/gcc.target/i386/ret-thunk-23.c | 2 +- gcc/testsuite/gcc.target/i386/ret-thunk-24.c | 2 +- 3 files changed, 3 insertions(+), 3 deletions(-) diff --git a/gcc/testsuite/gcc.target/i386/ret-thunk-22.c

[PATCH 2/4] x86: Add -mzero-caller-saved-regs=[skip|used-gpr|all-gpr|used|all]

2020-05-04 Thread H.J. Lu via Gcc-patches
Add -mzero-caller-saved-regs=[skip|used-gpr|all-gpr|used|all] command-line option and zero_caller_saved_regs("skip|used|all") function attribue: 1. -mzero-caller-saved-regs=skip and zero_caller_saved_regs("skip") Don't zero caller-saved registers upon function return. 2.

[PATCH 3/4] x86: Add ix86_any_return_p

2020-05-04 Thread H.J. Lu via Gcc-patches
Add ix86_any_return_p to check simple_return in a PARALLEL to support: (jump_insn 39 38 40 5 (parallel [ (simple_return) (unspec [ (const_int 0 [0]) ] UNSPEC_SIMPLE_RETURN) ]) "/tmp/x.c":105 -1 (nil) -> simple_return)

Re: PING[STAGE 1][PATCH][x86][1/3]: Add -mzero-caller-saved-regs=[skip|used-gpr|all-gpr|used|all]

2020-05-04 Thread Kees Cook via Gcc-patches
On Mon, May 04, 2020 at 11:51:49AM -0500, Qing Zhao wrote: > Hi, > > This is a PING for this patch for gcc11 stage 1. > > https://gcc.gnu.org/pipermail/gcc-patches/2020-April/544058.html > > > Please take a look on it. Hi! I

Re: [PATCH] contrib/vimrc: Reduce textwidth for commit messages

2020-05-04 Thread Martin Liška
On 4/29/20 7:14 PM, Jakub Jelinek via Gcc-patches wrote: On Wed, Apr 29, 2020 at 11:06:27AM -0600, Jeff Law via Gcc-patches wrote: On Tue, 2020-04-28 at 00:31 -0400, Patrick Palka via Gcc-patches wrote: The bundled vimrc script, when enabled, currently sets the text width to 80 characters for

[PR libstdc++/94747] negating a size_t does not show intent

2020-05-04 Thread Nathan Sidwell
Pushed this patch to dynamic_cast. We negate an offsetof value, which although well formed, it doesn't show intent well. The reason checkers trigger on this is that it is a cause of real bugs. So, negate a ptrdiff_t instead. nathan -- Nathan Sidwell 2020-05-04 Nathan Sidwell PR

[committed] i386: Use SBB more [PR94650]

2020-05-04 Thread Uros Bizjak via Gcc-patches
When returning 0 or -1, "SBB reg,reg" instruction that borrows carry flag can be used. Carry flag can be generated by converting compare with zero to a LTU compare with one, so e.g. return -(x == 0) generates: cmpq$1, %rdi sbbq%rax, %rax instead of: xorl

[C++] template parm cleanup

2020-05-04 Thread Nathan Sidwell
When working on 94827 I noticed that process_template_parm was doing more work than necessary -- using 'last_node' and later 'chainon'. This fixes that up to just walk the list once. Also, I didn't understand the reason why end_template_parms does a pop of the dummy header, and then a push

PING[STAGE 1][PATCH][x86][1/3]: Add -mzero-caller-saved-regs=[skip|used-gpr|all-gpr|used|all]

2020-05-04 Thread Qing Zhao via Gcc-patches
Hi, This is a PING for this patch for gcc11 stage 1. https://gcc.gnu.org/pipermail/gcc-patches/2020-April/544058.html Please take a look on it. Thanks. Qing > Begin forwarded message: > > From: Qing Zhao via Gcc-patches >

[PR 94827] Don't save tmpl parms in nested requirement

2020-05-04 Thread Nathan Sidwell
This patch removes the stashing of template parms that Jason noticed was unused when we fixed the 94827 ICE. pushed to trunk. nathan -- Nathan Sidwell 2020-05-04 Nathan Sidwell PR c++/94827 -- don't save parms in nested requirement * constraint.cc (tsubst_nested_requirement): TYPE

[PATCH] aarch64: fix conflicting declarations

2020-05-04 Thread Andreas Schwab
aarch64_get_extension_string_for_isa_flags is declared in "aarch64-protos.h", use that instead of re-declaring it improperly. * config/aarch64/driver-aarch64.c: Include "aarch64-protos.h". (aarch64_get_extension_string_for_isa_flags): Don't declare. ---

Re: [PATCH] Fix missed IPA-CP on by-ref argument directly passed through (PR ipa/93429)

2020-05-04 Thread Martin Jambor
Hi, On Mon, Jan 27 2020, Feng Xue OS wrote: > Current IPA does not propagate aggregate constant for by-ref argument > if it is simple pass-through of caller parameter. Here is an example, > >f1 (int *p) >{ > ... = *p; > ... >} > >f2 (int *p) >{ > *p = 2; >

Re: [PATCH][AArch64] Use __getauxval instead of getauxval in LSE detection code in libgcc

2020-05-04 Thread Szabolcs Nagy
The 05/04/2020 11:34, Kyrylo Tkachov wrote: > > -Original Message- > > From: Florian Weimer > > Sent: 04 May 2020 11:40 > > To: Kyrylo Tkachov > > Cc: gcc-patches@gcc.gnu.org; Joseph Myers > > Subject: Re: [PATCH][AArch64] Use __getauxval instead of getauxval in LSE > > detection code

[PATCH] gcc/config/rs6000: fix long double builtins for AIX

2020-05-04 Thread CHIGOT, CLEMENT via Gcc-patches
Descritpion: * This patch fixes the explicit call made by some C long double builtins when long doubles are on 64bit for AIX. Tests: * AIX 7.2, 7.1, 6.1: Build/Tests: OK Changelog: 2020-04-27 Clément Chigot gcc-10-gcc-config-rs6000-fix-long-double-builtins-for-AIX-V3.patch Description:

Re: [PATCH] c-attribs.c: Fix use of uninitialized variable nunits

2020-05-04 Thread Stefan Schulze Frielinghaus via Gcc-patches
On Mon, May 04, 2020 at 03:19:02PM +0200, Richard Biener wrote: > On Mon, May 4, 2020 at 1:44 PM Stefan Schulze Frielinghaus > wrote: > > > > On Tue, Apr 28, 2020 at 08:27:12PM +0200, Stefan Schulze Frielinghaus wrote: > > > On Tue, Apr 28, 2020 at 11:33:31AM +0200, Richard Biener wrote: > > > >

[committed] libstdc++: Make pmr::synchronized_pool_resource work without libpthread (PR 94936)

2020-05-04 Thread Jonathan Wakely via Gcc-patches
I implicitly assumed that programs using pmr::synchronized_pool_resource would also be using multiple threads, and so the weak symbols in gthr-posix.h would be resolved by linking to libpthread. If that isn't true then it crashes when trying to use pthread_key_create. This commit makes the pool

Re: [PATCH] build: Fix 32-bit TLS detection with 64-bit-default gas on Solaris/x86

2020-05-04 Thread Rainer Orth
Hi Jakub, > On Mon, May 04, 2020 at 03:19:30PM +0200, Rainer Orth wrote: >> I'm going to commit this to master shortly. Ok for the gcc-10 branch, >> too, given that it fixes a bootstrap failure? > > Ok, but see below. >> diff --git a/gcc/configure.ac b/gcc/configure.ac >> --- a/gcc/configure.ac

Re: [PATCH] build: Fix 32-bit TLS detection with 64-bit-default gas on Solaris/x86

2020-05-04 Thread Jakub Jelinek via Gcc-patches
On Mon, May 04, 2020 at 03:19:30PM +0200, Rainer Orth wrote: > I'm going to commit this to master shortly. Ok for the gcc-10 branch, > too, given that it fixes a bootstrap failure? Ok, but see below. > 2020-05-02 Rainer Orth > > * configure.ac : Add --32 to tls_as_opt on Solaris. >

[PATCH] build: Fix 32-bit TLS detection with 64-bit-default gas on Solaris/x86

2020-05-04 Thread Rainer Orth
I've recently tested i386-pc-solaris2.11 bootstrap on Solaris 11/x86 with only the bundled tools (using /usr/gnu/bin/as from binutils 2.30 in this case). It failed compiling libgo/runtime/proc.c, creating invalid assembly: proc.s: Assembler messages: proc.s:2092: Error: junk at end of line,

Re: [PATCH] c-attribs.c: Fix use of uninitialized variable nunits

2020-05-04 Thread Richard Biener via Gcc-patches
On Mon, May 4, 2020 at 1:44 PM Stefan Schulze Frielinghaus wrote: > > On Tue, Apr 28, 2020 at 08:27:12PM +0200, Stefan Schulze Frielinghaus wrote: > > On Tue, Apr 28, 2020 at 11:33:31AM +0200, Richard Biener wrote: > > > On Tue, Apr 28, 2020 at 10:03 AM Stefan Schulze Frielinghaus via > > >

Re: [PATCH] internal-fn: Avoid dropping the lhs of some calls [PR94941]

2020-05-04 Thread Jakub Jelinek via Gcc-patches
On Mon, May 04, 2020 at 03:03:01PM +0200, Richard Biener wrote: > We seem to have inconsistent use of rtx_equal_p vs. pointer comparison > of target and ops[0].value - is that because some are eventually MEMs > and some (targets?) are always registers? Of course I prefer the > cheaper !=

Re: [PATCH] internal-fn: Avoid dropping the lhs of some calls [PR94941]

2020-05-04 Thread Richard Biener via Gcc-patches
On Mon, May 4, 2020 at 2:55 PM Richard Sandiford wrote: > > create_output_operand coerces an output operand to the insn's > predicates, using a suggested rtx location if convenient. > But if that rtx location is actually required rather than > optional, the builder of the insn has to emit a move

[PATCH] internal-fn: Avoid dropping the lhs of some calls [PR94941]

2020-05-04 Thread Richard Sandiford
create_output_operand coerces an output operand to the insn's predicates, using a suggested rtx location if convenient. But if that rtx location is actually required rather than optional, the builder of the insn has to emit a move afterwards. (We could instead add a new interface that does this

[PATCH] tree-optimization/93891 - improve same-store disambiguation

2020-05-04 Thread Richard Biener
This doesn't fix the testcase at hand but is a first step. Bootstrap / regtest ongoing on x86_64-unknown-linux-gnu. We need a reference to assess alignment, fall back to the original reference tree if available. 2020-02-16 Richard Biener PR tree-optimization/93891 *

[PATCH] Adjust integer <-> pointer conversion IL checking

2020-05-04 Thread Richard Biener
This patch sits in my trees for quite some years and I always forget to push it - it usually gets triggered by weird targets (PSImode pointers/sizetype) which run into GIMPLE IL checking asserts for pointer -> integer conversions and the "sizetype" special-case not triggering. That special-case

[committed] i386: Use SHR to compare with large power-of-two constants [PR94650]

2020-05-04 Thread Uros Bizjak via Gcc-patches
Convert unsigned compares where m >= LARGE_POWER_OF_TWO and LARGE_POWER_OF_TWO represent an immediate where bit 33+ is set to use a SHR instruction and compare the result to 0. This avoids loading a large immediate with MOVABS insn. movabsq $1099511627775, %rax cmpq

Re: [PATCH] c-attribs.c: Fix use of uninitialized variable nunits

2020-05-04 Thread Stefan Schulze Frielinghaus via Gcc-patches
On Tue, Apr 28, 2020 at 08:27:12PM +0200, Stefan Schulze Frielinghaus wrote: > On Tue, Apr 28, 2020 at 11:33:31AM +0200, Richard Biener wrote: > > On Tue, Apr 28, 2020 at 10:03 AM Stefan Schulze Frielinghaus via > > Gcc-patches wrote: > > > > > > In function handle_vector_size_attribute local

RE: [PATCH][AArch64] Use __getauxval instead of getauxval in LSE detection code in libgcc

2020-05-04 Thread Kyrylo Tkachov
> -Original Message- > From: Florian Weimer > Sent: 04 May 2020 11:40 > To: Kyrylo Tkachov > Cc: gcc-patches@gcc.gnu.org; Joseph Myers > Subject: Re: [PATCH][AArch64] Use __getauxval instead of getauxval in LSE > detection code in libgcc > > * Kyrylo Tkachov: > > > This patch fixes

Re: arm: Fix vfp_operand_register for VFP HI regs

2020-05-04 Thread Christophe Lyon via Gcc-patches
On Fri, 1 May 2020 at 12:57, Kyrylo Tkachov wrote: > > > > > -Original Message- > > From: Christophe Lyon > > Sent: 30 April 2020 09:51 > > To: Kyrylo Tkachov > > Cc: gcc-patches@gcc.gnu.org > > Subject: Re: arm: Fix vfp_operand_register for VFP HI regs > > > > On Wed, 29 Apr 2020 at

[PATCH] tree-optimization/39612 - avoid issueing loads in SM when possible

2020-05-04 Thread Richard Biener
Currently store-motion emits a load of the value in the loop preheader even when the original loop does not contain any read of the reference. This avoids doing this. In the conditional store-motion case we need to mark the sunk stores with no-warning since the control dependence is too tricky

Re: [PATCH][AArch64] Use __getauxval instead of getauxval in LSE detection code in libgcc

2020-05-04 Thread Florian Weimer via Gcc-patches
* Kyrylo Tkachov: > This patch fixes > https://gcc.gnu.org/pipermail/gcc-patches/2020-May/545004.html by using > __getauxval in the libgcc code. > Bootstrap and testing on aarch64-none-linux-gnu shows no problem. > From what I could gather from my glibc testing the elf/check-localplt test >

[PATCH][AArch64] Use __getauxval instead of getauxval in LSE detection code in libgcc

2020-05-04 Thread Kyrylo Tkachov
Hi all, This patch fixes https://gcc.gnu.org/pipermail/gcc-patches/2020-May/545004.html by using __getauxval in the libgcc code. Bootstrap and testing on aarch64-none-linux-gnu shows no problem. >From what I could gather from my glibc testing the elf/check-localplt test >passes. Joseph,

Re: [PATCH] testsuite:analyzer: Fix header include for FreeBSD

2020-05-04 Thread Kamil Rytarowski
On 04.05.2020 12:04, Jakub Jelinek wrote: > On Mon, May 04, 2020 at 11:49:27AM +0200, Kamil Rytarowski wrote: Please include in your patch "|| defined(__NetBSD__)". >>> >>> is this ok for you? >>> >>> This is one reason why I'd prefer #define alloca __builtin_alloca and >>> leave the include

Re: [PATCH] testsuite:analyzer: Fix header include for FreeBSD

2020-05-04 Thread Jakub Jelinek via Gcc-patches
On Mon, May 04, 2020 at 11:49:27AM +0200, Kamil Rytarowski wrote: > >> Please include in your patch "|| defined(__NetBSD__)". > > > > is this ok for you? > > > > This is one reason why I'd prefer #define alloca __builtin_alloca and > > leave the include away. > > > > Andreas > > It looks

Re: [PATCH] testsuite:analyzer: Fix header include for FreeBSD

2020-05-04 Thread Kamil Rytarowski
On 03.05.2020 22:31, Andreas Tobler wrote: > On 03.05.20 18:27, Kamil Rytarowski wrote: >> On 01.05.2020 21:49, Andreas Tobler wrote: >>> Hi all, >>> >>> FreeBSD does not have the alloca.h header. Do not include it in the test >>> cases which do include alloca.h. >>> >>> There are two versions of

RE: Should ARMv8-A generic tuning default to -moutline-atomics

2020-05-04 Thread Kyrylo Tkachov
Hi all, > -Original Message- > From: Florian Weimer > Sent: 04 May 2020 09:35 > To: Joseph Myers > Cc: Kyrylo Tkachov ; Andrew Pinski > ; gcc-patches@gcc.gnu.org; nmeye...@amzn.com > Subject: Re: Should ARMv8-A generic tuning default to -moutline-atomics > > * Joseph Myers: > > > I

Re: Should ARMv8-A generic tuning default to -moutline-atomics

2020-05-04 Thread Florian Weimer via Gcc-patches
* Joseph Myers: > I think this change is what introduced a glibc testsuite regression in the > localplt test. > > https://sourceware.org/pipermail/libc-testresults/2020q2/006097.html > > Contents of elf/check-localplt.out: > > Extra PLT reference: libc.so: getauxval > > A reference to getauxval

Re: [PATCH] var-tracking.c: Fix possible use of uninitialized variable pre

2020-05-04 Thread Andreas Krebbel via Gcc-patches
On 04.05.20 09:27, Richard Biener wrote: > On Mon, May 4, 2020 at 8:14 AM Andreas Krebbel wrote: >> >> On 30.04.20 18:33, Richard Biener wrote: >>> On Thu, Apr 30, 2020 at 5:14 PM Andreas Krebbel >>> wrote: On 30.04.20 08:25, Richard Biener via Gcc-patches wrote: > On Wed, Apr 29,

Re: [PATCH] fix Ada bootstrap on Cygwin64 (PR bootstrap/94918)

2020-05-04 Thread Arnaud Charlet
> > If there is an issue, it would be in "s-oscons-tmplt.c" which is supposed > > to define Serial_Port_Descriptor as: > > > >subtype Serial_Port_Descriptor is System.Win32.HANDLE; > > > > See: > > > > #if defined (__MINGW32__) || defined (__CYGWIN__) > > # define TARGET_OS "Windows" > > #

Re: [PATCH] var-tracking.c: Fix possible use of uninitialized variable pre

2020-05-04 Thread Richard Biener via Gcc-patches
On Mon, May 4, 2020 at 8:14 AM Andreas Krebbel wrote: > > On 30.04.20 18:33, Richard Biener wrote: > > On Thu, Apr 30, 2020 at 5:14 PM Andreas Krebbel > > wrote: > >> > >> On 30.04.20 08:25, Richard Biener via Gcc-patches wrote: > >>> On Wed, Apr 29, 2020 at 5:56 PM Jeff Law wrote: > >

Ping: [PATCH] wwwdocs: Add D front-end section for GCC 10 changes

2020-05-04 Thread Iain Buclaw via Gcc-patches
Ping Another thing that forgot to mention was added __traits(getLinkage) support for class/struct types. Will include that too. Iain On 28 April 2020 01:12:19 CEST, Iain Buclaw wrote: >Hi, > >This patch adds a section on the D front-end about all changes that have >gone into the GCC 10

Re: [PATCH] var-tracking.c: Fix possible use of uninitialized variable pre

2020-05-04 Thread Andreas Krebbel via Gcc-patches
On 30.04.20 18:33, Richard Biener wrote: > On Thu, Apr 30, 2020 at 5:14 PM Andreas Krebbel wrote: >> >> On 30.04.20 08:25, Richard Biener via Gcc-patches wrote: >>> On Wed, Apr 29, 2020 at 5:56 PM Jeff Law wrote: On Tue, 2020-04-28 at 11:44 +0200, Richard Biener via Gcc-patches wrote: