Re: [PATCH] Add -fsanitize=shift{,-base,-exponent} support

On Tue, 8 Nov 2016, Jakub Jelinek wrote: > Hi! > > In the libsanitizer merge Maxim posted I've noticed that llvm split > the -fsanitize=shift option into suboption (at least one really weirdly > named, I have never seen the shift count being called exponent), this > patch implements the same with

Re: [PATCH] A special predicate for type size equality

On Mon, 7 Nov 2016, Martin Jambor wrote: > Hi, > > this has been in my TODO list for at least two years, probably longer, > although I do no longer remember why I added it there. The idea is to > introduce a special wrapper around operands_equal_p for TYPE_SIZE > comparisons, which would try sim

Re: [match.pd] Fix for PR35691

On Mon, 7 Nov 2016, Prathamesh Kulkarni wrote: > On 7 November 2016 at 23:06, Prathamesh Kulkarni > wrote: > > On 7 November 2016 at 15:43, Richard Biener wrote: > >> On Fri, 4 Nov 2016, Prathamesh Kulkarni wrote: > >> > >>> On 4 November 2016 at 13:41, Richard Biener wrote: > >>> > On Thu, 3 N

[patch, avr] Add flash size to device info and make wrap around default (was: Re: [patch, avr] Make pmem-wrap-around option as default)

On Thursday 03 November 2016 06:19 PM, Georg-Johann Lay wrote: On 03.11.2016 08:58, Pitchumani Sivanupandi wrote: Most of the AVR's 8k memorydevices have only rjmp instruction, not jmp. So, it is important to wrap around jump destination to check if it can reach backwards. Currently link spec

[PATCH] Fix regex_iterator end() state and operator==()

This fixes libstdc++/78236. I'm surprised that this bug was not revealed until now :P. Bootstrapped and tested under x86_64-linux-gnu. I'm happy with however many backports. -- Regards, Tim Shen commit 8aee66b743b5b0ef09cbc9587ebbacf6665ba0cb Author: Tim Shen Date: Mon Nov 7 21:50:49 2016 -0

[PATCH] Add -fsanitize=shift{,-base,-exponent} support

Hi! In the libsanitizer merge Maxim posted I've noticed that llvm split the -fsanitize=shift option into suboption (at least one really weirdly named, I have never seen the shift count being called exponent), this patch implements the same with the same option names for gcc. The interesting part i

[PATCH] Fix PR71727

Hi, Please find attached the patch that fixes PR71727. Please review the patch and let me know if its okay? Regression tested on Aarch64 with no regressions. Thanks, Naveen 2016-11-08 Naveen H.S * config/aarch64/aarch64.c (aarch64_builtin_support_vector_misalignment): New.

Re: [PATCH fix PR71767 2/4 : Darwin configury] Arrange for ld64 to be detected as Darwin's linker

> On 7 Nov 2016, at 13:53, Mike Stump wrote: > > On Nov 7, 2016, at 10:40 AM, Jeff Law wrote: >> >> On 11/07/2016 10:48 AM, Mike Stump wrote: >>> On Nov 6, 2016, at 11:39 AM, Iain Sandoe wrote: This is an initial patch in a series that converts Darwin's configury to detect ld64 fea

Re: [PATCH fix PR71767 2/4 : Darwin configury] Arrange for ld64 to be detected as Darwin's linker

> On 7 Nov 2016, at 13:53, Mike Stump wrote: > > On Nov 7, 2016, at 10:40 AM, Jeff Law wrote: >> >> On 11/07/2016 10:48 AM, Mike Stump wrote: >>> On Nov 6, 2016, at 11:39 AM, Iain Sandoe wrote: This is an initial patch in a series that converts Darwin's configury to detect ld64 fea

Re: [PATCH] have __builtin_object_size handle POINTER_PLUS with non-const offset (pr 77608)

It's taken me longer than I expected to finally get back to this project. Sorry about the delay. https://gcc.gnu.org/ml/gcc-patches/2016-09/msg01110.html Attached is an updated patch with this enhancement and reflecting you previous comment. Besides running the GCC test suite I tested the pa

[ssa-coalesce] Rename register_ssa_partition

Hi, In tree-ssa-coalesce, register_ssa_partition ) and register_ssa_partition_check have lost their meaning over various commits and now just verifies that ssa_var is indeed a SSA_NAME and not a virtual_operand_p. It is confusing when one look at if for the fist time and would expect more whil

Re: [PATCH fix PR71767 2/4 : Darwin configury] Arrange for ld64 to be detected as Darwin's linker

