Re: [PATCH, i386]: Additional fix for PR62599 with -mcmodel=medium -fpic

2016-05-13 Thread Uros Bizjak
On Fri, May 13, 2016 at 1:20 AM, H.J. Lu wrote: >>> testsuite/gcc.target/i386/pr61599-{1,2}.c testcases expose a failure >>> with -mcmodel -fpic, where: >>> >>> /tmp/ccfpoxHY.o: In function `bar': >>> pr61599-2.c:(.text+0xe): relocation truncated to fit: R_X86_64_PC32 >>> against symbol `a' defin

[PATCH] Import config.sub and config.guess from upstream.

2016-05-13 Thread Jakub Sejdak
From: Kuba Sejdak --- ChangeLog| 4 +++ config.guess | 93 config.sub | 8 -- 3 files changed, 59 insertions(+), 46 deletions(-) diff --git a/ChangeLog b/ChangeLog index eac1cc6..e092878 100644 --- a/ChangeLog +++ b/Change

Re: [PATCH] Import config.sub and config.guess from upstream.

2016-05-13 Thread Jakub Sejdak
Is it OK for trunk, gcc-4.9, gcc-5 and gcc-6 branches? 2016-05-13 9:52 GMT+02:00 Jakub Sejdak : > From: Kuba Sejdak > > --- > ChangeLog| 4 +++ > config.guess | 93 > > config.sub | 8 -- > 3 files changed, 59 insertions(+

Re: "const" qualifier vs. OpenACC data/OpenMP map clauses

2016-05-13 Thread Jakub Jelinek
On Wed, May 11, 2016 at 03:50:28PM +0200, Thomas Schwinge wrote: > Hi! > > Ping. Or should I open a PR for that? I've posted a mail about this to the OpenMP lang committee, will see if something comes out of that. Thanks for bringing this up and yes, a PR is useful. Jakub

Re: [Patch ARM/AArch64 09/11] Add missing vrnd{,a,m,n,p,x} tests.

2016-05-13 Thread Jiong Wang
On 12/05/16 13:56, Christophe Lyon wrote: On 12 May 2016 at 10:45, Jiong Wang wrote: On 11/05/16 14:23, Christophe Lyon wrote: 2016-05-02 Christophe Lyon * gcc/testsuite/gcc.target/aarch64/advsimd-intrinsics/vrnd.c: New. * gcc/testsuite/gcc.target/aarch64/advsimd-intri

[i386] Fix SEGV in ix86_in_large_data_p (PR target/71080)

2016-05-13 Thread Rainer Orth
When running the full testsuite with -mcmodel=medium, several tests FAIL with a NULL pointer dereference as described in the PR. Since these calls ultimately originate from get_named_section(NULL_TREE, ...) calls, the obvious fix seems the right thing to do. Bootstrapped without regressions on x8

Re: [patch] Tidy up RTL libfunc machinery

2016-05-13 Thread Eric Botcazou
> The middle-end parts are ok, I'm leaving backend parts for their > maintainers to comment > but those parts are ok as well if they do not comment within a > reasonable amount of time. Thanks, I have applied the patch. > Did you check the internals manual if it needs adjustment? The only relate

Re: [i386] Fix SEGV in ix86_in_large_data_p (PR target/71080)

2016-05-13 Thread Uros Bizjak
On Fri, May 13, 2016 at 10:48 AM, Rainer Orth wrote: > When running the full testsuite with -mcmodel=medium, several tests FAIL > with a NULL pointer dereference as described in the PR. > > Since these calls ultimately originate from get_named_section(NULL_TREE, > ...) calls, the obvious fix seems

[PATCH, PR71084] Further improve of CFG change tracking in CSE

2016-05-13 Thread Ilya Enkovich
Hi, This patch improves cse_cfg_altered computation by taking into account cleanup_cfg returned values. This resolves another case of invalidated dominance info. Bootstrapped and regtested on x86_64-unknown-linux-gnu. OK for turnk? Thanks, Ilya -- gcc/ 2016-05-13 Ilya Enkovich * c

Re: Re: Re: [ARM] Enable __fp16 as a function parameter and return type.

2016-05-13 Thread Tejas Belagod
On 11/05/16 16:46, Joseph Myers wrote: On Wed, 11 May 2016, Tejas Belagod wrote: AFAICS, I don't think it mandates a double-rounding behavior for double to __fp16 conversions and I don't see a change in stand between the two versions of ACLE on the behavior of __fp16. It's not a change betwee

Re: [PATCH] [rtlfe] Barebones implementation of "__RTL"; next steps?

2016-05-13 Thread Richard Biener
On Thu, May 12, 2016 at 4:29 PM, David Malcolm wrote: > On Wed, 2016-05-11 at 11:57 +0200, Richard Biener wrote: >> On Wed, May 11, 2016 at 3:31 AM, Trevor Saunders < >> tbsau...@tbsaunde.org> wrote: >> > On Tue, May 10, 2016 at 05:01:00PM -0400, David Malcolm wrote: >> > > [CCing Prasad since thi

Re: [PATCH] Fix ASAN bootstrap (uninitialized variable warning)

2016-05-13 Thread Richard Biener
On Thu, May 12, 2016 at 6:28 PM, Martin Liška wrote: > Hi. > > Following patch is needed to survive --with-build-config=bootstrap-asan: > > ../../gcc/tree-vect-patterns.c: In function ‘gimple* > vect_recog_mask_conversion_pattern(vec*, tree_node**, tree_node**)’: > ../../gcc/tree-vect-patterns.c:

Re: [PATCH 3/3] Enhance dumps of IVOPTS

2016-05-13 Thread Bin.Cheng
On Thu, May 12, 2016 at 5:41 PM, Martin Liška wrote: > On 05/12/2016 03:51 PM, Bin.Cheng wrote: >> On Thu, May 12, 2016 at 1:13 PM, Martin Liška wrote: >>> On 05/10/2016 03:16 PM, Bin.Cheng wrote: Another way is to remove the use of id for struct iv_inv_expr_ent once for all. We can ch

Re: [PATCH, PR71084] Further improve of CFG change tracking in CSE

2016-05-13 Thread Richard Biener
On Fri, May 13, 2016 at 11:03 AM, Ilya Enkovich wrote: > Hi, > > This patch improves cse_cfg_altered computation by taking into account > cleanup_cfg returned values. This resolves another case of invalidated > dominance info. > > Bootstrapped and regtested on x86_64-unknown-linux-gnu. OK for tu

Re: Thoughts on memcmp expansion (PR43052)

2016-05-13 Thread Richard Biener
On Thu, May 12, 2016 at 7:14 PM, Bernd Schmidt wrote: > On 05/02/2016 03:14 PM, Richard Biener wrote: > >> I think it fits best in tree-ssa-strlen.c:strlen_optimize_stmt for the >> moment. > > > I've done this in this version. Note that this apparently means it won't be > run at -O unlike the prev

Re: [PATCH, DOC] Document ASAN_OPTIONS="halt_on_error" env variable.

2016-05-13 Thread Martin Liška
On 05/12/2016 02:44 PM, Jakub Jelinek wrote: > I think it isn't obvious that one needs to put halt_on_error=0 or > halt_on_error=1 into those options and what to do if you need multiple > options in there. What about changing the last sentence to: This can be overridden through a corresponding en

Re: [PATCH 3/3] Enhance dumps of IVOPTS

2016-05-13 Thread Martin Liška
On 05/13/2016 11:43 AM, Bin.Cheng wrote: > On Thu, May 12, 2016 at 5:41 PM, Martin Liška wrote: >> On 05/12/2016 03:51 PM, Bin.Cheng wrote: >>> On Thu, May 12, 2016 at 1:13 PM, Martin Liška wrote: On 05/10/2016 03:16 PM, Bin.Cheng wrote: > Another way is to remove the use of id for struc

[PATCH][RFC] Introduce BIT_FIELD_INSERT

2016-05-13 Thread Richard Biener
The following patch adds BIT_FIELD_INSERT, an operation to facilitate doing bitfield inserts on registers (as opposed to currently where we'd have a BIT_FIELD_REF store). Originally this was developed as part of bitfield lowering where bitfield stores were lowered into read-modify-write cycles an

Re: [patch] Fix PR tree-optimization/70884

2016-05-13 Thread Eric Botcazou
> Hmm, the patch looks obvious if it was the intent to allow constant > pool replacements > _not_ only when the whole constant pool entry may go away. But I > think the intent was > to not do this otherwise it will generate worse code by forcing all > loads from the constant pool to appear at > fu

Re: [PATCH, DOC] Document ASAN_OPTIONS="halt_on_error" env variable.

2016-05-13 Thread Jakub Jelinek
On Fri, May 13, 2016 at 12:26:57PM +0200, Martin Liška wrote: > On 05/12/2016 02:44 PM, Jakub Jelinek wrote: > > I think it isn't obvious that one needs to put halt_on_error=0 or > > halt_on_error=1 into those options and what to do if you need multiple > > options in there. > > What about changin

Re: [PATCH][ARM] PR target/70830: Avoid POP-{reglist}^ when returning from interrupt handlers

2016-05-13 Thread Kyrill Tkachov
Hi Christophe, On 12/05/16 20:57, Christophe Lyon wrote: On 12 May 2016 at 11:48, Ramana Radhakrishnan wrote: On Thu, May 5, 2016 at 12:50 PM, Kyrill Tkachov wrote: Hi all, In this PR we deal with some fallout from the conversion to unified assembly. We now end up emitting instructions like

[gomp4.5] Further progress on OpenMP 4.5 parsing

2016-05-13 Thread Jakub Jelinek
Hi! This patch adds parsing of new OpenMP 4.5 constructs (though, collapse(n) is still not handled), but we don't do anything about those during resolve and later. Committed to gomp-4_5-branch. 2016-05-13 Jakub Jelinek * parse.c (decode_omp_directive): Use gfc_match_omp_end_critical

Re: [PATCH, DOC] Document ASAN_OPTIONS="halt_on_error" env variable.

2016-05-13 Thread Martin Liška
On 05/13/2016 01:03 PM, Jakub Jelinek wrote: > On Fri, May 13, 2016 at 12:26:57PM +0200, Martin Liška wrote: >> On 05/12/2016 02:44 PM, Jakub Jelinek wrote: >>> I think it isn't obvious that one needs to put halt_on_error=0 or >>> halt_on_error=1 into those options and what to do if you need multip

Re: [PATCH] Improve other 13 define_insns

2016-05-13 Thread Kirill Yukhin
Hello, On 12 May 20:42, Jakub Jelinek wrote: > On Thu, May 12, 2016 at 05:20:02PM +0300, Kirill Yukhin wrote: > > > 2016-05-04 Jakub Jelinek > > > > > > * config/i386/sse.md (sse_shufps_, sse_storehps, sse_loadhps, > > > sse_storelps, sse_movss, avx2_vec_dup, avx2_vec_dupv8sf_1, > > > sse

Re: [PATCH] Improve other 13 define_insns

2016-05-13 Thread Jakub Jelinek
On Fri, May 13, 2016 at 02:46:36PM +0300, Kirill Yukhin wrote: > NP. Below is the patch which fixes both issues. > It also revealed, that for "*and" pattern 32 byte long > internal buffer is not enough. > I've extended bunch of such buffers to 128 bytes. > > Probably we might want to re-factor all

Re: [patch] Fix PR tree-optimization/70884

2016-05-13 Thread Richard Biener
On Fri, May 13, 2016 at 1:01 PM, Eric Botcazou wrote: >> Hmm, the patch looks obvious if it was the intent to allow constant >> pool replacements >> _not_ only when the whole constant pool entry may go away. But I >> think the intent was >> to not do this otherwise it will generate worse code by

Re: [PATCH, GCC] PR middle-end/55299, fold bitnot through ASR and rotates

2016-05-13 Thread Mikhail Maltsev
On 05/11/2016 10:52 AM, Marc Glisse wrote: > +/* ~((~X) >> Y) -> X >> Y (for arithmetic shift). */ > +(simplify > + (bit_not (convert? (rshift (bit_not @0) @1))) > + (if (!TYPE_UNSIGNED (TREE_TYPE (@0)) > + && TYPE_PRECISION (type) <= TYPE_PRECISION (TREE_TYPE (@0))) > + (convert (rshift

Re: Re: Re: [ARM] Enable __fp16 as a function parameter and return type.

2016-05-13 Thread Joseph Myers
On Fri, 13 May 2016, Tejas Belagod wrote: > > It's not a change between the two versions of ACLE. It's a change > > relative to the early (pre-ACLE) __fp16 specification (or, at least, a > > clarification thereto in email on 12 Aug 2008) that was used as a basis > > for the original implementatio

Re: [PATCH 3/3] Enhance dumps of IVOPTS

2016-05-13 Thread H.J. Lu
On Fri, May 13, 2016 at 3:44 AM, Martin Liška wrote: > On 05/13/2016 11:43 AM, Bin.Cheng wrote: >> On Thu, May 12, 2016 at 5:41 PM, Martin Liška wrote: >>> On 05/12/2016 03:51 PM, Bin.Cheng wrote: On Thu, May 12, 2016 at 1:13 PM, Martin Liška wrote: > On 05/10/2016 03:16 PM, Bin.Cheng w

[PATCH][combine] PR middle-end/71074 Check that const_op is >= 0 before potentially shifting in simplify_comparison

2016-05-13 Thread Kyrill Tkachov
Hi all, In this PR we may end up shifting a negative value left in simplify_comparison. The statement is: const_op <<= INTVAL (XEXP (op0, 1)); This patch guards the block of that statement by const_op >= 0. I _think_ that's a correct thing to do for that trasnformation: "If we have (compare (xsh

[PATCH] Fix part of PR42587

2016-05-13 Thread Richard Biener
The recent subreg-CSE allows a part of PR42587 to be fixed with minimal surgery to the bswap pass - namely adding support for BIT_FIELD_REF. Bootstrapped and tested on x86_64-unknown-linux-gnu, applied to trunk. Richard. 2016-05-13 Richard Biener PR tree-optimization/42587 *

[PATCH][obvious] Fix param name in dump file

2016-05-13 Thread Kyrill Tkachov
Hi all, The name of the parameter is max-completely-peel-times. Applying to trunk as obvious. Thanks, Kyrill 2016-05-13 Kyrylo Tkachov * tree-ssa-loop-ivcanon.c (try_unroll_loop_completely): Change --param max-completely-peeled-times to --param max-completely-peel-times in dump

Re: [PATCH 7/7] SMS remove dependence on doloop: To identify read/write register as loop induction variable

2016-05-13 Thread Bernd Schmidt
On 05/05/2016 08:03 AM, Shiva Chen wrote: - /* We do not handle setting only part of the register. */ - if (DF_REF_FLAGS (adef) & DF_REF_READ_WRITE) -return GRD_INVALID; - This isn't right, at least not without other changes. This prevents using references where the register is set as

[PATCH][obvious] Typo fix in tree-ssa-loop-ivanon.c

2016-05-13 Thread Kyrill Tkachov
Committing as obvious. Thanks, Kyrill 2016-05-12 Kyrylo Tkachov * tree-ssa-loop-ivanon.c (try_unroll_loop_completely): Typo fix in comment. diff --git a/gcc/tree-ssa-loop-ivcanon.c b/gcc/tree-ssa-loop-ivcanon.c index 9d92276dbbbfe3a768b9e8b0c90ee60e05c885fb..e8f67953231f54ce2517a55e1

[PATCH] Enable libgloss support for ARC in top-level configure.ac

2016-05-13 Thread Anton Kolesov
ARC support has been added to libgloss in this patch to newlib repository: https://sourceware.org/git/?p=newlib-cygwin.git;a=commit;h=acdfcb0a0af54715bc37ed1c767bfe901b679357 2016-05-13 Anton Kolesov * configure.ac: Add ARC support to libgloss. * configure: Regenerate --- conf

Re: [PATCH 3/3] Enhance dumps of IVOPTS

2016-05-13 Thread Martin Liška
On 05/13/2016 02:11 PM, H.J. Lu wrote: > On Fri, May 13, 2016 at 3:44 AM, Martin Liška wrote: >> On 05/13/2016 11:43 AM, Bin.Cheng wrote: >>> On Thu, May 12, 2016 at 5:41 PM, Martin Liška wrote: On 05/12/2016 03:51 PM, Bin.Cheng wrote: > On Thu, May 12, 2016 at 1:13 PM, Martin Liška wro

Re: [PATCH 3/3] Enhance dumps of IVOPTS

2016-05-13 Thread Kyrill Tkachov
Hi Martin, On 13/05/16 13:39, Martin Liška wrote: On 05/13/2016 02:11 PM, H.J. Lu wrote: On Fri, May 13, 2016 at 3:44 AM, Martin Liška wrote: On 05/13/2016 11:43 AM, Bin.Cheng wrote: On Thu, May 12, 2016 at 5:41 PM, Martin Liška wrote: On 05/12/2016 03:51 PM, Bin.Cheng wrote: On Thu, May

Re: [PATCH 3/3] Enhance dumps of IVOPTS

2016-05-13 Thread Richard Biener
On Fri, May 13, 2016 at 2:43 PM, Kyrill Tkachov wrote: > Hi Martin, > > > On 13/05/16 13:39, Martin Liška wrote: >> >> On 05/13/2016 02:11 PM, H.J. Lu wrote: >>> >>> On Fri, May 13, 2016 at 3:44 AM, Martin Liška wrote: On 05/13/2016 11:43 AM, Bin.Cheng wrote: > > On Thu, May 12,

Re: [PATCH 3/3] Enhance dumps of IVOPTS

2016-05-13 Thread Martin Liška
On 05/13/2016 02:46 PM, Richard Biener wrote: > Use them for HOST_WIDE_INT printing, for [u]int64_t use the PRI stuff. > > Richard. Thanks you both, installed as r236208. Martin

Re: [PATCH][combine] PR middle-end/71074 Check that const_op is >= 0 before potentially shifting in simplify_comparison

2016-05-13 Thread Bernd Schmidt
On 05/13/2016 02:21 PM, Kyrill Tkachov wrote: Hi all, In this PR we may end up shifting a negative value left in simplify_comparison. The statement is: const_op <<= INTVAL (XEXP (op0, 1)); This patch guards the block of that statement by const_op >= 0. I _think_ that's a correct thing to do for

[PTX] atomic_compare_exchange_$n

2016-05-13 Thread Nathan Sidwell
The atomic_compare_exchange_$n builtins have an exciting calling convention where the 4th ('weak') parm is dropped in a real call. This caused gcc.dg/atomic-noinline.c to fail on PTX as the prototype didn't match the use. Fixed thusly when emitting the ptx prototyp. Also fixed is the subseq

RUN_UNDER_VALGRIND statistics for GCC 7

2016-05-13 Thread Martin Liška
Hello. I've tried to apply the same patch for the current trunk and tried to separate reported errors to a different categories by a simple script ([1]). There are number (complete report: [2]): SECTION: gfortran error types: 3534, total errors: 113282 error types: 90.15%, total errors: 27.

Re: Thoughts on memcmp expansion (PR43052)

2016-05-13 Thread Bernd Schmidt
On 05/13/2016 12:20 PM, Richard Biener wrote: I'm not much of a fan of C++-ification (in this case it makes review harder) but well ... I felt it was a pretty natural way to structure the code to avoid duplicating the same logic across more functions, and we might as well use the language for

Re: Thoughts on memcmp expansion (PR43052)

2016-05-13 Thread Richard Biener
On Fri, May 13, 2016 at 3:05 PM, Bernd Schmidt wrote: > On 05/13/2016 12:20 PM, Richard Biener wrote: >> >> I'm not much of a fan of C++-ification (in this case it makes review >> harder) but well ... > > > I felt it was a pretty natural way to structure the code to avoid > duplicating the same lo

Re: Thoughts on memcmp expansion (PR43052)

2016-05-13 Thread Bernd Schmidt
On 05/13/2016 03:07 PM, Richard Biener wrote: On Fri, May 13, 2016 at 3:05 PM, Bernd Schmidt wrote: Huh? Can you elaborate? When you have a builtin taking a size in bytes then a byte is 8 bits, not BITS_PER_UNIT bits. That makes no sense to me. I think the definition of a byte depends on t

Re: [PATCH][combine] PR middle-end/71074 Check that const_op is >= 0 before potentially shifting in simplify_comparison

2016-05-13 Thread Kyrill Tkachov
On 13/05/16 14:01, Bernd Schmidt wrote: On 05/13/2016 02:21 PM, Kyrill Tkachov wrote: Hi all, In this PR we may end up shifting a negative value left in simplify_comparison. The statement is: const_op <<= INTVAL (XEXP (op0, 1)); This patch guards the block of that statement by const_op >= 0.

Re: [PATCH][combine] PR middle-end/71074 Check that const_op is >= 0 before potentially shifting in simplify_comparison

2016-05-13 Thread Bernd Schmidt
On 05/13/2016 03:22 PM, Kyrill Tkachov wrote: /* We only want to handle integral modes. This catches VOIDmode, CCmode, and the floating-point modes. An exception is that we @@ -11649,7 +11649,8 @@ simplify_comparison (enum rtx_code code, /* Try to simplify the compare to

Re: [PATCH, GCC] PR middle-end/55299, fold bitnot through ASR and rotates

2016-05-13 Thread Marc Glisse
On Fri, 13 May 2016, Mikhail Maltsev wrote: I don't know if we might want some :c / single_use restrictions, maybe on the outer convert and the rshift/rotate. I don't think :c can be used here. Oups, typo for :s. As for :s, I added it, as you suggested. :s will be ignored when there is n

Re: [ARM] Enable __fp16 as a function parameter and return type.

2016-05-13 Thread Ramana Radhakrishnan
On Thu, Apr 28, 2016 at 10:20 AM, Matthew Wahab wrote: > Hello, > > The ARM target supports the half-precision floating point type __fp16 > but does not allow its use as a function return or parameter type. This > patch removes that restriction and defines the ACLE macro > __ARM_FP16_ARGS to indic

Re: [AArch64] Remove an unused reload hook.

2016-05-13 Thread James Greenhalgh
On Thu, Apr 28, 2016 at 03:11:59PM +0100, Matthew Wahab wrote: > Hello, > > Yvan Roux pointed out that the patch at > https://gcc.gnu.org/ml/gcc-patches/2016-02/msg01713.html was never > committed. > > From the original submission: > > The LEGITIMIZE_RELOAD_ADDRESS macro is only needed for rel

[patch, avr] Fix unrecognizable insn ICE for avr (PR71103)

2016-05-13 Thread Pitchumani Sivanupandi
avr-gcc crashes for following test as it couldn't recognize the instruction pattern. struct st {   unsigned char uc1;   unsigned int *ui1; }; unsigned int ui1; struct st foo () {   struct st ret;   ret.uc1 = 6;   ret.ui1 = &ui1;   return ret; } $ avr-gcc -mmcu=atmega328p -O1 test.c (-- snip --) 

Re: [Patch ARM/AArch64 01/11] Fix typo in vreinterpret.c test comment.

2016-05-13 Thread James Greenhalgh
On Wed, May 11, 2016 at 03:23:51PM +0200, Christophe Lyon wrote: > 2016-05-02 Christophe Lyon > > * gcc.target/aarch64/advsimd-intrinsics/vreinterpret.c: Fix typo in > comment. This one would have been OK to commit as obvious. OK for trunk. Thanks, James

[Patch, testsuite] PR70227, skip g++.dg/lto/pr69589_0.C on targets without -rdynamic support

2016-05-13 Thread Jiong Wang
This patch skip g++.dg/lto/pr69589_0.C on typical arm & aarch64 bare-metal targets as they don't support "-rdynamic". spu-unknown-elf is known to be failing also, but I'd leave it to those who are more familar with all relevant spu targets. dg-skip-if is used instead of dg-xfail-if because the l

Re: [Patch ARM/AArch64 02/11] We can remove useless #ifdefs from these tests: vmul, vshl and vtst.

2016-05-13 Thread James Greenhalgh
On Wed, May 11, 2016 at 03:23:52PM +0200, Christophe Lyon wrote: > 2016-05-02 Christophe Lyon > > * gcc.target/aarch64/advsimd-intrinsics/vmul.c: Remove useless #ifdef. > * gcc.target/aarch64/advsimd-intrinsics/vshl.c: Likewise. > * gcc.target/aarch64/advsimd-intrinsics/vtst.c

Re: Thoughts on memcmp expansion (PR43052)

2016-05-13 Thread Bernd Schmidt
On 05/13/2016 03:53 PM, Joseph Myers wrote: * In C: a byte is the minimal addressable unit; an N-byte object is made up of N "unsigned char" objects, with successive addresses in terms of incrementing an "unsigned char *" pointer. A byte is at least 8 bits. * In GCC, at the level of GNU C APIs

Re: [testuite,AArch64] Make scan for 'br' more robust

2016-05-13 Thread James Greenhalgh
On Wed, May 04, 2016 at 11:55:42AM +0200, Christophe Lyon wrote: > On 4 May 2016 at 10:43, Kyrill Tkachov wrote: > > > > Hi Christophe, > > > > > > On 02/05/16 12:50, Christophe Lyon wrote: > >> > >> Hi, > >> > >> I've noticed a "regression" of AArch64's noplt_3.c in the gcc-6-branch > >> because

[Patch, ARM] PR71061, length pop* pattern in epilogue correctly

2016-05-13 Thread Jiong Wang
For thumb mode, this is causing wrong size calculation and may affect some rtl pass, for example bb-order where copy_bb_p needs accurate insn length info. This have eventually part of the reason for https://gcc.gnu.org/ml/gcc-patches/2016-05/msg00639.html where bb-order failed to do the bb copy.

Re: [Patch ARM/AArch64 04/11] Add forgotten vsliq_n_u64 test.

2016-05-13 Thread James Greenhalgh
On Wed, May 11, 2016 at 03:23:54PM +0200, Christophe Lyon wrote: > 2016-05-02 Christophe Lyon > > * gcc.target/aarch64/advsimd-intrinsics/vsli_n.c: Add check for > vsliq_n_u64. > And vsliq_n_s64 ? OK with that change. Thanks, James > Change-Id: I90bb2b225ffd7bfd54a0827a0264ac20271f5

Re: [Patch ARM/AArch64 03/11] AdvSIMD tests: be more verbose.

2016-05-13 Thread James Greenhalgh
On Wed, May 11, 2016 at 03:23:53PM +0200, Christophe Lyon wrote: > It is useful to have more detailed information in the logs when checking > validation results: instead of repeating the intrinsic name, we now print > its return type too. > > 2016-05-02 Christophe Lyon > > * gcc.target/a

Re: Thoughts on memcmp expansion (PR43052)

2016-05-13 Thread Joseph Myers
On Fri, 13 May 2016, Bernd Schmidt wrote: > On 05/13/2016 03:07 PM, Richard Biener wrote: > > On Fri, May 13, 2016 at 3:05 PM, Bernd Schmidt wrote: > > > Huh? Can you elaborate? > > > > When you have a builtin taking a size in bytes then a byte is 8 bits, > > not BITS_PER_UNIT bits. > > That ma

Re: [Patch ARM/AArch64 05/11] Add missing vreinterpretq_p{8,16} tests.

2016-05-13 Thread James Greenhalgh
On Wed, May 11, 2016 at 03:23:55PM +0200, Christophe Lyon wrote: > 2016-05-02 Christophe Lyon > > * gcc.target/aarch64/advsimd-intrinsics/vreinterpret.c: Add > missing tests for vreinterpretq_p{8,16}. OK. Thanks, James

Re: [PATCH 3/3] Enhance dumps of IVOPTS

2016-05-13 Thread H.J. Lu
On Fri, May 13, 2016 at 5:51 AM, Martin Liška wrote: > On 05/13/2016 02:46 PM, Richard Biener wrote: >> Use them for HOST_WIDE_INT printing, for [u]int64_t use the PRI stuff. >> >> Richard. > > Thanks you both, installed as r236208. > It isn't fixed: /export/gnu/import/git/sources/gcc/gcc/tree-s

Re: [Patch ARM/AArch64 04/11] Add forgotten vsliq_n_u64 test.

2016-05-13 Thread Christophe Lyon
On 13 May 2016 at 16:08, James Greenhalgh wrote: > On Wed, May 11, 2016 at 03:23:54PM +0200, Christophe Lyon wrote: >> 2016-05-02 Christophe Lyon >> >> * gcc.target/aarch64/advsimd-intrinsics/vsli_n.c: Add check for >> vsliq_n_u64. >> > > And vsliq_n_s64 ? > Damn! You are right, I missed

Re: [Patch ARM/AArch64 06/11] Add missing vtst_p8 and vtstq_p8 tests.

2016-05-13 Thread James Greenhalgh
On Wed, May 11, 2016 at 03:23:56PM +0200, Christophe Lyon wrote: > 2016-05-02 Christophe Lyon > > * gcc.target/aarch64/advsimd-intrinsics/vtst.c: Add tests > for vtst_p8 and vtstq_p8. And vtst_p16 and vtstq_p16 too please. vtst_s64 vtstq_s64 vtst_u64 vtstq_u64 are also missing (AA

Re: [Patch ARM/AArch64 07/11] Add missing vget_lane fp16 tests.

2016-05-13 Thread James Greenhalgh
On Wed, May 11, 2016 at 03:23:57PM +0200, Christophe Lyon wrote: > 2016-05-02 Christophe Lyon > > * gcc.target/aarch64/advsimd-intrinsics/vget_lane.c: Add fp16 tests. OK. Thanks, James

Re: [Patch ARM/AArch64 06/11] Add missing vtst_p8 and vtstq_p8 tests.

2016-05-13 Thread Christophe Lyon
On 13 May 2016 at 16:37, James Greenhalgh wrote: > On Wed, May 11, 2016 at 03:23:56PM +0200, Christophe Lyon wrote: >> 2016-05-02 Christophe Lyon >> >> * gcc.target/aarch64/advsimd-intrinsics/vtst.c: Add tests >> for vtst_p8 and vtstq_p8. > > And vtst_p16 and vtstq_p16 too please. >

Re: [PATCH 3/3] Enhance dumps of IVOPTS

2016-05-13 Thread H.J. Lu
On Fri, May 13, 2016 at 7:17 AM, H.J. Lu wrote: > On Fri, May 13, 2016 at 5:51 AM, Martin Liška wrote: >> On 05/13/2016 02:46 PM, Richard Biener wrote: >>> Use them for HOST_WIDE_INT printing, for [u]int64_t use the PRI stuff. >>> >>> Richard. >> >> Thanks you both, installed as r236208. >> > > I

[PATCH] libstdc++/71073 add system_header pragma to Debug Mode headers

2016-05-13 Thread Jonathan Wakely
Self-explanatory. Tested x86_64-linux, committed to trunk. PR libstdc++/71073 * include/debug/bitset: Add #pragma GCC system_header. * include/debug/deque: Likewise. * include/debug/list: Likewise. * include/debug/map: Likewise. * include/debug/set:

Re: [Patch ARM/AArch64 06/11] Add missing vtst_p8 and vtstq_p8 tests.

2016-05-13 Thread James Greenhalgh
On Fri, May 13, 2016 at 04:41:33PM +0200, Christophe Lyon wrote: > On 13 May 2016 at 16:37, James Greenhalgh wrote: > > On Wed, May 11, 2016 at 03:23:56PM +0200, Christophe Lyon wrote: > >> 2016-05-02 Christophe Lyon > >> > >> * gcc.target/aarch64/advsimd-intrinsics/vtst.c: Add tests > >>

[PATCH][MIPS] Enable LSA/DLSA for MSA

2016-05-13 Thread Robert Suchanek
Hi, The below enables LSA/DLSA instructions for -mmsa. Ok to commit? Regards, Robert * config/mips/mips.h (ISA_HAS_LSA): Enable for -mmsa. (ISA_HAS_DLSA): Ditto. --- gcc/config/mips/mips.h | 6 -- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/gcc/config/mips

[PATCH][MIPS] Correct latency of loads in M5100

2016-05-13 Thread Robert Suchanek
Hi, A small patch to correct the latency for M5100. Ok to commit? Regards, Robert 2016-05-13 Matthew Fortune * config/mips/m5100.md (m51_int_load): Update the latency to 2. --- gcc/config/mips/m5100.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/gcc/config/

Re: [Patch ARM/AArch64 08/11] Add missing vstX_lane fp16 tests.

2016-05-13 Thread James Greenhalgh
On Wed, May 11, 2016 at 03:23:58PM +0200, Christophe Lyon wrote: > 2016-05-02 Christophe Lyon > > * gcc.target/aarch64/advsimd-intrinsics/vstX_lane.c: Add fp16 tests. OK. Thanks, James

Re: [Patch ARM/AArch64 09/11] Add missing vrnd{,a,m,n,p,x} tests.

2016-05-13 Thread James Greenhalgh
On Wed, May 11, 2016 at 03:23:59PM +0200, Christophe Lyon wrote: > 2016-05-02 Christophe Lyon > > * gcc/testsuite/gcc.target/aarch64/advsimd-intrinsics/vrnd.c: New. > * gcc/testsuite/gcc.target/aarch64/advsimd-intrinsics/vrndX.inc: New. > * gcc/testsuite/gcc.target/aarch64/adv

Re: PATCH: PR target/70738: Add -mgeneral-regs-only option

2016-05-13 Thread H.J. Lu
On Thu, May 12, 2016 at 10:54 AM, H.J. Lu wrote: >>> Here is a patch to add >>> -mgeneral-regs-only option to x86 backend. We can update >>> spec for interrupt handle to recommend compiling interrupt handler >>> with -mgeneral-regs-only option and add a note for compiler >>> implementers. >>> >>

[Patch, avr] Include INCOMING_FRAME_SP_OFFSET when printing stack usage

2016-05-13 Thread Senthil Kumar Selvaraj
Hi, This trivial patch adds INCOMING_FRAME_SP_OFFSET to current_function_static_stack_size, thus fixing the 2 (or 3, for 3 byte PC devices) byte difference between reported and actual values when using -fstack-usage. The patch came about because of this discussion (https://gcc.gnu.org

Re: [Patch ARM/AArch64 11/11] Add missing tests for vreinterpret, operating of fp16 type.

2016-05-13 Thread James Greenhalgh
On Wed, May 11, 2016 at 03:24:01PM +0200, Christophe Lyon wrote: > 2016-05-04 Christophe Lyon > > * gcc.target/aarch64/advsimd-intrinsics/vreinterpret.c: Add fp16 > tests. > * gcc.target/aarch64/advsimd-intrinsics/vreinterpret_p128.c: Likewise. > * gcc.target/aarch64/ad

Re: [Patch ARM/AArch64 10/11] Add missing tests for intrinsics operating on poly64 and poly128 types.

2016-05-13 Thread James Greenhalgh
On Wed, May 11, 2016 at 03:24:00PM +0200, Christophe Lyon wrote: > 2016-05-02 Christophe Lyon > > * gcc.target/aarch64/advsimd-intrinsics/arm-neon-ref.h (result): > Add poly64x1_t and poly64x2_t cases if supported. > * gcc.target/aarch64/advsimd-intrinsics/compute-ref-data.h >

Re: [patch] Fix PR tree-optimization/70884

2016-05-13 Thread Martin Jambor
Hi, On Fri, May 13, 2016 at 01:01:50PM +0200, Eric Botcazou wrote: > > Hmm, the patch looks obvious if it was the intent to allow constant > > pool replacements > > _not_ only when the whole constant pool entry may go away. But I > > think the intent was > > to not do this otherwise it will gener

RE: [PATCH] MIPS: Ensure that lo_sums do not contain an unaligned symbol

2016-05-13 Thread Andrew Bennett
Hi Matthew, Many thanks for reviewing this patch. Upon closer validation of the patch I have found this this approach will not work in all cases so unfortunately I am going to have to abandon it. I will be shortly posting a new patch to fix this issue in the middle-end. Regards, Andrew >

Re: [Patch ARM/AArch64 00/11][testsuite] AdvSIMD intrinsics update

2016-05-13 Thread James Greenhalgh
On Wed, May 11, 2016 at 03:23:50PM +0200, Christophe Lyon wrote: > Hi, > > A few months ago, we decided it was time to remove neon-testgen.ml > and its generated tests. I did it, just to realize too late that > some intrinsics were not covered anymore, so I reverted the removal. > > This patch se

[PTX] assembler name mangling

2016-05-13 Thread Nathan Sidwell
I've committed this cleanup to use TARGET_MANGLE_DECL_ASSEMBLER_NAME rather than the ad-hoc solution the ptx backend currently has. I didn't address it during the earlier set of cleanups as I felt there were more important things to address then. I did discover an issue in langhooks, where t

[PATCH PR69848/partial]Propagate comparison into VEC_COND_EXPR if target supports

2016-05-13 Thread Bin Cheng
Hi, As PR69848 reported, GCC vectorizer now generates comparison outside of VEC_COND_EXPR for COND_REDUCTION case, as below: _20 = vect__1.6_8 != { 0, 0, 0, 0 }; vect_c_2.8_16 = VEC_COND_EXPR <_20, { 0, 0, 0, 0 }, vect_c_2.7_13>; _21 = VEC_COND_EXPR <_20, ivtmp_17, _19>; This results in in

Re: [Patch, testsuite] PR70227, skip g++.dg/lto/pr69589_0.C on targets without -rdynamic support

2016-05-13 Thread Mike Stump
On May 13, 2016, at 6:53 AM, Jiong Wang wrote: > > This patch skip g++.dg/lto/pr69589_0.C on typical arm & aarch64 > bare-metal targets as they don't support "-rdynamic". > > OK for trunk? Ok.

Re: [PTX] assembler name mangling

2016-05-13 Thread Alexander Monakov
Hello, On Fri, 13 May 2016, Nathan Sidwell wrote: > I've committed this cleanup to use TARGET_MANGLE_DECL_ASSEMBLER_NAME rather > than the ad-hoc solution the ptx backend currently has. I didn't address it > during the earlier set of cleanups as I felt there were more important things > to addre

Re: [PATCH, i386]: Additional fix for PR62599 with -mcmodel=medium -fpic

2016-05-13 Thread Uros Bizjak
On Fri, May 13, 2016 at 9:07 AM, Uros Bizjak wrote: > On Fri, May 13, 2016 at 1:20 AM, H.J. Lu wrote: > testsuite/gcc.target/i386/pr61599-{1,2}.c testcases expose a failure with -mcmodel -fpic, where: /tmp/ccfpoxHY.o: In function `bar': pr61599-2.c:(.text+0xe): relocation

Re: [PATCH PR69848/partial]Propagate comparison into VEC_COND_EXPR if target supports

2016-05-13 Thread Richard Biener
On May 13, 2016 6:02:27 PM GMT+02:00, Bin Cheng wrote: >Hi, >As PR69848 reported, GCC vectorizer now generates comparison outside of >VEC_COND_EXPR for COND_REDUCTION case, as below: > > _20 = vect__1.6_8 != { 0, 0, 0, 0 }; > vect_c_2.8_16 = VEC_COND_EXPR <_20, { 0, 0, 0, 0 }, vect_c_2.7_13>; >

Re: [PATCH, i386]: Additional fix for PR62599 with -mcmodel=medium -fpic

2016-05-13 Thread Uros Bizjak
On Fri, May 13, 2016 at 6:51 PM, Uros Bizjak wrote: > On Fri, May 13, 2016 at 9:07 AM, Uros Bizjak wrote: >> On Fri, May 13, 2016 at 1:20 AM, H.J. Lu wrote: >> > testsuite/gcc.target/i386/pr61599-{1,2}.c testcases expose a failure > with -mcmodel -fpic, where: > > /tmp/ccfpoxHY.o

Re: Allow embedded timestamps by C/C++ macros to be set externally (3)

2016-05-13 Thread Dhole
On 16-05-12 11:16:57, Bernd Schmidt wrote: > On 05/12/2016 02:36 AM, Dhole wrote: > >+ error_at (input_location, "environment variable SOURCE_DATE_EPOCH > >must " > >+"expand to a non-negative integer less than or equal to %wd", > >+MAX_SOURCE_DATE_EPOCH); > > >+/* Th

[PATCH] Use HOST_WIDE_INT_C some more in the i386 backend

2016-05-13 Thread Jakub Jelinek
Hi! I found a couple of spots where we can use the HOST_WIDE_INT_C macro. Bootstrapped/regtested on x86_64-linux and i686-linux, ok for trunk? 2016-05-13 Jakub Jelinek * config/i386/i386.c (ix86_compute_frame_layout, ix86_expand_prologue, ix86_expand_split_stack_prologue): Us

[PATCH] Allow XMM16-XMM31 in vpmaddubsw

2016-05-13 Thread Jakub Jelinek
Hi! This is either AVX2 or for EVEX AVX512BW (& AVX512VL) instruction, thus the patch adds it as a separate alternative guarded with avx512bw isa attribute. Bootstrapped/regtested on x86_64-linux and i686-linux, ok for trunk? 2016-05-13 Jakub Jelinek * config/i386/sse.md (avx2_pmaddu

[PATCH] Fix up vpmulhrsw for -mavx512vl -mno-avx512bw

2016-05-13 Thread Jakub Jelinek
Hi! vpmulhrsw is AVX512BW & AVX512VL insn, so we shouldn't enable it just when AVX512VL is on. Fixed thusly, bootstrapped/regtested on x86_64-linux and i686-linux, ok for trunk? 2016-05-13 Jakub Jelinek * config/i386/sse.md (*_pmulhrsw3): Use constraint x instead of v in seco

[PATCH] Fix up vpshufb for -mavx512vl -mno-avx512bw

2016-05-13 Thread Jakub Jelinek
Hi! vpshufb is AVX512BW & AVX512VL insn, so we shouldn't allow it for AVX512VL only. Bootstrapped/regtested on x86_64-linux and i686-linux, ok for trunk? 2016-05-13 Jakub Jelinek * config/i386/sse.md (_pshufb3): Use constraint x instead of v in second alternative, add avx512b

[PATCH] Fix up vpalignr for -mavx512vl -mno-avx512bw

2016-05-13 Thread Jakub Jelinek
Hi! vpalignr is AVX512BW & VL, so we shouldn't enable it just for VL. Bootstrapped/regtested on x86_64-linux and i686-linux, ok for trunk? 2016-05-13 Jakub Jelinek * config/i386/sse.md (_palignr): Use constraint x instead of v in second alternative, add avx512bw alter

[PATCH] Allow XMM16-XMM31 in vpbroadcast*

2016-05-13 Thread Jakub Jelinek
Hi! These insns are either AVX512VL or AVX512VL & BW, this patch allows using XMM16+ where possible. Bootstrapped/regtested on x86_64-linux and i686-linux, ok for trunk? 2016-05-13 Jakub Jelinek * config/i386/sse.md (pbroadcast_evex_isa): New mode attr. (avx2_pbroadcast): Add

Re: [PATCH, i386]: Additional fix for PR62599 with -mcmodel=medium -fpic

2016-05-13 Thread H.J. Lu
On Fri, May 13, 2016 at 9:51 AM, Uros Bizjak wrote: > On Fri, May 13, 2016 at 9:07 AM, Uros Bizjak wrote: >> On Fri, May 13, 2016 at 1:20 AM, H.J. Lu wrote: >> > testsuite/gcc.target/i386/pr61599-{1,2}.c testcases expose a failure > with -mcmodel -fpic, where: > > /tmp/ccfpoxHY.o

[PATCH] Fix --enable-checking=fold bootstrap (PR bootstrap/71071)

2016-05-13 Thread Jakub Jelinek
Hi! Since a recent change, TYPE_ALIAS_SET of types can change during folding. This patch arranges for it not to be checksummed. Bootstrapped/regtested on x86_64-linux and i686-linux (normal bootstrap) and built (nonbootstrap) + regtested the --enable-checking=yes,extra,fold,rtl version (previousl

Re: [PATCH, i386]: Additional fix for PR62599 with -mcmodel=medium -fpic

2016-05-13 Thread Uros Bizjak
On Fri, May 13, 2016 at 7:20 PM, H.J. Lu wrote: > On Fri, May 13, 2016 at 9:51 AM, Uros Bizjak wrote: >> On Fri, May 13, 2016 at 9:07 AM, Uros Bizjak wrote: >>> On Fri, May 13, 2016 at 1:20 AM, H.J. Lu wrote: >>> >> testsuite/gcc.target/i386/pr61599-{1,2}.c testcases expose a failure >>

Re: [PATCH] Use HOST_WIDE_INT_C some more in the i386 backend

2016-05-13 Thread Uros Bizjak
On Fri, May 13, 2016 at 7:11 PM, Jakub Jelinek wrote: > Hi! > > I found a couple of spots where we can use the HOST_WIDE_INT_C macro. > > Bootstrapped/regtested on x86_64-linux and i686-linux, ok for trunk? > > 2016-05-13 Jakub Jelinek > > * config/i386/i386.c (ix86_compute_frame_layout

[PATCH, testsuite]: Hancle AVX in gcc.dg/vect/tree-vect.h

2016-05-13 Thread Uros Bizjak
Also remove unneeded XOP handling. 2016-05-13 Uros Bizjak * gcc.dg/vect/tree-vect.h (check_vect): Handle AVX2, remove XOP handling. Tested on x86_64-linux-gnu AVX target and committed to mainline SVN. Patch will be backported to other release branches. Uros. Index: gcc.dg/vect/tree-

Re: [PATCH, PR71084] Further improve of CFG change tracking in CSE

2016-05-13 Thread Bernhard Reutner-Fischer
On May 13, 2016 11:50:33 AM GMT+02:00, Richard Biener wrote: >On Fri, May 13, 2016 at 11:03 AM, Ilya Enkovich > wrote: >> Hi, >> >> This patch improves cse_cfg_altered computation by taking into >account >> cleanup_cfg returned values. This resolves another case of >invalidated >> dominance info

  1   2   >