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 :

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,

[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

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: 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.

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

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

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'? > >

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

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: >> >>

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

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,

[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

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 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 >

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 > *

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

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 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

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:

[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,

[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,

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

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 >

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. >

[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

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

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 for extracting last PC from stack frame for ARM in fast unwind routine. More details can be found here

[PATCH 2/7] Libsanitizer merge from upstream r285547.

This is just reapplied patch for SPARC by David S. Miller. From 0ff8d1c408b076970c323361922c35033aaae245 Mon Sep 17 00:00:00 2001 From: Maxim Ostapenko Date: Tue, 25 Oct 2016 20:00:43 +0300 Subject: [PATCH 2/7] libsanitizer/ PR sanitizer/63958 Reapply: 2014-10-14

[PATCH 3/7] Libsanitizer merge from upstream r285547.

This patch adjusts the fix for https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61771 to extract the last PC from the stack frame if no valid FP is available for ARM. From 6dc6e4f761080cf19a161fb0e27c1fd584688f40 Mon Sep 17 00:00:00 2001 From: Maxim Ostapenko Date: Tue, 25

[PATCH 4/7] Libsanitizer merge from upstream r285547.

This is rewritten Jakub's fix for https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63888. Upstream now supports new approach for ODR violation detection: compiler emits new __odr_asan_XXX symbol for each instrumented global that indicates whether this global was already registered and the library

[PATCH 5/7] Libsanitizer merge from upstream r285547.

This patch just combines minimal necessary changes to support new libasan ABI. This patch doesn't try to implement odr indicators at compiler part, it simply pass a zero stub to runtime. The actual implementation of odr indicators goes in patch 7. From 33f6f98faa86c61b9895db0d71e0e88a9ae4fa59

[PATCH 6/7] Libsanitizer merge from upstream r285547.

This patch just adds several tests backported from upstream. From b4677ed64e7aee1af7772750e0b18ed8271f4757 Mon Sep 17 00:00:00 2001 From: Maxim Ostapenko Date: Tue, 1 Nov 2016 16:52:13 +0300 Subject: [PATCH 6/7] Backport several testcases for ASan from upstream. gcc/

[PATCH 7/7] Libsanitizer merge from upstream r285547.

This patch tries to implement odr indicators functionality at compiler side. We emit new __odr_asan_XXX symbol for each instrumented global that indicates whether this global was already registered and the library checks this indicator symbol at runtime. For some globals (e.g. static or

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

On Mon, Nov 07, 2016 at 11:22:28AM +0300, Maxim Ostapenko wrote: > 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 for extracting last PC from stack

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

On Mon, Nov 07, 2016 at 11:31:18AM +0300, Maxim Ostapenko wrote: > --- a/gcc/asan.c > +++ b/gcc/asan.c > @@ -1329,6 +1329,16 @@ asan_needs_local_alias (tree decl) >return DECL_WEAK (decl) || !targetm.binds_local_p (decl); > } > > +/* Return true if DECL, a global var, is an artificial ODR

[PATCH] Fix PR78189

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-optimization/78189 * tree-vect-data-refs.c

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

[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

[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

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

[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

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

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

[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

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

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: [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.

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

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

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

[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

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

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

[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 *

[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][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

[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

[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

[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.

[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

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:

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

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

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

[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

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

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:

[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

[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 *

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

[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

[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

[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][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

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

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

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

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

[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) ==

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

[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 --

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

[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

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

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

[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

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

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

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

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

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

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

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

On Mon, Nov 7, 2016 at 9:20 AM, Jakub Jelinek wrote: > On Mon, Nov 07, 2016 at 12:14:39PM +0300, Maxim Ostapenko wrote: >> libubsan is definitely compatible. > > Nice. > >> For libtsan we have several changes: >> >> 1) Several interceptors (34 of them) were added and

[PATCH] rs6000: Do swdiv at expand time

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 new instructions, e.g. the constant loads are not CSEd inside an

Re: [PATCH] fix a few minor nits in -Walloca documentation

On Sat, Nov 5, 2016 at 3:25 AM, Jeff Law wrote: > On 11/04/2016 06:26 PM, Martin Sebor wrote: >> >> While experimenting with -Walloca and cross-referencing the manual >> I noticed a few minor nits that I thought could stand to corrected >> and/or clarified. Attached is a patch.

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

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

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

On Fri, Nov 4, 2016 at 9: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 understand, the main case where we do not want to

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

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 for extracting last PC

Re: [PATCH, RFC] Introduce -fsanitize=use-after-scope (v3)

On Mon, Nov 07, 2016 at 11:03:11AM +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

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   >