> On 7 Nov 2016, at 13:56, Mike Stump wrote: > > On Nov 7, 2016, at 9:59 AM, Iain Sandoe wrote: >> >>> On 7 Nov 2016, at 09:51, Mike Stump wrote: >>> >>> [ possible dup ] >>> Begin forwarded message: From: Mike Stump Subject: Re: [PATCH fix PR71767 2/4 : Darwin configu

Re: [PATCH] [ARC] define SIZE_TYPE and PTRDIFF_TYPE correctly

On 11/03/2016 03:57 AM, Claudiu Zissulescu wrote: > Hi Vineet, > > Thank you for your contribution. > >> gcc/ >> 2016-10-28 Vineet Gupta >> >> * config/arc/arc.h (SIZE_TYPE): define as unsigned int. >> * (PTRDIFF_TYPE): define as int. >> > Approved and committed. However, the entry

[openacc] add support for common block data

This patch adds support for variables inside common blocks in OpenACC data clauses. The fortran FE changes are fairly straightforward. gfc_match_omp_variable_list already has support for common block data, so all I had to do was teach gfc_match_omp_map_clause to accept common common block arguments

Re: RFA (libstdc++): C++ PATCH to implement C++17 noexcept in type system

On 7 November 2016 at 22:49, Jason Merrill wrote: > Tested x86_64-pc-linux-gnu. Are the libstdc++ changes OK for trunk? Yes, I like the approach, thanks.

RFA (libstdc++): C++ PATCH to implement C++17 noexcept in type system

This patch implements P0012R1: "Make exception-specifications be part of the type system, version 5". Some of the conversion logic was already there from the transactional memory TS implementation, and indeed the semantics of the noexcept function pointer conversion was modeled on that wording, bu

Re: [PATCH] Fix PR78229

On November 7, 2016 9:22:38 PM GMT+01:00, Jakub Jelinek wrote: >On Mon, Nov 07, 2016 at 01:20:20PM +0100, Richard Biener wrote: >> Bootstrapped and tested on x86_64-unknown-linux-gnu, applied to trunk >> and branch. >> >> Richard. >> >> 2016-11-07 Richard Biener >> >> PR target/78229 >

Re: [PATCH fix PR71767 2/4 : Darwin configury] Arrange for ld64 to be detected as Darwin's linker

On Nov 7, 2016, at 9:59 AM, Iain Sandoe wrote: > >> On 7 Nov 2016, at 09:51, Mike Stump wrote: >> >> [ possible dup ] >> >>> Begin forwarded message: >>> >>> From: Mike Stump >>> Subject: Re: [PATCH fix PR71767 2/4 : Darwin configury] Arrange for ld64 to >>> be detected as Darwin's linker >

Re: [PATCH fix PR71767 2/4 : Darwin configury] Arrange for ld64 to be detected as Darwin's linker

On Nov 7, 2016, at 10:40 AM, Jeff Law wrote: > > On 11/07/2016 10:48 AM, Mike Stump wrote: >> On Nov 6, 2016, at 11:39 AM, Iain Sandoe wrote: >>> This is an initial patch in a series that converts Darwin's configury to >>> detect ld64 features, rather than the current process of hard-coding the

Re: [PATCH, rs6000] Modify include paths in config.gcc for Advance Toolchain builds

On 11/7/16 3:11 PM, Segher Boessenkool wrote: On Mon, Nov 07, 2016 at 01:20:29PM -0600, Peter Bergner wrote: Is this ok for trunk and the GCC 6 branch? * config.gcc (powerpc*-*-*, rs6000*-*-*): Remove setting of INCLUDE_EXTRA_SPEC for Advance Toolchain builds. Okay. Thanks,

Re: [GCC][PATCH] Fix ada compile error on Windows x86_64 (committed as r241907 under the obvious rule)

> The changes in r240999 re-arranged includes and > left out signal.h for Windows x86 builds. > > This breaks the build and prevents GCC builds from > completing with messages such as: > > adaint.c:3317:19: error: 'SIGINT' undeclared (first use in this function); > did you mean 'SAIT'? > > e

Re: [PATCH, rs6000] Modify include paths in config.gcc for Advance Toolchain builds

On Mon, Nov 07, 2016 at 01:20:29PM -0600, Peter Bergner wrote: > Gabriel and I have been tracking down an include path issue for GCC 6 > Advance Toolchain builds (ie, --with-advance-toolchain=...). The solution > that fixes the problem for us is to configure with --with-local-prefix=... > and remo

Re: [PATCH] rs6000: Do swdiv at expand time

On Mon, Nov 07, 2016 at 08:26:01AM -0500, David Edelsohn wrote: > On Mon, Nov 7, 2016 at 4:32 AM, Segher Boessenkool > wrote: > > We transform floating point divide instructions to a faster series of > > simple instructions, "swdiv". Currently we do not do that until the > > first splitter pass,

[gomp4] remove GOVD_USE_DEVPTR

It looks like gomp-4_0-branch was using the GOVD_USE_DEVPTR attribute for deviceptr data mappings to implicitly set the implicit data clause for deviceptr variables to present inside OpenACC offloading regions that are nested inside an ACC DATA region. This isn't necessary anymore because the gimpl

Re: [PATCH] Fix PR78189

Hi Richard, On 7 November 2016 at 09:01, Richard Biener wrote: > > The following fixes an oversight when computing alignment in the > vectorizer. > > Bootstrapped and tested on x86_64-unknown-linux-gnu, applied to trunk. > > Richard. > > 2016-11-07 Richard Biener > > PR tree-optimizat

[PATCH] DejaGNU support for AIX visibility

Adding visibility support for AIX to GCC requires the DejaGNU testsuite to know about XCOFF "hidden". DejaGNU labels XCOFF as "coff". The appended patch adds a regex to the DejaGNU hidden-scan-for procedure to recognize AIX XCOFF hidden pseudo-ops in assembly language. Committed. - David Inde

[gomp4] backport nvptx_exec cleanups

gomp-4_0-branch already contains the default nvptx runtime enhancements that was recently applied to trunk . However, I made some tweaks in trunk involving both pthread locking and formatting changes, which this patch backports. I've applie

Re: [PATCH] Do not simplify "(and (reg) (const bit))" to if_then_else.

On 10/31/2016 08:56 PM, Dominik Vogt wrote: combine_simplify_rtx() tries to replace rtx expressions with just two possible values with an experession that uses if_then_else: (if_then_else (condition) (value1) (value2)) If the original expression is e.g. (and (reg) (const_int 2)) I'm not

Re: [PATCH] Fix PR78229

On Mon, Nov 07, 2016 at 01:20:20PM +0100, Richard Biener wrote: > Bootstrapped and tested on x86_64-unknown-linux-gnu, applied to trunk > and branch. > > Richard. > > 2016-11-07 Richard Biener > > PR target/78229 > * config/i386/i386.c (ix86_gimple_fold_builtin): Do not adjust >

[patch, committed, fortran] Final bug-fixing part of PR 78226

OK, this is it (I hope). Finding the different parts where location information was missing is now complete (I hope), at least the testsuite passes with the addition of the warnings. I had a bit more time than expected today, so I split this up the way I did. I will submit a patch soon to alway

[PATCH] print_rtx: implement support for reuse IDs (v2)

On Tue, 2016-10-25 at 14:47 +0200, Bernd Schmidt wrote: > On 10/21/2016 10:27 PM, David Malcolm wrote: > > Thanks. I attemped to use those fields of recog_data, but it > > doesn't > > seem to be exactly what's needed here. > > Yeah, I may have been confused. I'm not sure that just looking at > SC

C++ PATCH to announce template instantiations if not -quiet

It occurred to me that a simple trace of template instantiations would fit simply into the stream of function declarations that announce_function prints when -quiet is not specified to the compiler. Tested x86_64-pc-linux-gnu, applying to trunk. commit ae7b4a929fbd05de433451a1d92794d962366646 Auth

[PATCH, rs6000] Modify include paths in config.gcc for Advance Toolchain builds

Gabriel and I have been tracking down an include path issue for GCC 6 Advance Toolchain builds (ie, --with-advance-toolchain=...). The solution that fixes the problem for us is to configure with --with-local-prefix=... and removing the following hunk from config.gcc. Gabriel has confirmed this fi

Re: [PATCH fix PR71767 2/4 : Darwin configury] Arrange for ld64 to be detected as Darwin's linker

Iain, It certainly looks like you dropped a file here. The proposed ChangeLog shows... * config.in: Likewise. but the previously proposed hunk from... diff --git a/gcc/config.in b/gcc/config.in index a736de3..a7ff3ee 100644 --- a/gcc/config.in +++ b/gcc/config.in @@ -1934,6 +1934,18 @@ #en

Re: [PATCH v2] aarch64: Add split-stack initial support

On 14/10/2016 15:59, Wilco Dijkstra wrote: > Hi, > Thanks for the thoughtful review and sorry for late response. >> Split-stack prologue on function entry is as follow (this goes before the >> usual function prologue): > >> mrsx9, tpidr_el0 >> movx10, - > > As Jiong alread

[PATCH] A special predicate for type size equality

Hi, this has been in my TODO list for at least two years, probably longer, although I do no longer remember why I added it there. The idea is to introduce a special wrapper around operands_equal_p for TYPE_SIZE comparisons, which would try simple pointer equality before calling more complex opera

Re: [PATCH fix PR71767 2/4 : Darwin configury] Arrange for ld64 to be detected as Darwin's linker

On 11/07/2016 10:48 AM, Mike Stump wrote: On Nov 6, 2016, at 11:39 AM, Iain Sandoe wrote: This is an initial patch in a series that converts Darwin's configury to detect ld64 features, rather than the current process of hard-coding them on target system version. So, I really do hate to ask,

[hsa-branch] Remove superfluous lastprivate check

Hi, this is another simple cleanup that I forgot to commit, which just removes a lastprivate check (which hsa now can handle) at a place where it cannot ever be anyway. Committed to the hsa branch, will include it in the pile of OpenMP stuff to request to merge to trunk later this week. Thanks,

[hsa-branch] Append UID to local variable names

Hi, when looking at stuff to merge to trunk, I have found out that this patch has slipped thorough the cracks. It adds the UID to names of private symbols so that variables with the same name but different scope, particularly OpenMP re-mapped ones, do not clash. Committed to the hsa branch, will

Re: [PATCH fix PR71767 2/4 : Darwin configury] Arrange for ld64 to be detected as Darwin's linker

On Sun, 6 Nov 2016, Iain Sandoe wrote: > This adds an option --with-ld64[=version] that allows the configurer to New configure options should be documented in install.texi. -- Joseph S. Myers jos...@codesourcery.com

Re: [PATCH, Darwin] Fix PR57438 by avoiding empty function bodies and trailing labels.

On Nov 6, 2016, at 12:13 PM, Iain Sandoe wrote: > > OK for trunk? > OK for open branches? For the darwin parts, Ok. > 2016-11-06 Iain Sandoe > > PR target/57438 > * config/i386/i386.c (ix86_code_end): Note that we emitted code where > the > function might otherwise appear

Re: [PATCH, Darwin] fix for PR67710 : Update 'as' specs and inputs to handle newer assembler versions.

On Nov 6, 2016, at 12:53 PM, Iain Sandoe wrote: > OK for trunk? > OK for open branches? Ok. > 2016-11-06 Iain Sandoe > Rainer Orth > > target/PR67710 > * config.in: Regenerate > * config/darwin-driver.c (darwin_driver_init): Emit a version string > for the assemble

Re: [PATCH fix PR71767 3/4 : Darwin sections] Fix PR71767 - adjust the sections used in response to ld64 version.

On Nov 6, 2016, at 11:40 AM, Iain Sandoe wrote: > > OK for trunk? > OK for open branches? Ok. > 2016-11-06 Iain Sandoe > > PR target/71767 > * config/darwin-sections.def (picbase_thunk_section): New. > * config/darwin.c (darwin_init_sections): Set up picbase thunk section.

Re: [PATCH fix PR71767 4/4 : testsuite] Fix testsuite fallout from section and linker sym visibility changes.

On Nov 6, 2016, at 11:41 AM, Iain Sandoe wrote: > OK for trunk (after the relevant patches are applied)? > OK for open branches (likewise)? Ok. > PR target/71767 > > * g++.dg/abi/key2.C: Adjust for changed Darwin sections and > linker-visible symbols. > * g++.dg/torture/darwi

Re: [PATCH fix PR71767 1/4 : ld64 atoms] Make PIC indirections and constant labels linker-visible.

On Nov 6, 2016, at 11:37 AM, Iain Sandoe wrote: > OK for trunk? > OK for open branches? Ok. > 2016-11-06 Iain Sandoe > > PR target/71767 > * config/darwin.c (imachopic_indirection_name): Make data section > indirections > linker-visible. > * config/darwin.h (ASM_GENE

Re: [PATCH fix PR71767 2/4 : Darwin configury] Arrange for ld64 to be detected as Darwin's linker

> On 7 Nov 2016, at 09:51, Mike Stump wrote: > > [ possible dup ] > >> Begin forwarded message: >> >> From: Mike Stump >> Subject: Re: [PATCH fix PR71767 2/4 : Darwin configury] Arrange for ld64 to >> be detected as Darwin's linker >> Date: November 7, 2016 at 9:48:53 AM PST >> To: Iain Sand

Re: [PATCH][GCC/TESTSUITE] Make test for traditional-cpp depend on

On Nov 1, 2016, at 8:46 AM, Tamar Christina wrote: > > A glibc update recently broke this test by adding a CPP > macro that uses the ## string function which traditional-cpp > does not support. > The change in glibc that made the test fail is from > 6962682ffe5e5f0373047a0b894fee7a774be254. > >

Re: [PATCH fix PR71767 2/4 : Darwin configury] Arrange for ld64 to be detected as Darwin's linker

On Nov 6, 2016, at 11:39 AM, Iain Sandoe wrote: > This is an initial patch in a series that converts Darwin's configury to > detect ld64 features, rather than the current process of hard-coding them on > target system version. So, I really do hate to ask, but does this have to be a config optio

Re: [match.pd] Fix for PR35691

On 7 November 2016 at 23:06, Prathamesh Kulkarni wrote: > On 7 November 2016 at 15:43, Richard Biener wrote: >> On Fri, 4 Nov 2016, Prathamesh Kulkarni wrote: >> >>> On 4 November 2016 at 13:41, Richard Biener wrote: >>> > On Thu, 3 Nov 2016, Marc Glisse wrote: >>> > >>> >> On Thu, 3 Nov 2016, R

Re: [match.pd] Fix for PR35691

On 7 November 2016 at 15:43, Richard Biener wrote: > On Fri, 4 Nov 2016, Prathamesh Kulkarni wrote: > >> On 4 November 2016 at 13:41, Richard Biener wrote: >> > On Thu, 3 Nov 2016, Marc Glisse wrote: >> > >> >> On Thu, 3 Nov 2016, Richard Biener wrote: >> >> >> >> > > > > The transform would also

Re: [Patch, rtl] PR middle-end/78016, keep REG_NOTE order during insn copy

On 07/11/16 17:04, Bernd Schmidt wrote: On 11/03/2016 03:00 PM, Eric Botcazou wrote: FWIW here's a more complete version of my patch which I'm currently testing. Let me know if you think it's at least a good enough intermediate step to be installed. It is, thanks. Testing showed the same i

Re: [Patch, rtl] PR middle-end/78016, keep REG_NOTE order during insn copy

On 11/03/2016 03:00 PM, Eric Botcazou wrote: FWIW here's a more complete version of my patch which I'm currently testing. Let me know if you think it's at least a good enough intermediate step to be installed. It is, thanks. Testing showed the same issue as Jiong found, so I've committed it w

Re: [PATCH 0/2] strncmp builtin expansion improvement

On Mon, 2016-11-07 at 15:26 +0100, Richard Biener wrote: > Your patchset doesn't contain a testcase so I really wonder which > case > we know the string length but it is not constant. > > Yes, there's COND_EXPR handling in c_strlen but that should be mostly > dead code -- the real code should be u

Re: [PATCH] Make direct emission of time profiler counter

On 7 November 2016 at 09:58, Martin Liška wrote: > On 11/05/2016 09:38 AM, Jan Hubicka wrote: >> Looks OK if it passes. >> >> Honza > > Thanks, fixed on trunk as r241894. > Martin Thanks, this fixed the problems I reported. Christophe

[PATCH,testsuite] MIPS: Upgrade to MIPS IV if using (HAS_MOVN) with MIPS III.

Hi, The (HAS_MOVN) option should cause an upgrade to MIPS IV if the target is pre-MIPS IV. However, the upgrade condition checks for "$isa < 3", which means that we won't upgrade if we're targeting MIPS III. This results in failures for the movcc-{1,2,3}.c and branch-cost-2.c tests when the targe

Re: Fix build of jit (was Re: [PATCH, RFC] Introduce -fsanitize=use-after-scope (v3))

On Mon, Nov 07, 2016 at 11:07:13AM -0500, David Malcolm wrote: > The patch (r241896) introduced an error in the build of the jit: > > ../../src/gcc/jit/jit-builtins.c:62:1: error: invalid conversion from > ‘int’ to ‘gcc::jit::built_in_attribute’ [-fpermissive] > }; > ^ > > which seems to be due

Re: [PATCH] Fix DSE not to consider calls as reads from function's body (PR target/77834)

On 11/04/2016 05:35 PM, Jakub Jelinek wrote: 2016-11-04 Jakub Jelinek PR target/77834 * dse.c (dse_step5): Call scan_reads even if just insn_info->frame_read. Improve and fix dump file messages. Sounds reasonable, and I checked and it seems not to change code gene

Fix build of jit (was Re: [PATCH, RFC] Introduce -fsanitize=use-after-scope (v3))

On Mon, 2016-11-07 at 11:03 +0100, Martin Liška wrote: > Hello. > > After discussion with Jakub, I'm resending new version of the patch, > where I changed following: > 1) gimplify_ctxp->live_switch_vars is used to track variables > introduced in switch_expr. Every time >a case_label_expr is se

[PING, PATCH] Do not simplify "(and (reg) (const bit))" to if_then_else.

Ping. https://gcc.gnu.org/ml/gcc-patches/2016-10/msg02525.html On Mon, Oct 31, 2016 at 08:56:10PM +0100, Dominik Vogt wrote: > The attached patch does a little change in > combine.c:combine_simplify_rtx() to prevent a "simplification" > where the rtl code gets more complex in reality. The comple

Re: [patch, fortran, committed] Fill in some more locations

Am 07.11.2016 um 16:25 schrieb Thomas Koenig: PR fortran/78826 ... should have been PR 78226.

[patch, fortran, committed] Fill in some more locations

Hello world, I have committed the little patchlet below as obvious, after regression-testing. Regards Thomas 2016-11-07 Thomas Koenig PR fortran/78826 * match.c (gfc_match_select_type): Add where for expr1. * resolve.c (resolev_select_type): Add where for e

Re: [PATCH] rtx_writer: avoid printing trailing default values

On Fri, 2016-11-04 at 20:40 +0100, Bernd Schmidt wrote: > On 11/04/2016 08:25 PM, David Malcolm wrote: > > > return m_compact; > > Ok with this one plus a comment. > Thanks. Using m_compact required turning the static function into a (private) member function. For reference, here's

Re: Simplify X / X, 0 / X and X % X

On 11/07/2016 03:02 AM, Richard Biener wrote: On Sat, Nov 5, 2016 at 3:30 AM, Jeff Law wrote: On 11/04/2016 02:07 PM, Marc Glisse wrote: Hello, since we were discussing this recently... The condition is copied from the existing 0 % X case, visible in the context of the diff. As far as I un

Re: [RFA] Fix various PPC build failures due to int-in-boolean-context code

On Fri, Oct 28, 2016 at 09:12:29AM -0600, Jeff Law wrote: > > The PPC port is stumbling over the new integer in boolean context warnings. > > In particular this code from rs6000_option_override_internal is > problematical: > > HOST_WIDE_INT flags = ((TARGET_DEFAULT) ? TARGET_DEFAULT

Re: [PATCH] Fix -O0 AVX512 comparison ICE (PR target/78227)

On Mon, Nov 7, 2016 at 2:02 PM, Jakub Jelinek wrote: > Hi! > > The following testcases ICE at -O0, because ix86_expand_sse_cmp avoid using > the passed in dest only if optimize or if there is some value overlap, but > we actually need to do that also if we have a maskcmp where we want to use > a d

[PATCH] Avoid peeling for gaps if accesses are aligned

Currently we force peeling for gaps whenever element overrun can occur but for aligned accesses we know that the loads won't trap and thus we can avoid this. Bootstrap and regtest running on x86_64-unknown-linux-gnu (I expect some testsuite fallout here so didn't bother to invent a new testcase).

Re: [PATCH 0/2] strncmp builtin expansion improvement

On Sun, Nov 6, 2016 at 5:32 AM, Aaron Sawdey wrote: > On Fri, 2016-11-04 at 20:43 -0600, Jeff Law wrote: >> So what's the motivation here? When we don't have any constants >> then >> I'd think we'd be better off punting into the library. > > When none of the args to strncmp are constant, I'd be i

[GCC][PATCH] Fix ada compile error on Windows x86_64 (committed as r241907 under the obvious rule)

Hi all, The changes in r240999 re-arranged includes and left out signal.h for Windows x86 builds. This breaks the build and prevents GCC builds from completing with messages such as: adaint.c:3317:19: error: 'SIGINT' undeclared (first use in this function); did you mean 'SAIT'? else if (si

Re: New option -flimit-function-alignment

On 10/14/2016 08:28 PM, Bernd Schmidt wrote: On 10/12/2016 09:27 PM, Denys Vlasenko wrote: Yes, something like "if max_skip >= func_size, temporarily lower max_skip to func_size-1" (because otherwise we can create padding bigger-or-equal to the entire function in size, which is stupid - it's bet

Re: [PATCH][GCC/TESTSUITE] Make test for traditional-cpp depend on

Ping. From: gcc-patches-ow...@gcc.gnu.org on behalf of Tamar Christina Sent: Tuesday, November 1, 2016 3:46:07 PM To: GCC Patches; r...@cebitec.uni-bielefeld.de; mikest...@comcast.net Cc: nd Subject: [PATCH][GCC/TESTSUITE] Make test for traditional-cpp d

[GCC][AArch64][PATCH][Testsuite] Fix failing test vector_initialization_nostack.c

Hi all, This fixes (PR78142) by turning off scheduling for the test. r241590 is causing more registers to be used and so the SP registered happens to be picked and used. This test I believe was checking explicitly that the SP is not used if not needed. Ran regression tests on aarch64-none-linu

Re: [PATCH, GCC, wwwdocs] Document new Cortex-M23 and Cortex-M33 processors support in ARM backend

On 07/11/16 14:00, Thomas Preudhomme wrote: What about ARM maintainers? Fine with me too. Thanks, Kyrill Best regards, Thomas On 04/11/16 22:16, Gerald Pfeifer wrote: On Fri, 4 Nov 2016, Thomas Preudhomme wrote: This patch document the newly added support in GCC 7 for Cortex-M23 and Cor

Re: [PATCH, GCC, wwwdocs] Document new Cortex-M23 and Cortex-M33 processors support in ARM backend

What about ARM maintainers? Best regards, Thomas On 04/11/16 22:16, Gerald Pfeifer wrote: On Fri, 4 Nov 2016, Thomas Preudhomme wrote: This patch document the newly added support in GCC 7 for Cortex-M23 and Cortex-M33 processors [1][2]. : Is this ok for ? Surely so for me. Gerald

[PATCH] Fix PR78224

The following fixes an ICE with call cdce where it fails to handle PHIs in the fallthru destination of a call with EH. My simple fix is to simply split the fallthru edge if the dest may contain PHI nodes. This may also remove the need to free dominance info (hope there's a testcase for that -- I

[ARM][GCC][PATCHv2 2/3] Add missing Poly64_t intrinsics to GCC

Hi all, This patch (2 of 3) adds the following NEON intrinsics to the ARM back-end of GCC: * vget_lane_p64 Added new tests for these and ran regression tests on aarch64-none-linux-gnu and on arm-none-linux-gnueabihf. Ok for trunk? Thanks, Tamar gcc/ 2016-11-04 Tamar Christina * co

[AArch64][ARM][GCC][PATCHv2 3/3] Add tests for missing Poly64_t intrinsics to GCC

Hi all, This patch (3 of 3) adds updates tests for the NEON intrinsics added by the previous patches: Ran regression tests on aarch64-none-linux-gnu and on arm-none-linux-gnueabihf. Ok for trunk? Thanks, Tamar gcc/testsuite/ 2016-11-04 Tamar Christina * gcc.target/aarch64/advsimd-

[AArch64][GCC][PATCHv2 1/3] Add missing Poly64_t intrinsics to GCC

Hi all, This patch (1 of 3) adds the following NEON intrinsics to the Aarch64 back-end of GCC: * vsli_n_p64 * vsliq_n_p64 * vld1_p64 * vld1q_p64 * vld1_dup_p64 * vld1q_dup_p64 * vst1_p64 * vst1q_p64 * vld2_p64 * vld3_p64 * vld4_p64 * vld2q_p64 * vld3q_p64 * vld4q_p64 * vld2_dup_p64 * vld3_d

Re: Add missing symbols for versioned namespace

On 03/11/16 21:54 +0100, François Dumont wrote: Hi I might not be the right one to propose this patch as I am not sure that I fully understand gnu-versioned-namespace.ver organization. But with it following test failures when using versioned namespace vanish: FAIL: 20_util/allocator/over

[PATCH][AArch64] Optimized implementation of search_line_fast for the CPP lexer

This patch contains an implementation of search_line_fast for the CPP lexer. It's based in part on the AArch32 (ARM) code but incorporates new instructions available in AArch64 (reduction add operations) plus some tricks for reducing the realignment overheads. We assume a page size of 4k, but tha

Re: [PATCH] Fix nonoverlapping_memrefs_p ICE (PR target/77834, take 3)

On Mon, 7 Nov 2016, Jakub Jelinek wrote: > On Fri, Nov 04, 2016 at 08:07:37PM +0100, Richard Biener wrote: > > >If/once this is in, I'm planning to test/submit a patch adding > > > /* If one decl is known to be a function or label in a function and > > > the other is some kind of data, they c

Re: [PATCH] rs6000: Do swdiv at expand time

On Mon, Nov 7, 2016 at 4:32 AM, Segher Boessenkool wrote: > We transform floating point divide instructions to a faster series of > simple instructions, "swdiv". Currently we do not do that until the > first splitter pass, which is much too late for most optimisations > that can happen on those n

[PATCH] Fix nonoverlapping_memrefs_p ICE (PR target/77834, take 3)

On Fri, Nov 04, 2016 at 08:07:37PM +0100, Richard Biener wrote: > >If/once this is in, I'm planning to test/submit a patch adding > > /* If one decl is known to be a function or label in a function and > > the other is some kind of data, they can't overlap. */ > > if ((TREE_CODE (exprx) == F

Re: [PATCH] combine lhs zero_extract fix (PR78186)

On Mon, Nov 07, 2016 at 02:00:46PM +0100, Christophe Lyon wrote: > > Confirmed. What a nasty, nasty bug, and it has been here for decades > > it seems. Could you please open a PR? > > > Sure, I've created PR78232 for this. Thanks! I have a patch btw, it's regstrapping. Not sure it is fully cor

[committed] Move 3 gcc.target/i386/*.C tests

Hi! Richard noticed 3 misplaced tests - C++ tests don't belong into gcc.target/ which tests just C. I've bootstrapped/regtested this on x86_64-linux and i686-linux and committed to trunk as obvious. 2016-11-07 Jakub Jelinek PR middle-end/71529 * gcc.target/i386/pr71529.C: Mov

Re: [patch,avr] Add new option -mabsdata.

On 07.11.2016 13:54, Georg-Johann Lay wrote: This patch adds a new command line option -mabsdata which can be ised to set attribute absdata for all data in static storage so it can be accessed by LDS and STS instructions. This is only useful for some reduced Tiny devices like ATtiny40. For othe

[PATCH] Fix -O0 AVX512 comparison ICE (PR target/78227)

Hi! The following testcases ICE at -O0, because ix86_expand_sse_cmp avoid using the passed in dest only if optimize or if there is some value overlap, but we actually need to do that also if we have a maskcmp where we want to use a different mode than dest has. Bootstrapped/regtested on x86_64-lin

Re: [PATCH] combine lhs zero_extract fix (PR78186)

On 7 November 2016 at 10:14, Segher Boessenkool wrote: > Hi Christophe, > > On Fri, Nov 04, 2016 at 02:31:28PM +0100, Christophe Lyon wrote: >> Since this commit I have noticed execution failures on "old" arm targets: >> >> gcc.dg/torture/pr48124-4.c -O1 execution test >> gcc.dg/torture/pr4

[patch,avr] Add new option -mabsdata.

This patch adds a new command line option -mabsdata which can be ised to set attribute absdata for all data in static storage so it can be accessed by LDS and STS instructions. This is only useful for some reduced Tiny devices like ATtiny40. For other reduced Tiny where all of SRAM fits LDS /

[PATCH] Fix PR78205 -- fix BB SLP "gap" handling

The following moves a overly conservative check that we do not access excess elements when vectorizing a BB to a place where we can do a better job with respect to the elements we actually use. This means that for the included testcase we are not confused by the read from c[4] but just do not vec

Re: [PATCH] Fix PR78228

On Mon, 7 Nov 2016, Jakub Jelinek wrote: > On Mon, Nov 07, 2016 at 01:17:25PM +0100, Richard Biener wrote: > > > > The following fixes phiopt to not introduce undefined behavior > > in its abs replacement code in case we negate only positive values > > in the original code. > > > > Bootstrapped

Re: [PATCH] Fix PR78228

On Mon, Nov 07, 2016 at 01:17:25PM +0100, Richard Biener wrote: > > The following fixes phiopt to not introduce undefined behavior > in its abs replacement code in case we negate only positive values > in the original code. > > Bootstrapped and tested on x86_64-unknown-linux-gnu, applied to trunk

[PATCH] Fix PR78229

Bootstrapped and tested on x86_64-unknown-linux-gnu, applied to trunk and branch. Richard. 2016-11-07 Richard Biener PR target/78229 * config/i386/i386.c (ix86_gimple_fold_builtin): Do not adjust EH info. * g++.dg/pr78229.C: New testcase. Index: gcc/config/i

[PATCH] Fix PR78218

Bootstrapped and tested on x86_64-unknown-linux-gnu, applied. Richard. 2016-11-07 Richard Biener PR tree-optimization/78218 * gimple-ssa-store-merging.c (pass_store_merging::terminate_all_aliasing_chains): Drop unused argument, fix alias check to also consider

[PATCH] Fix PR78228

The following fixes phiopt to not introduce undefined behavior in its abs replacement code in case we negate only positive values in the original code. Bootstrapped and tested on x86_64-unknown-linux-gnu, applied to trunk. Richard. 2016-11-07 Richard Biener PR tree-optimization/7822

Re: [PATCH][AArch64] Fix PR target/77822: Use tighter predicates for zero_extract patterns

On Mon, Oct 17, 2016 at 05:15:21PM +0100, Kyrill Tkachov wrote: > Hi all, > > For the attached testcase the code ends up trying to extract bits outside the > range of the normal register widths. The aarch64 patterns for ubfz and tbnz > end up accepting such operands and emitting invalid assembly >

Re: [PATCH][AArch64] Fix PR target/77822: Use tighter predicates for zero_extract patterns

Ping. Thanks, Kyrill On 31/10/16 12:10, Kyrill Tkachov wrote: Ping. Thanks, Kyrill On 24/10/16 14:12, Kyrill Tkachov wrote: On 24/10/16 12:29, Kyrill Tkachov wrote: Ping. https://gcc.gnu.org/ml/gcc-patches/2016-10/msg01321.html I just noticed my original ChangeLog entry was truncated. I

Re: Ping^6 Re: [Patch AArch64] Add floatdihf2 and floatunsdihf2 patterns

On Fri, Oct 21, 2016 at 05:31:14PM +0100, James Greenhalgh wrote: > On Wed, Oct 12, 2016 at 04:56:52PM +0100, James Greenhalgh wrote: > > On Wed, Sep 28, 2016 at 05:17:14PM +0100, James Greenhalgh wrote: > > > On Wed, Sep 21, 2016 at 10:42:03AM +0100, James Greenhalgh wrote: > > > > On Tue, Sep 13,

[RFC] Fix PR rtl-optimization/59461

It's a missed optimization of a redundant zero-extension on the SPARC, which originally comes from PR rtl-optimization/58295 for ARM. The extension is eliminated on the ARM because the load is explicitly zero-extended in RTL; on the SPARC the load is implicitly zero-extended by means of LOAD_EXT

Re: [PATCH 0/7] Libsanitizer merge from upstream r285547.

On 07/11/16 13:04, Jakub Jelinek wrote: On Mon, Nov 07, 2016 at 11:22:28AM +0300, Maxim Ostapenko wrote: Hi, this patch set performs libsanitizer merge from upstream. Patch 1 is the library merge itself. Patch 2 is the reapplied change for SPARC by David S. Miller. Patch 3 changes heuristic

Re: [PATCH][1/2] GIMPLE Frontend, C FE parts (and GIMPLE parser)

On Mon, 7 Nov 2016, Richard Biener wrote: > On Fri, 4 Nov 2016, Jakub Jelinek wrote: > > > Hi! > > > > Just 2 nits: > > > > On Fri, Oct 28, 2016 at 01:46:57PM +0200, Richard Biener wrote: > > > +/* Return a pointer to the Nth token in PARERs tokens_buf. */ > > > > PARSERs ? > > Fixed. > > >

  1   2   >