Re: [ipa-vrp] ice in set_value_range

2016-11-09 Thread kugan
Hi Andrew, On 09/11/16 17:02, Andrew Pinski wrote: Either this patch or the patch for "Handle unary pass-through jump functions for ipa-vrp" caused a bootstrap failure on aarch64-linux-gnu. Bootstrap comparison failure! gcc/go/types.o differs gcc/fortran/class.o differs gcc/tree-ssa-live.o diffe

Re: [PATCH] Fix PR78189

2016-11-09 Thread Richard Biener
On Tue, 8 Nov 2016, Richard Biener wrote: > On Tue, 8 Nov 2016, Richard Biener wrote: > > > On Mon, 7 Nov 2016, Christophe Lyon wrote: > > > > > Hi Richard, > > > > > > > > > On 7 November 2016 at 09:01, Richard Biener wrote: > > > > > > > > The following fixes an oversight when computing ali

[PATCH] Fix PR78007

2016-11-09 Thread Richard Biener
The following implements vectorization of bswap via VEC_PERM_EXPR on the corresponding QImode vector. ARM already has backend handling via the builtin_vectorized_call hook and thus there were already testcases available. It doesn't end up working for vect-bswap16.c because we have a promoted arg

Re: [PATCH] combine: Do not call simplify from inside change_zero_ext (PR78232)

2016-11-09 Thread Eric Botcazou
> This can also happen if a splitter in the MD uses nonzero_bits (for > example). I tried to make the splitting code in combine save and restore > the i2dest reg_stat info, but that causes one of the acats tests to fail. Could you elaborate? Did you try to temporarily invalidate it, e.g. by mean

Re: [match.pd] Fix for PR35691

2016-11-09 Thread Prathamesh Kulkarni
On 8 November 2016 at 16:46, Richard Biener wrote: > On Tue, 8 Nov 2016, Prathamesh Kulkarni wrote: > >> On 8 November 2016 at 13:23, Richard Biener wrote: >> > On Mon, 7 Nov 2016, Prathamesh Kulkarni wrote: >> > >> >> On 7 November 2016 at 23:06, Prathamesh Kulkarni >> >> wrote: >> >> > On 7 No

Re: [PATCH] Fix PR78189

2016-11-09 Thread Bin.Cheng
On Tue, Nov 8, 2016 at 9:11 AM, Richard Biener wrote: > On Mon, 7 Nov 2016, Christophe Lyon wrote: > >> 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 te

Re: [PATCH] combine: Do not call simplify from inside change_zero_ext (PR78232)

2016-11-09 Thread Segher Boessenkool
On Wed, Nov 09, 2016 at 09:30:59AM +0100, Eric Botcazou wrote: > > This can also happen if a splitter in the MD uses nonzero_bits (for > > example). I tried to make the splitting code in combine save and restore > > the i2dest reg_stat info, but that causes one of the acats tests to fail. > > Cou

Re: [PATCH] debug/PR78112: remove recent duplicates for DW_TAG_subprogram attributes

2016-11-09 Thread Richard Biener
On Tue, Nov 8, 2016 at 3:24 PM, Pierre-Marie de Rodat wrote: > Hello, > > This patch comes from the discussion for PR debug/78112 > (https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78112). It seems to fix > the regression Rainer detected on x86_64-apple-darwin, although I still > wonder why remainin

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

2016-11-09 Thread Bin.Cheng
On Tue, Nov 8, 2016 at 9:11 AM, Christophe Lyon wrote: > Hi, > > On 7 November 2016 at 23:56, Jonathan Wakely wrote: >> 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. > > The new te

Re: [patch, avr] Add flash size to device info and make wrap around default

2016-11-09 Thread Pitchumani Sivanupandi
On Tuesday 08 November 2016 02:57 PM, Georg-Johann Lay wrote: On 08.11.2016 08:08, Pitchumani Sivanupandi wrote: I have updated patch to include the flash size as well. Took that info from device headers (it was fed into crt's device information note section also). The new option would render

Re: [match.pd] Fix for PR35691

2016-11-09 Thread Richard Biener
On Wed, 9 Nov 2016, Prathamesh Kulkarni wrote: > On 8 November 2016 at 16:46, Richard Biener wrote: > > On Tue, 8 Nov 2016, Prathamesh Kulkarni wrote: > > > >> On 8 November 2016 at 13:23, Richard Biener wrote: > >> > On Mon, 7 Nov 2016, Prathamesh Kulkarni wrote: > >> > > >> >> On 7 November 20

Re: [PATCH] combine: Do not call simplify from inside change_zero_ext (PR78232)

2016-11-09 Thread Eric Botcazou
> You need more than that, e.g. reg_nonzero_bits_for_combine looks at the > nonzero_bits field no matter what else. Right, this one is sticky, although you can invalidate it globally by means of nonzero_sign_valid AFAICS. > I didn't have the heart to look into this testcase breakage; the > save/

[Patch, Fortran, committed] PR 71894: [OOP] ICE in gfc_add_component_ref, at fortran/class.c:227

2016-11-09 Thread Janus Weil
Hi all, I have committed to trunk another obvious patch to fix an ICE on invalid code: https://gcc.gnu.org/viewcvs?rev=241993&root=gcc&view=rev Cheers, Janus

Re: [PATCH] S390: Fix PR/77822.

2016-11-09 Thread Dominik Vogt
On Tue, Nov 08, 2016 at 05:46:04PM +0100, Matthias Klose wrote: > On 08.11.2016 15:38, Dominik Vogt wrote: > > The attached patch fixes PR/77822 on s390/s390x dor gcc-6 *only*. > > See https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77822 > > > > Bootstrapped and regression tested on s390 and s390x b

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

2016-11-09 Thread Richard Biener
On Mon, 7 Nov 2016, Richard Biener wrote: > 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

Re: [Patch, Fortran, committed] PR 71894: [OOP] ICE in gfc_add_component_ref, at fortran/class.c:227

2016-11-09 Thread Andre Vehreschild
Hi Janus, may I ask you to attach also the "obvious" patches to the mail you send to the list? It is far more comfortable to take a look at the patch in the mail than open the webpage. Furthermore is it considered polite to attach the patches even when they are obvious. This is not to treat you ba

[PATCH 0/6][ARM] Implement support for ACLE Coprocessor Intrinsics

2016-11-09 Thread Andre Vieira (lists)
Hello, This patch series aims to implement support for the ACLE coprocessor intrinsics. To do so I first refactor the NEON builtin framework to accommodate the usage of this framework to implement other builtins. I then port the existing CRC builtins to this framework to reduce code-duplication an

Re: [PATCH] combine: Do not call simplify from inside change_zero_ext (PR78232)

2016-11-09 Thread Segher Boessenkool
On Wed, Nov 09, 2016 at 10:28:37AM +0100, Eric Botcazou wrote: > > You need more than that, e.g. reg_nonzero_bits_for_combine looks at the > > nonzero_bits field no matter what else. > > Right, this one is sticky, although you can invalidate it globally by means > of > nonzero_sign_valid AFAICS.

[PATCH 1/6][ARM] Refactor NEON builtin framework to work for other builtins

2016-11-09 Thread Andre Vieira (lists)
Hi, Refactor NEON builtin framework such that it can be used to implement other builtins. Is this OK for trunk? Regards, Andre gcc/ChangeLog 2016-11-09 Andre Vieira * config/arm/arm-builtins.c (neon_builtin_datum): Rename to .. (arm_builtin_datum): ... this. (arm_init_neon_built

[PATCH 2/6][ARM] Move CRC builtins to refactored framework

2016-11-09 Thread Andre Vieira (lists)
Hi, This patch refactors the implementation of the ARM ACLE CRC builtins to use the builtin framework. Is this OK for trunk? Regards, Andre gcc/ChangeLog 2016-11-09 Andre Vieira * config/arm/arm-builtins.c (arm_unsigned_binop_qualifiers): New. (UBINOP_QUALIFIERS): New. (si_UP): Define

[PATCH 3/6][ARM] Implement support for ACLE Coprocessor CDP intrinsics

2016-11-09 Thread Andre Vieira (lists)
Hi, This patch implements support for the ARM ACLE Coprocessor CDP intrinsics. See below a table mapping the intrinsics to their respective instructions: ++--+ | Intrinsic signature

[PATCH 4/6][ARM] Implement support for ACLE Coprocessor LDC and STC intrinsics

2016-11-09 Thread Andre Vieira (lists)
Hi, This patch implements support for the ARM ACLE Coprocessor LDC and STC intrinsics. See below a table mapping the intrinsics to their respective instructions: ++--+ | Intrinsic signature

[PATCH 5/6][ARM] Implement support for ACLE Coprocessor MCR and MRC intrinsics

2016-11-09 Thread Andre Vieira (lists)
Hi, This patch implements support for the ARM ACLE Coprocessor MCR and MRC intrinsics. See below a table mapping the intrinsics to their respective instructions: +---+---+ | Intrinsic signature

[PATCH 6/6][ARM] Implement support for ACLE Coprocessor MCRR and MRRC intrinsics

2016-11-09 Thread Andre Vieira (lists)
Hi, This patch implements support for the ARM ACLE Coprocessor MCR and MRC intrinsics. See below a table mapping the intrinsics to their respective instructions: +---+---+ | Intrinsic signature

Re: [PATCH] Convert character arrays to string csts

2016-11-09 Thread Richard Biener
On Fri, Nov 4, 2016 at 2:33 PM, Martin Liška wrote: > On 11/03/2016 02:00 PM, Jan Hubicka wrote: >>> On 11/03/2016 01:12 PM, Martin Liška wrote: + tree init = DECL_INITIAL (decl); + if (init + && TREE_READONLY (decl) + && can_convert_ctor_to_string_cst (init))

Re: [Patch, Fortran, committed] PR 71894: [OOP] ICE in gfc_add_component_ref, at fortran/class.c:227

2016-11-09 Thread Janus Weil
Hi Andre, sorry, didn't want to be impolite. I was just not aware that it makes a big difference whether I attach the patch or post the link. Patch below. Cheers, Janus Index: gcc/fortran/class.c === --- gcc/fortran/class.c(Rev

Re: [PATCH 7/7, GCC, ARM, V8M] Added support for ARMV8-M Security Extension cmse_nonsecure_caller intrinsic

2016-11-09 Thread Kyrill Tkachov
Hi Andre, On 25/10/16 17:30, Andre Vieira (lists) wrote: On 24/08/16 12:01, Andre Vieira (lists) wrote: On 25/07/16 14:28, Andre Vieira (lists) wrote: This patch adds support ARMv8-M's Security Extension's cmse_nonsecure_caller intrinsic. This intrinsic is used to check whether an entry functi

Re: [PATCH, vec-tails] Support loop epilogue vectorization

2016-11-09 Thread Bin.Cheng
On Tue, Nov 1, 2016 at 12:38 PM, Yuri Rumyantsev wrote: > Hi All, > > I re-send all patches sent by Ilya earlier for review which support > vectorization of loop epilogues and loops with low trip count. We > assume that the only patch - vec-tails-07-combine-tail.patch - was not > approved by Jeff.

Reject out-of-range bit pos in bit-fields insns operating on a register.

2016-11-09 Thread Andreas Schwab
As seen by the testcase in PR77822, combine can generate out-of-range bit pos in a bit-field insn, unless the pattern explicitly rejects it. This only makes a difference for expressions that are undefined at runtime. Without that we would either generate bad assembler or ICE in output_btst.

Re: Prevent aliasing between arguments in calls to move_alloc

2016-11-09 Thread Paul Richard Thomas
Hi Steve, Committed as r241995. Thanks. Paul On 8 November 2016 at 20:43, Steve Kargl wrote: > Yes. I saw Ian's analysis in c.l.f. It seems we both got > caught out on this one. The patch looks fine. > > -- > steve > > On Tue, Nov 08, 2016 at 08:26:37PM +0100, Paul Richard Thomas wrote: >> H

[PATCH] Remove flag_evaluation_order (Java remains)

2016-11-09 Thread Richard Biener
The Java FE was the only thing setting flag_evaluation_order to nonzero thus the following removes that flag and adjusts code checking it (a followup will remove the now unused arg from tree_swap_operands_p). Bootstrap / regtest in progress on x86_64-unknown-linux-gnu. Richard. 2016-11-09 Rich

[PATCH, GCC/testsuite] Fix PR78269: noexcept-type9.C should be a compile test

2016-11-09 Thread Thomas Preudhomme
g++.dg/cpp1z/noexcept-type9.C contains a dg-error but is currently a runtime test. The intent was clearly to make it a compile test to test that error, especially since there is no main. This patch changes it to a compile test. ChangeLog entry is as follows: *** gcc/testsuite/ChangeLog *** 20

[PATCH] Remove redundant canonicalize_comparison from DOM

2016-11-09 Thread Richard Biener
DOM currently canonicalizes GIMPLE_COND operand order. But it does so before constant/copy propagation into that stmt (so quite pointless) and furthermore we fold the stmt if we constant/copy propagated into it and fold already does the canonicalization. Thus, removed. Bootstrap & regtest pendi

Re: [PATCH, vec-tails] Support loop epilogue vectorization

2016-11-09 Thread Yuri Rumyantsev
Thanks Richard for your comments. Your proposed to handle epilogue loop just like normal short-trip loop but this is not exactly truth since e.g. epilogue must not be peeled for alignment. It is not clear for me what are my next steps? Should I re-design the patch completely or simply decompose th

Re: [PATCH, vec-tails] Support loop epilogue vectorization

2016-11-09 Thread Bin.Cheng
On Wed, Nov 9, 2016 at 11:28 AM, Yuri Rumyantsev wrote: > Thanks Richard for your comments. > Your proposed to handle epilogue loop just like normal short-trip loop > but this is not exactly truth since e.g. epilogue must not be peeled > for alignment. Yes there must be some differences, my motiva

Re: [PATCH] DECL_RTL and DECL_RTL_INCOMING in RTL dumps

2016-11-09 Thread Bernd Schmidt
On 11/08/2016 07:03 PM, David Malcolm wrote: int __RTL("rtl-combine") f1 (int n) { (function "f1" (param "n" (DECL_RTL (reg/v:SI %1 [ n ]) ) ;; DECL_RTL The ;; DECL_RTL etc. comments seem somewhat redundant and add clutter. Please remove those. Also, why is the closing paren

Re: [PATCH, vec-tails] Support loop epilogue vectorization

2016-11-09 Thread Yuri Rumyantsev
I am familiar with SVE extension and understand that implemented approach might be not suitable for ARM. The main point is that only load/store instructions are masked but all other calculations are not (we did special conversion for reduction statements to implement merging predication semantic).

[WIP C++ PATCH] P0217R3 - C++17 structured bindings

2016-11-09 Thread Jakub Jelinek
Hi! The following patch is a WIP on P0217R3 - decomposition declarations. It contains various FIXMEs, Jason, do you think you could finish it up? The most important unfinished part in the patch is that cp_finish_decomp for classes doesn't try to check if std::tuple_size::value is well-formed inte

Re: [PATCH, vec-tails] Support loop epilogue vectorization

2016-11-09 Thread Bin.Cheng
On Wed, Nov 9, 2016 at 12:12 PM, Yuri Rumyantsev wrote: > I am familiar with SVE extension and understand that implemented > approach might be not suitable for ARM. The main point is that only > load/store instructions are masked but all other calculations are not > (we did special conversion for

Re: [RFC] Check number of uses in simplify_cond_using_ranges().

2016-11-09 Thread Richard Biener
On Tue, Nov 8, 2016 at 5:18 PM, Marc Glisse wrote: > On Tue, 8 Nov 2016, Dominik Vogt wrote: > >> On Fri, Nov 04, 2016 at 01:54:20PM +0100, Richard Biener wrote: >>> >>> On Fri, Nov 4, 2016 at 12:08 PM, Dominik Vogt >>> wrote: On Fri, Nov 04, 2016 at 09:47:26AM +0100, Richard Biener wro

[Patch, fortran] PR44265 - Link error with reference to parameter array in specification expression

2016-11-09 Thread Paul Richard Thomas
Dear All, The title of this PR says what this is all about, except that it applies uniquely applicable to character function result string lengths. Ian Harvey wrote the first patch for this PR for which many thanks. However, two issues came up that took some little while to understand; (i) In com

[PING][PATCH v2][aarch64] Add mcpu flag for Qualcomm falkor core

2016-11-09 Thread Siddhesh Poyarekar
Ping! Siddhesh On 4 November 2016 at 21:17, Siddhesh Poyarekar wrote: > This adds an mcpu option for the upcoming Qualcomm Falkor core. This > is identical to the qdf24xx part that was added earlier and hence > retains the same tuning structure and continues to have the a57 > pipeline for now.

Re: [PATCH, vec-tails] Support loop epilogue vectorization

2016-11-09 Thread Richard Biener
On Wed, 9 Nov 2016, Yuri Rumyantsev wrote: > Thanks Richard for your comments. > Your proposed to handle epilogue loop just like normal short-trip loop > but this is not exactly truth since e.g. epilogue must not be peeled > for alignment. But if we know the epilogue data-refs are aligned we shou

[PATCH, ARM] Enable ldrd/strd peephole rules unconditionally

2016-11-09 Thread Bernd Edlinger
Hi! This patch enables the ldrd/strd peephole rules unconditionally. It is meant to fix cases, where the patch to reduce the sha512 stack usage splits ldrd/strd instructions into separate ldr/str insns, but is technically independent from the other patch: See https://gcc.gnu.org/ml/gcc-patches/

Re: [PATCH, RFC] Improve ivopts group costs

2016-11-09 Thread Bin.Cheng
On Thu, Nov 3, 2016 at 4:00 PM, Bin.Cheng wrote: > On Thu, Nov 3, 2016 at 1:35 PM, Evgeny Kudryashov > wrote: >> Hello, >> >> I'm facing the following problem related to ivopts. The problem is that GCC >> generates a lot of induction variables and as a result there is an >> unnecessary increase

Re: [WIP C++ PATCH] P0217R3 - C++17 structured bindings

2016-11-09 Thread Jakub Jelinek
Hi! On Wed, Nov 09, 2016 at 01:24:22PM +0100, Jakub Jelinek wrote: > The following patch is a WIP on P0217R3 - decomposition declarations. > It contains various FIXMEs, Jason, do you think you could finish it up? Some more comments: Invalid? code like int arr[2]; extern int x, y; auto [ x, y ] =

[PATCH] Remove unneeded gcc_assert in gimplifier (PR sanitizer/78270)

2016-11-09 Thread Martin Liška
As shown in the attached test-case, the assert cannot always be true. Patch can bootstrap on ppc64le-redhat-linux and survives regression tests. Ready to be installed? Martin >From b55459461f3f7396a094be6801082715ddb4b30d Mon Sep 17 00:00:00 2001 From: marxin Date: Wed, 9 Nov 2016 11:52:00 +0100

Re: [PATCH] Remove unneeded gcc_assert in gimplifier (PR sanitizer/78270)

2016-11-09 Thread Jakub Jelinek
On Wed, Nov 09, 2016 at 02:16:45PM +0100, Martin Liška wrote: > As shown in the attached test-case, the assert cannot always be true. > Patch can bootstrap on ppc64le-redhat-linux and survives regression tests. > > Ready to be installed? > Martin > >From b55459461f3f7396a094be6801082715ddb4b30d M

Re: [PATCH] Remove unneeded gcc_assert in gimplifier (PR sanitizer/78270)

2016-11-09 Thread Martin Liška
On 11/09/2016 02:29 PM, Jakub Jelinek wrote: > On Wed, Nov 09, 2016 at 02:16:45PM +0100, Martin Liška wrote: >> As shown in the attached test-case, the assert cannot always be true. >> Patch can bootstrap on ppc64le-redhat-linux and survives regression tests. >> >> Ready to be installed? >> Martin

[PATCH] Fix folding of memcmp("a", "a", 2) (PR, tree-optimization/78257)

2016-11-09 Thread Martin Liška
Hello. Following patch fixes [1] (f0 function), where we have off-by-one error. Patch can bootstrap on ppc64le-redhat-linux and survives regression tests. Ready to be installed? Martin [1] https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78257#c0 >From d7c46e5eb4d295d7653eae188cae04a8f7f0719f Mon Se

Re: [PATCH] Fix folding of memcmp("a", "a", 2) (PR, tree-optimization/78257)

2016-11-09 Thread Richard Biener
On Wed, Nov 9, 2016 at 2:56 PM, Martin Liška wrote: > Hello. > > Following patch fixes [1] (f0 function), where we have off-by-one error. > Patch can bootstrap on ppc64le-redhat-linux and survives regression tests. > > Ready to be installed? Ok (w/o the unrelated gimplify.c hunk). Richard. > Ma

[PATCH] Remove unused parameter from tree_swap_operands_p

2016-11-09 Thread Richard Biener
Bootstrapped on x86_64-unknown-linux-gnu, testing in progress. Richard. 2016-11-09 Richard Biener * fold-const.c (tree_swap_operands_p): Remove unused arg. * fold-const.c (tree_swap_operands_p): Likewise. (fold_binary_loc): Adjust. (fold_ternary_loc): Likewise

Re: [PATCH] Fix folding of memcmp("a", "a", 2) (PR, tree-optimization/78257)

2016-11-09 Thread Martin Liška
On 11/09/2016 02:58 PM, Richard Biener wrote: > On Wed, Nov 9, 2016 at 2:56 PM, Martin Liška wrote: >> Hello. >> >> Following patch fixes [1] (f0 function), where we have off-by-one error. >> Patch can bootstrap on ppc64le-redhat-linux and survives regression tests. >> >> Ready to be installed? >

[arm-embedded][PATCH, gcc/ARM, ping] Add support for Cortex-M23

2016-11-09 Thread Thomas Preudhomme
Hi, We have decided to backport this patch to add support for ARM Cortex-M23 to our embedded-6-branch. 2016-11-09 Thomas Preud'homme Backport from mainline 2016-11-04 Thomas Preud'homme gcc/ * config/arm/arm-arches.def (armv8-m.base): Set Cortex-M23 as representativ

[arm-embedded][PATCH, gcc/ARM, ping] Add support for Cortex-M33

2016-11-09 Thread Thomas Preudhomme
Hi, We have decided to backport this patch to add support for ARM Cortex-M33 to our embedded-6-branch. 2016-11-09 Thomas Preud'homme Backport from mainline 2016-11-04 Thomas Preud'homme gcc/ * config/arm/arm-arches.def (armv8-m.main+dsp): Set Cortex-M33 as represent

Re: [RFC] Check number of uses in simplify_cond_using_ranges().

2016-11-09 Thread Dominik Vogt
On Wed, Nov 09, 2016 at 01:43:58PM +0100, Richard Biener wrote: > On Tue, Nov 8, 2016 at 5:18 PM, Marc Glisse wrote: > > On Tue, 8 Nov 2016, Dominik Vogt wrote: > > > >> On Fri, Nov 04, 2016 at 01:54:20PM +0100, Richard Biener wrote: > >>> > >>> On Fri, Nov 4, 2016 at 12:08 PM, Dominik Vogt > >>>

Re: [PATCH] Fix folding of memcmp("a", "a", 2) (PR, tree-optimization/78257)

2016-11-09 Thread Richard Biener
On Wed, Nov 9, 2016 at 3:08 PM, Martin Liška wrote: > On 11/09/2016 02:58 PM, Richard Biener wrote: >> On Wed, Nov 9, 2016 at 2:56 PM, Martin Liška wrote: >>> Hello. >>> >>> Following patch fixes [1] (f0 function), where we have off-by-one error. >>> Patch can bootstrap on ppc64le-redhat-linux an

Re: [patch, avr] Add flash size to device info and make wrap around default

2016-11-09 Thread Georg-Johann Lay
On 09.11.2016 10:14, Pitchumani Sivanupandi wrote: On Tuesday 08 November 2016 02:57 PM, Georg-Johann Lay wrote: On 08.11.2016 08:08, Pitchumani Sivanupandi wrote: I have updated patch to include the flash size as well. Took that info from device headers (it was fed into crt's device informatio

Re: [RFC] Check number of uses in simplify_cond_using_ranges().

2016-11-09 Thread Richard Biener
On Wed, Nov 9, 2016 at 3:30 PM, Dominik Vogt wrote: > On Wed, Nov 09, 2016 at 01:43:58PM +0100, Richard Biener wrote: >> On Tue, Nov 8, 2016 at 5:18 PM, Marc Glisse wrote: >> > On Tue, 8 Nov 2016, Dominik Vogt wrote: >> > >> >> On Fri, Nov 04, 2016 at 01:54:20PM +0100, Richard Biener wrote: >> >>

Re: [PATCH 7/7, GCC, ARM, V8M] Added support for ARMV8-M Security Extension cmse_nonsecure_caller intrinsic

2016-11-09 Thread Andre Vieira (lists)
On 09/11/16 10:26, Kyrill Tkachov wrote: > > @@ -1832,6 +1834,17 @@ arm_init_builtins (void) > = add_builtin_function ("__builtin_arm_stfscr", ftype_set_fpscr, > ARM_BUILTIN_SET_FPSCR, BUILT_IN_MD, NULL, NULL_TREE); > } > + > + if (arm_arch_cmse) > +{ > + tree

Re: [PATCHv2 6/7, GCC, ARM, V8M] ARMv8-M Security Extension's cmse_nonsecure_call: use __gnu_cmse_nonsecure_call

2016-11-09 Thread Andre Vieira (lists)
On 27/10/16 11:01, Andre Vieira (lists) wrote: > On 25/10/16 17:30, Andre Vieira (lists) wrote: >> On 24/08/16 12:01, Andre Vieira (lists) wrote: >>> On 25/07/16 14:26, Andre Vieira (lists) wrote: This patch extends support for the ARMv8-M Security Extensions 'cmse_nonsecure_call' to use

Re: [Patch, fortran] PR44265 - Link error with reference to parameter array in specification expression

2016-11-09 Thread Dominique d'Humières
Sorry for the bad news, but while gfortran regtests with regression with the patch I still get a link error with the original test: % gfc pr44265.f90 Undefined symbols for architecture x86_64: "___fruits_MOD_names", referenced from: _MAIN__ in ccyeNqa1.o ld: symbol(s) not found for archit

Re: [PATCH] fix PR68468

2016-11-09 Thread Bernd Schmidt
On 11/05/2016 06:14 PM, Waldemar Brodkorb wrote: Hi, the following patch fixes PR68468. Patch is used for a while in Buildroot without issues. 2016-11-05 Waldemar Brodkorb PR gcc/68468 * libgcc/unwind-dw2-fde-dip.c: fix build on FDPIC targets. This is ok. Bernd

[PATCH] Add testcase for PR middle-end/77718

2016-11-09 Thread Jakub Jelinek
Hi! I've noticed this PR is still open, but has been fixed, though apparently Bernd's patch from the PR has been applied as is with the needed tab eaten (instead 7 spaces) and no testcase has been added. Tested with cross-compiler and the r240625 change reverted and as is and running the resultin

[PING][PATCH][aarch64] Improve Logical And Immediate Expressions

2016-11-09 Thread Michael Collison
Ping. Link to original post: https://gcc.gnu.org/ml/gcc-patches/2016-10/msg02305.html

Re: [PATCH] Fix folding of memcmp("a", "a", 2) (PR, tree-optimization/78257)

2016-11-09 Thread David Edelsohn
This patch breaks bootstrap on AIX and probably on any 32 bit host. Thanks, David /nasfarm/edelsohn/src/src/gcc/fold-const-call.c: In function 'tree_node* fold_const_call(combined_fn, tree, tree, tree, tree)': /nasfarm/edelsohn/src/src/gcc/fold-const-call.c:1541:36: error: cannot convert 'size_t*

[PATCH, Fortran] PR78259: ICE in gfc_trans_subcomponent_assign, at fortran/trans-expr.c:7330

2016-11-09 Thread Fritz Reese
All, Will commit the below to trunk as an obvious fix for PR78259. (The regression was introduced in r241626 from https://gcc.gnu.org/ml/fortran/2016-10/msg00206.html). The patch clears regtests. --- Fritz Reese From: Fritz O. Reese Date: Wed, 9 Nov 2016 10:59:17 -0500 Subject: [PATCH] Fix ICE

Re: [PATCH] Add testcase for PR middle-end/77718

2016-11-09 Thread Richard Biener
On November 9, 2016 4:34:27 PM GMT+01:00, Jakub Jelinek wrote: >Hi! > >I've noticed this PR is still open, but has been fixed, though >apparently Bernd's patch from the PR has been applied as is with >the needed tab eaten (instead 7 spaces) and no testcase has been added. > >Tested with cross-com

[PATCH] (v2) print-rtl-function.c: add (param) directive to dump

2016-11-09 Thread David Malcolm
On Wed, 2016-11-09 at 12:59 +0100, Bernd Schmidt wrote: > On 11/08/2016 07:03 PM, David Malcolm wrote: > > int __RTL("rtl-combine") f1 (int n) > > { > > (function "f1" > > (param "n" > > (DECL_RTL > > (reg/v:SI %1 [ n ]) > > ) ;; DECL_RTL > > The ;; DECL_RTL etc. comments seem somew

[PATCH, GCC/ARM] Fix ICE when compiling empty FIQ interrupt handler in ARM mode

2016-11-09 Thread Thomas Preudhomme
Hi, This patch fixes the following ICE when building when compiling an empty FIQ interrupt handler in ARM mode: empty_fiq_handler.c:5:1: error: insn does not satisfy its constraints: } ^ (insn/f 13 12 14 (set (reg/f:SI 13 sp) (plus:SI (reg/f:SI 11 fp) (const_int 4 [0x4])

Re: [PATCH] fix PR68468

2016-11-09 Thread Jakub Jelinek
On Wed, Nov 09, 2016 at 04:08:39PM +0100, Bernd Schmidt wrote: > On 11/05/2016 06:14 PM, Waldemar Brodkorb wrote: > >Hi, > > > >the following patch fixes PR68468. > >Patch is used for a while in Buildroot without issues. > > > >2016-11-05 Waldemar Brodkorb Two spaces before < instead of just one

Re: [PATCH, LIBGCC] Avoid count_leading_zeros with undefined result (PR 78067)

2016-11-09 Thread James Greenhalgh
On Sat, Oct 29, 2016 at 05:47:54AM +, Bernd Edlinger wrote: > On 10/28/16 16:05, Bernd Edlinger wrote: > > On 10/27/16 22:23, Joseph Myers wrote: > >> On Thu, 27 Oct 2016, Bernd Edlinger wrote: > >> > >>> Hi, > >>> > >>> by code reading I became aware that libgcc can call count_leading_zeros >

Re: [PATCH, GCC/ARM] Fix PR77933: stack corruption on ARM when using high registers and lr

2016-11-09 Thread Thomas Preudhomme
I've reworked the patch following comments from Wilco [1] (sorry could not find it in my MUA for some reason). [1] https://gcc.gnu.org/ml/gcc-patches/2016-11/msg00317.html == Context == When saving registers, function thumb1_expand_prologue () aims at minimizing the number of push instructio

Re: [PATCH] (v2) print-rtl-function.c: add (param) directive to dump

2016-11-09 Thread Bernd Schmidt
On 11/09/2016 05:46 PM, David Malcolm wrote: OK for trunk if it passes bootstrap and regrtest? gcc/ChangeLog: * print-rtl-function.c (print_any_param_name): New function. (print_param): New function. (print_rtx_function): Call print_param for each argument. * pri

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

2016-11-09 Thread Joseph Myers
On Wed, 9 Nov 2016, Richard Biener wrote: > I'll push back c_parser to the header and put inlines I need to export > there as well. > > Joseph, is this (with regard to the inlines) your preference as well? I'm not clear what the proposal is. If some patch is now proposed different from what wa

Re: [PATCH, LIBGCC] Avoid count_leading_zeros with undefined result (PR 78067)

2016-11-09 Thread Joseph Myers
On Wed, 9 Nov 2016, James Greenhalgh wrote: > These new test cases look like they are going to be out of exponent range > for _Float16 - so the testcases will fail for a target which tests either > of: > > gcc.dg/torture/fp-int-convert-float16.c > gcc.dg/torture/fp-int-convert-float16-timode.

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

2016-11-09 Thread Sandra Loosemore
On 11/07/2016 05:54 AM, Georg-Johann Lay wrote: @@ -15261,6 +15262,13 @@ GCC supports the following AVR devices a @include avr-mmcu.texi +@item -mabsdata +@opindex mabsdata + +Assume that all data in static stocage can be accessed by LDS / STS s/stocage/storage/ +inctructions. This option

Re: [PATCH] enable -fprintf-return-value by default

2016-11-09 Thread Sandra Loosemore
On 11/08/2016 08:13 PM, Martin Sebor wrote: The -fprintf-return-value optimization has been disabled since the last time it caused a bootstrap failure on powerpc64le. With the underlying problems fixed GCC has bootstrapped fine on all of powerpc64, powerpc64le and x86_64 and tested with no regre

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

2016-11-09 Thread Toma Tabacu
> From: Moore, Catherine [mailto:catherine_mo...@mentor.com] > Sent: 08 November 2016 20:47 > To: Toma Tabacu; gcc-patches@gcc.gnu.org > Cc: Matthew Fortune > Subject: RE: [PATCH,testsuite] MIPS: Upgrade to MIPS IV if using (HAS_MOVN) > with MIPS III. > > > > > -Original Message- > > Fro

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

2016-11-09 Thread Richard Biener
On November 9, 2016 5:45:32 PM GMT+01:00, Joseph Myers wrote: >On Wed, 9 Nov 2016, Richard Biener wrote: > >> I'll push back c_parser to the header and put inlines I need to >export >> there as well. >> >> Joseph, is this (with regard to the inlines) your preference as well? > >I'm not clear wha

Re: [PATCH, LIBGCC] Avoid count_leading_zeros with undefined result (PR 78067)

2016-11-09 Thread Bernd Edlinger
On 11/09/16 17:52, Joseph Myers wrote: > On Wed, 9 Nov 2016, James Greenhalgh wrote: > >> These new test cases look like they are going to be out of exponent range >> for _Float16 - so the testcases will fail for a target which tests either >> of: >> >> gcc.dg/torture/fp-int-convert-float16.c >>

[Patch, Fortran, committed] PR 60777: [F03] RECURSIVE function rejected in specification expression

2016-11-09 Thread Janus Weil
Hi all, I just committed a close-to-obvious fix for PR60777 (ok'd by Steve in the PR), see attachment: https://gcc.gnu.org/viewcvs?rev=242009&root=gcc&view=rev Cheers, Janus Index: gcc/fortran/expr.c === --- gcc/fortran/expr.c (Rev

Re: [PATCH] Fix PR78189

2016-11-09 Thread Christophe Lyon
On 9 November 2016 at 09:36, Bin.Cheng wrote: > On Tue, Nov 8, 2016 at 9:11 AM, Richard Biener wrote: >> On Mon, 7 Nov 2016, Christophe Lyon wrote: >> >>> Hi Richard, >>> >>> >>> On 7 November 2016 at 09:01, Richard Biener wrote: >>> > >>> > The following fixes an oversight when computing alignm

Re: [PATCH 2/4] BRIG (HSAIL) frontend: The FE itself.

2016-11-09 Thread Pekka Jääskeläinen
Hi Martin, On Fri, Nov 4, 2016 at 4:58 PM, Martin Jambor wrote: > I personally primarily want this for debugging purposes, and we should > try to eventually report errors in a more comprehensible way than > HSAILasm. But more generally, and more importantly, if the input, > whether human readabl

[PATCH, i386]: Fix PR78626, wrong code with -fschedule-insns

2016-11-09 Thread Uros Bizjak
Hello! We need earlyclobber on output operand of doubleword shift insns, since we have to prevent (partial) output matching %ecx as count argument. 2016-11-09 Uros Bizjak PR target/78262 * config/i386/i386.md (*3_doubleword): Mark operand 0 as earlyclobber. testsuite/ChangeLog:

C++ PATCH for C++17 auto non-type template parameters

2016-11-09 Thread Jason Merrill
This was pretty straightforward; auto template parameters look more or less the same as any other parameter with dependent type, so substitution works without any changes, we just need to do auto deduction in a couple of places. The most involved bit was handling deduction of a type parameter from

C++ PATCH for some C++17 class deduction issues

2016-11-09 Thread Jason Merrill
Discussion at the meeting led me to notice that class template deduction wasn't working with template template parameters. This patch also improves a few diagnostic issues. Tested x86_64-pc-linux-gnu, applying to trunk. commit 66d2621abd8ed6bb47d0fda747a572e00aece25e Author: Jason Merrill Date:

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

2016-11-09 Thread Moore, Catherine
> -Original Message- > From: Toma Tabacu [mailto:toma.tab...@imgtec.com] > Sent: Wednesday, November 9, 2016 12:19 PM > Subject: RE: [PATCH,testsuite] MIPS: Upgrade to MIPS IV if using (HAS_MOVN) > with MIPS III. > > No, I don't have write access. > I would be grateful if you could comm

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

2016-11-09 Thread Jason Merrill
On Mon, Nov 7, 2016 at 2:24 AM, Richard Biener wrote: > The issue with moving is that I failed to export the definition of > c_parser in c-parser.h due to gengtype putting vec > handlers into gtype-c.h but not gtype-objc.h and thus objc bootstrap > fails :/ > > I believe (well, I hope) that code

[Patch, Fortran, committed] PR 46459: ICE (segfault): Invalid read in compare_actual_formal [error recovery]

2016-11-09 Thread Janus Weil
Hi all, I have committed yet another obvious ice-on-invalid fix: https://gcc.gnu.org/viewcvs?rev=242020&root=gcc&view=rev Cheers, Janus Index: gcc/fortran/interface.c === --- gcc/fortran/interface.c (Revision 241993) +++ gcc/for

[PATCH], PowerPC ISA 3.0, allow QImode/HImode to go into vector registers

2016-11-09 Thread Michael Meissner
The PowerPC ISA 3.0 (power9) has new instructions that make it feasible to allow QImode and HImode to be allocated to vector registers. The new instructions are: * load byte with zero extend * load half word with zero extend * store byte * store half word * extract byte from v

Re: [Patch, fortran] PR44265 - Link error with reference to parameter array in specification expression

2016-11-09 Thread Dominique d'Humières
> Le 9 nov. 2016 à 20:09, Paul Richard Thomas a > écrit : > > Dear Dominique, > > I am deeply embarrassed. This is the consequence of an additional > condition added at the last minute. No reason to be embarrassed;-) > The attached removes it and makes sure that the original bug is tested >

Re: [PATCH], PowerPC ISA 3.0, allow QImode/HImode to go into vector registers

2016-11-09 Thread Michael Meissner
Of course it would help, if I actually attached the patches: -- Michael Meissner, IBM IBM, M/S 2506R, 550 King Street, Littleton, MA 01460-6245, USA email: meiss...@linux.vnet.ibm.com, phone: +1 (978) 899-4797 Index: gcc/config/rs6000/rs6000.md

Re: [PATCH] (v2) print-rtl-function.c: add (param) directive to dump

2016-11-09 Thread David Malcolm
On Wed, 2016-11-09 at 17:46 +0100, Bernd Schmidt wrote: > On 11/09/2016 05:46 PM, David Malcolm wrote: > > > OK for trunk if it passes bootstrap and regrtest? > > > > gcc/ChangeLog: > > * print-rtl-function.c (print_any_param_name): New function. > > (print_param): New function. > > (

[PATCH] simplify-rtx: Transform (xor (and (xor A B) C) B) with C const

2016-11-09 Thread Segher Boessenkool
match.pd transforms (A&C)|(B&~C) to ((A^B)&C)^B, which is fewer operations if C is not const (and it is not on simple tests at least, this transform is done very early already). Various processors have "insert" instructions that can do this, but combine cannot build those from the xor-and-xor, esp

[PATCH] libstdc++: Allow using without lock free atomic int

2016-11-09 Thread Pauli
Compiling programs using std::future for old arm processors fails. The problem is caused by preprocessor check for atomic lock free int. Future can be changed to work correctly without lock free atomics with minor changes to exception_ptr implementation. Without lock free atomics there is questio

[PATCH] libstdc++: Improve code generation for atomic reference counting

2016-11-09 Thread Pauli
Atomic reference counting generates pessimistic code in platforms where builtin atomics could optimize code for following branch with subtract instruction. To allow better code generation with compile time constant addition can be checked for negative value. Those cases can then be better optimize

Re: [PATCH] simplify-rtx: Transform (xor (and (xor A B) C) B) with C const

2016-11-09 Thread Bernd Schmidt
On 11/09/2016 10:13 PM, Segher Boessenkool wrote: * simplify-rtx.c (simplify_binary_operation_1): Simplify (xor (and (xor A B) C) B) to (ior (and A C) (and B ~C)) and (xor (and (xor A B) C) B) to (ior (and A ~C) (and B C)) if C is a const_int. I think one of the

[ARM] PR 78253 do not resolve weak ref locally

2016-11-09 Thread Christophe Lyon
Hi, PR 78253 shows that the handling of weak references has changed for ARM with gcc-5. When r220674 was committed, default_binds_local_p_2 gained a new parameter (weak_dominate), which, when true, implies that a reference to a weak symbol defined locally will be resolved locally, even though it

Review debug message generation

2016-11-09 Thread François Dumont
Hi Here is a proposal to review how we generate the debug output in case of assertion failure. It removes usage of format_word which, as a side effect will fix PR 77459. Should I reference this PR in the ChangeLog ? I introduced a print_literal function to avoid using strlen on them.

  1   2   >