Re: Committed: skip gcc.dg/pr46309.c for CRIS

2014-01-10 Thread Hans-Peter Nilsson
> From: Mike Stump > Date: Sat, 11 Jan 2014 01:55:09 +0100 > On Jan 10, 2014, at 11:26 AM, Hans-Peter Nilsson > wrote: > > This patch "fixes" a regression on trunk and the 4.8 branch: > > > Index: gcc/testsuite/gcc.dg/pr46309.c > > ==

Re: Committed: skip gcc.dg/pr46309.c for CRIS

2014-01-10 Thread Mike Stump
On Jan 10, 2014, at 11:26 AM, Hans-Peter Nilsson wrote: > This patch "fixes" a regression on trunk and the 4.8 branch: > Index: gcc/testsuite/gcc.dg/pr46309.c > === > --- gcc/testsuite/gcc.dg/pr46309.c(revision 206534) > +++ gcc

Re: [PATCH] Fix unaligned access generated by IVOPTS

2014-01-10 Thread Eric Botcazou
[Sorry for dropping the ball here] > I think that may_be_unaligned_p is just seriously out-dated ... shouldn't it > be sth like > > get_object_alignment_1 (ref, &align, &bitpos); > if step * BITS_PER_UNIT + bitpos is misaligned > ... > > or rather all this may_be_unaligned_p stuff should be

Re: [GOOGLE] Move LIPO linking before profile annotation

2014-01-10 Thread Xinliang David Li
Ok. David On Fri, Jan 10, 2014 at 1:50 PM, Dehao Chen wrote: > This patch moves the LIPO linking before profile annotation so that > iterative-early-inline can cover functions from aux-module. > > Bootstrapped and passed regression test and benchmark test. > > OK for google-4_8? > > Thanks, > De

Re: [RFA] [PATCH][PR tree-optimization/59749] Fix recently introduced ree bug

2014-01-10 Thread Jeff Law
On 01/10/14 14:52, Jakub Jelinek wrote: There is one thing I still worry about, if some target has an insn to say sign extend or zero extend a short memory load into HARD_REGNO_NREGS () > 1 register, but the other involved register has the only one (or fewer) hard registers available to it. Cons

PATCH: Add PROCESSOR_INTEL

2014-01-10 Thread H.J. Lu
Hi, This patch adds PROCESSOR_INTEL, which is almost identical to PROCESSOR_SILVERMONT, except that __tune_slm__/__tune_silvermont__ aren't defined for -mtune=intel. OK for trunk? Thanks. H.J. --- 2014-01-10 H.J. Lu * config/i386/i386-c.c (ix86_target_macros_internal): Handle

Re: [RFA][PATCH][PR middle-end/59743] Fix ree improvement when defining insn is after use insn

2014-01-10 Thread Jeff Law
On 01/10/14 15:03, Jakub Jelinek wrote: On Fri, Jan 10, 2014 at 02:58:59PM -0700, Jeff Law wrote: +2014-01-10 Jeff Law + + PR middle-end/59743 + * ree.c (combine_reaching_defs): Ensure the defining statement + occurs before the extension when optimizing extensions with +

Re: [RFA][PATCH][PR middle-end/59743] Fix ree improvement when defining insn is after use insn

2014-01-10 Thread Jakub Jelinek
On Fri, Jan 10, 2014 at 02:58:59PM -0700, Jeff Law wrote: > +2014-01-10 Jeff Law > + > + PR middle-end/59743 > + * ree.c (combine_reaching_defs): Ensure the defining statement > + occurs before the extension when optimizing extensions with > + different source and destination har

[RFA][PATCH][PR middle-end/59743] Fix ree improvement when defining insn is after use insn

2014-01-10 Thread Jeff Law
As mentioned in the PR, we have a case where within a block the reaching def (defining insn) occurs after the use (extension insn). This obviously can only happen in a loop and AFAICT should only happen for an uninitialized use. Anyway, in this situation we'd segfault in reg_used_between_p

Re: [RFA] [PATCH][PR tree-optimization/59749] Fix recently introduced ree bug

2014-01-10 Thread Jakub Jelinek
On Fri, Jan 10, 2014 at 02:36:47PM -0700, Jeff Law wrote: > As mentioned in the PR. We have a memory load and two extensions. > > The first extension requires a copy because its source and > destination are not the same hard register. > > The second extension does not require a copy and has a wi

[GOOGLE] Move LIPO linking before profile annotation

2014-01-10 Thread Dehao Chen
This patch moves the LIPO linking before profile annotation so that iterative-early-inline can cover functions from aux-module. Bootstrapped and passed regression test and benchmark test. OK for google-4_8? Thanks, Dehao Index: gcc/auto-profile.c

PR ipa/58585 (virtual inheritance ICE)

2014-01-10 Thread Jan Hubicka
Hi, this patch fixes checking ICE in the attached testcase that happens because we end up devirtualizing into a function that is not withing possible polymorphic call target. The defauled analysis is in the PR and also in comment bellow. The basic problem is that we build type inheritance tree

[RFA] [PATCH][PR tree-optimization/59749] Fix recently introduced ree bug

2014-01-10 Thread Jeff Law
As mentioned in the PR. We have a memory load and two extensions. The first extension requires a copy because its source and destination are not the same hard register. The second extension does not require a copy and has a wider mode than the first extension. In this case we have to make

[PATCH,rs6000,committed] Fix pasto in rs6000-builtin.def

2014-01-10 Thread Bill Schmidt
I happened to notice an obvious error in one of the builtin tables. Bootstrapped and tested on powerpc64{,le}-unknown-linux-gnu with no regressions. Committed as obvious. Thanks, Bill 2014-01-10 Bill Schmidt * config/rs6000/rs6000-builtin.def: Fix pasto for VPKSDUS. Index: gcc/con

Re: [PATCH] Don't perform the REGNO != REGNO REE for modes other than scalar integral (PR rtl-optimization/59754)

2014-01-10 Thread Jeff Law
On 01/10/14 13:45, Jakub Jelinek wrote: Hi! The new REGNO != REGNO optimization assumes that a lowpart subreg of the extended reg will be the value of the original expression. But that is the case of only scalar integral modes, e.g. for vector mode extensions a lowpart subreg will occupy just sa

[PATCH] Don't perform the REGNO != REGNO REE for modes other than scalar integral (PR rtl-optimization/59754)

2014-01-10 Thread Jakub Jelinek
Hi! The new REGNO != REGNO optimization assumes that a lowpart subreg of the extended reg will be the value of the original expression. But that is the case of only scalar integral modes, e.g. for vector mode extensions a lowpart subreg will occupy just say half or even fewer vector elements, but

[committed] Fix predcom (PR tree-optimization/59745)

2014-01-10 Thread Jakub Jelinek
Hi! split_data_refs_to_components used the name_expansions affine cache through determine_offset, and since my patch uses it even more often, but if it returns NULL, we don't free the cache and it can contain garbage next time we perform tree_predictive_commoning_loop. Bootstrapped/regtested on x

[C++ PATCH] Emit array initialization from ctor as loop if possible (PR c++/59659)

2014-01-10 Thread Jakub Jelinek
Hi! The following patch attempts to improve both compile time and generated code for array initializers if there are many consecutive same elements. Without the patch even T1 instead of T10 takes a minute to compile and generates hundreds of kilobytes large routine. Without the patch, we

[PATCH] Avoid introducing undefined behavior in sccp (PR tree-optimization/59387)

2014-01-10 Thread Jakub Jelinek
Hi! If folded_casts is true, sccp can introduce undefined behavior even when there was none in the original loop, e.g. all actual additions performed in unsigned type and then cast back to signed. The following patch fixes that by turning the arithmetic stmts added by sccp use unsigned operations

Committed: skip gcc.dg/pr46309.c for CRIS

2014-01-10 Thread Hans-Peter Nilsson
This patch "fixes" a regression on trunk and the 4.8 branch: Running /tmp/hpautotest-gcc48/gcc/gcc/testsuite/gcc.dg/dg.exp ... ... FAIL: gcc.dg/pr46309.c scan-tree-dump-times reassoc2 "Optimizing range tests [^\r\n]*_[0-9]* -.0, 31. and -.128, 159.[\n\r]* into" 1 The comment in the test seen in

Re: [PATCH, AARCH64] : Fix target/59744 - invalid use of CMN

2014-01-10 Thread Andrew Pinski
On Fri, Jan 10, 2014 at 9:38 AM, Richard Earnshaw wrote: > On 10/01/14 17:37, Andrew Pinski wrote: >> On Fri, Jan 10, 2014 at 7:14 AM, Richard Earnshaw wrote: >>> It's incorrect to use CMN to compare with a negated operand if the >>> following condition is an inequality. This is because of bound

Re: [PATCH, AARCH64] : Fix target/59744 - invalid use of CMN

2014-01-10 Thread Andrew Pinski
On Fri, Jan 10, 2014 at 7:14 AM, Richard Earnshaw wrote: > It's incorrect to use CMN to compare with a negated operand if the > following condition is an inequality. This is because of boundary > conditions when the negated operations overflow (or when zero), since > the flags are then not the sw

Re: [PATCH, AARCH64] : Fix target/59744 - invalid use of CMN

2014-01-10 Thread Richard Earnshaw
On 10/01/14 17:37, Andrew Pinski wrote: > On Fri, Jan 10, 2014 at 7:14 AM, Richard Earnshaw wrote: >> It's incorrect to use CMN to compare with a negated operand if the >> following condition is an inequality. This is because of boundary >> conditions when the negated operations overflow (or when

Re: [PATCH] Allocate all target globals using GC for SWITCHABLE_TARGETs

2014-01-10 Thread Richard Henderson
On 01/09/2014 03:34 PM, Jakub Jelinek wrote: > 2014-01-09 Jakub Jelinek > > * target-globals.c (save_target_globals): Allocate < 4KB structs using > GC in payload of target_globals struct instead of allocating them on > the heap and the larger structs separately using GC. >

Re: [PATCH] Fix ifcvt (PR rtl-optimization/58668)

2014-01-10 Thread Christophe Lyon
On 10 January 2014 17:45, Jakub Jelinek wrote: > On Fri, Jan 10, 2014 at 05:44:22PM +0100, Christophe Lyon wrote: >> It seems this patch causes several regressions in gfortran on ARM too: >> gfortran.dg/default_format_1.f90 >> gfortran.dg/default_format_denormal_1.f90 >> gfortran.dg/fmt_bz_bn.f >>

Re: [PATCH] Fix ifcvt (PR rtl-optimization/58668)

2014-01-10 Thread Jakub Jelinek
On Fri, Jan 10, 2014 at 05:44:22PM +0100, Christophe Lyon wrote: > It seems this patch causes several regressions in gfortran on ARM too: > gfortran.dg/default_format_1.f90 > gfortran.dg/default_format_denormal_1.f90 > gfortran.dg/fmt_bz_bn.f > gfortran.dg/fmt_read_bz_bn.f90 > gfortran.dg/g77/f77-e

Fwd: [PATCH] Prevent crash when expand movstr fail when movstr pattern is defined

2014-01-10 Thread Kito Cheng
Hi, expand_movstr is work fine when we don't define movstr pattern or always expand it successfully, however it's will crash when if movstr expand fail since expand_insn expect always expand successfully (it's place a gcc_unreachable() when expand fail). this patch use maybe_expand_insn instead o

Re: [PATCH][IRA] Analysis of register usage of functions for usage by IRA.

2014-01-10 Thread Tom de Vries
On 10-01-14 12:39, Richard Earnshaw wrote: On 09/01/14 20:42, Tom de Vries wrote: On 09-01-14 15:41, Richard Earnshaw wrote: On 30/03/13 16:10, Tom de Vries wrote: On 29/03/13 13:54, Tom de Vries wrote: I split the patch up into 10 patches, to facilitate further review: ... 0001-Add-command-l

Re: [PATCH] Fix ifcvt (PR rtl-optimization/58668)

2014-01-10 Thread Christophe Lyon
Hello, It seems this patch causes several regressions in gfortran on ARM too: gfortran.dg/default_format_1.f90 gfortran.dg/default_format_denormal_1.f90 gfortran.dg/fmt_bz_bn.f gfortran.dg/fmt_read_bz_bn.f90 gfortran.dg/g77/f77-edit-t-in.f gfortran.dg/list_read_4.f90 gfortran.dg/namelist_11.f gfor

Re: [PATCH] Fix PR59715

2014-01-10 Thread Tom de Vries
On 09-01-14 13:33, Richard Biener wrote: On Thu, 9 Jan 2014, Tom de Vries wrote: On 09-01-14 10:16, Richard Biener wrote: This fixes PR59715 by splitting critical edges again before code sinking. The critical edge splitting done before PRE was designed to survive until sinking originally, bu

Re: [PATCH i386 10/8] [AVX512] Add missing AVX-512ER patterns, intrinsics, tests.

2014-01-10 Thread Jakub Jelinek
On Fri, Jan 10, 2014 at 07:20:38PM +0300, Kirill Yukhin wrote: > @@ -28920,6 +28927,7 @@ static const struct builtin_description > bdesc_special_args[] = >{ OPTION_MASK_ISA_AVX512F, CODE_FOR_avx512f_movntv16sf, > "__builtin_ia32_movntps512", IX86_BUILTIN_MOVNTPS512, UNKNOWN, (int) > VOID_FTY

[PATCH i386 10/8] [AVX512] Add missing AVX-512ER patterns, intrinsics, tests.

2014-01-10 Thread Kirill Yukhin
Hello, It seems that we miss few more intrinsics. I've also added some missing substed predicates Also I've fixed bogus rcp14 pattern and removeed some redundant subst attributes. Bootstrapped. New & existing tests pass (expcept for those mentioned in PR about REE). Is it ok for trunk? gcc/

Re: [ARM] Fix incorrect restore of FP registers in nested APCS frame

2014-01-10 Thread Eric Botcazou
> > /* { dg-do run } */ > > /* { dg-require-effective-target arm_neon_hw } */ > > /* { dg-options "-fno-omit-frame-pointer -mapcs-frame -O" } */ > > /* { dg-add-options arm_neon } */ > > Yes, something like that. OK, installed after checking that this doesn't change the test, thanks. -- Eric Bo

Re: libsanitizer merge from upstream r196090

2014-01-10 Thread Jakub Jelinek
On Fri, Jan 10, 2014 at 03:50:33PM +0400, Maxim Ostapenko wrote: > On Fri, Jan 10, 2014 at 10:39 AM, Jakub Jelinek wrote: > > > Some of the tsan tests seems to FAIL randomly for quite a while > > (since they were added), didn't have time to look if it is just > bugs in the test or > > some compil

[PATCH, AARCH64] : Fix target/59744 - invalid use of CMN

2014-01-10 Thread Richard Earnshaw
It's incorrect to use CMN to compare with a negated operand if the following condition is an inequality. This is because of boundary conditions when the negated operations overflow (or when zero), since the flags are then not the swapped version of the comparison. This patch restricts the pattern

[Committed] S/390: expand_tbegin remove jump over cc extraction

2014-01-10 Thread Andreas Krebbel
Hi, with the attached patch the jump over the extraction of CC into a GPR for the good case (CC0) is removed. Starting with tbegin being flagged as "returns_twice" this generates slow code and prevents the backend optimization for nonescaping transaction from working correctly. This fixes the fo

Re: libsanitizer merge from upstream r196090

2014-01-10 Thread Uros Bizjak
On Fri, Jan 10, 2014 at 10:23 AM, Uros Bizjak wrote: > On Mon, Dec 2, 2013 at 4:49 PM, Uros Bizjak wrote: > Does it support using libbacktrace in GCC? >>> >>> Not on it's own, but the support in the upstream maintained files >>> is there, so hopefully it will be just a matter of follow-up pa

[patch] fix docs (libstdc++/59698, libstdc++/59687, libstdc++/59699)

2014-01-10 Thread Jonathan Wakely
Three documentation bugs fixed by the attached patches 2014-01-10 Jonathan Wakely PR libstdc++/59698 * doc/xml/manual/status_cxx1998.xml (iso.1998.specific): Markup and stylistic improvements. * doc/xml/manual/codecvt.xml (std.localization.facet.codecvt): Likewise and updat

Re: [ARM] Fix incorrect restore of FP registers in nested APCS frame

2014-01-10 Thread Richard Earnshaw
On 10/01/14 12:28, Eric Botcazou wrote: >> Can you not use (or add something similar to) { dg-add-options arm_neon }? > > Probably, yes, something like: > > /* { dg-do run } */ > /* { dg-require-effective-target arm_neon_hw } */ > /* { dg-options "-fno-omit-frame-pointer -mapcs-frame -O" } */ > /

Re: [PATCH, SH] Improve builtin strnlen for small lengths

2014-01-10 Thread kkojima
Christian Bruel wrote: > This patch unrolls string compare for length < 8 and residual bytes > after the word at a time loops (with cmp/str), using base+offset > addressing mode. > It also allows the builtin to be inlined for non-constant lengths. > > No new regressions. Upgraded test case to han

[PATCH, SH] Improve builtin strnlen for small lengths

2014-01-10 Thread Christian Bruel
Hello, This patch unrolls string compare for length < 8 and residual bytes after the word at a time loops (with cmp/str), using base+offset addressing mode. It also allows the builtin to be inlined for non-constant lengths. No new regressions. Upgraded test case to handle former case. OK for tru

Re: [ARM] Fix incorrect restore of FP registers in nested APCS frame

2014-01-10 Thread Eric Botcazou
> Can you not use (or add something similar to) { dg-add-options arm_neon }? Probably, yes, something like: /* { dg-do run } */ /* { dg-require-effective-target arm_neon_hw } */ /* { dg-options "-fno-omit-frame-pointer -mapcs-frame -O" } */ /* { dg-add-options arm_neon } */ -- Eric Botcazou

[PATCH] Fix PR59374 for real

2014-01-10 Thread Richard Biener
Dependence analysis in the vectorizer assumes that all references take part in vectorization - which is of course the case for loop-based vectorization. For BB vectorization unrelated DRs can be inbetween vectorized stmts though. The original fix properly verified that when accounting DDRs that

Re: maybe_fold_stmt (was: [gomp4] #pragma omp target* fixes)

2014-01-10 Thread Jakub Jelinek
On Fri, Jan 10, 2014 at 12:51:10PM +0100, Thomas Schwinge wrote: > > > +static bool > > > +maybe_fold_stmt (gimple_stmt_iterator *gsi) > > > +{ > > > + bool changed = false; > > > + struct gimplify_omp_ctx *ctx; > > > + for (ctx = gimplify_omp_ctxp; ctx; ctx = ctx->outer_context) > > > +if (

Re: [ARM] Fix incorrect restore of FP registers in nested APCS frame

2014-01-10 Thread Richard Earnshaw
On 10/01/14 11:48, Eric Botcazou wrote: >> Some tests can do it because they don't produce an executable. > > OK, there is some related dg-skip-if magic in 20090811-1.c, maybe Christophe > can try that too. > Can you not use (or add something similar to) { dg-add-options arm_neon }? R.

Re: lto-plugin: mismatch between ld's architecture and GCC's configure --host

2014-01-10 Thread Thomas Schwinge
Hi! Ping, after another month. I've only received a private note from one build machinery manintainer who found this beyond his level of expertise, and wished me luck to find someone else to review. Any takers in the new year? On Sat, 14 Dec 2013 13:12:28 +0100, I wrote: > Ping, after another m

Re: maybe_fold_stmt (was: [gomp4] #pragma omp target* fixes)

2014-01-10 Thread Thomas Schwinge
Hi! Ping. On Thu, 19 Dec 2013 17:44:25 +0100, I wrote: > Ping. > > On Fri, 13 Dec 2013 11:13:03 +0100, I wrote: > > On Thu, 5 Sep 2013 18:11:05 +0200, Jakub Jelinek wrote: > > > 4) the reference testcase showed a problem with fold_stmt calls > > > we do very early, during gimplification, becaus

Re: libsanitizer merge from upstream r196090

2014-01-10 Thread Maxim Ostapenko
Hi all, On Fri, Jan 10, 2014 at 10:39 AM, Jakub Jelinek wrote: > Some of the tsan tests seems to FAIL randomly for quite a while > (since they were added), didn't have time to look if it is just bugs in the test or > some compiler issue or library issue. When I've commited these tsan tests,

Re: [ARM] Fix incorrect restore of FP registers in nested APCS frame

2014-01-10 Thread Eric Botcazou
> Some tests can do it because they don't produce an executable. OK, there is some related dg-skip-if magic in 20090811-1.c, maybe Christophe can try that too. -- Eric Botcazou

Re: [gomp4] OpenACC structured blocks (was: PING: Fwd: Re: [patch] implement Cilk Plus simd loops on trunk)

2014-01-10 Thread Thomas Schwinge
Hi! Ping. On Thu, 19 Dec 2013 17:49:09 +0100, I wrote: > Ping. > > On Tue, 10 Dec 2013 13:53:39 +0100, I wrote: > > At the end of this email you can find the patch that I'd like to apply to > > gomp-4_0-branch for OpenACC structured blocks, after the following two > > have been approved: > > >

Re: [ARM] Fix incorrect restore of FP registers in nested APCS frame

2014-01-10 Thread Richard Earnshaw
On 10/01/14 09:46, Eric Botcazou wrote: >> This new test fails when GCC is configured as target >> arm-none-linux-gnueabihf and --with-float=hard: >> tools/arm-none-linux-gnueabihf/bin/ld: error: ./neon-nested-apcs.exe >> uses VFP register arguments, /tmp/cc9LLqES.o does not >> tools/arm-none-linux

Re: [PATCH][IRA] Analysis of register usage of functions for usage by IRA.

2014-01-10 Thread Richard Earnshaw
On 09/01/14 20:42, Tom de Vries wrote: > On 09-01-14 15:41, Richard Earnshaw wrote: >> On 30/03/13 16:10, Tom de Vries wrote: >>> On 29/03/13 13:54, Tom de Vries wrote: I split the patch up into 10 patches, to facilitate further review: ... 0001-Add-command-line-option.patch 000

Re: [PATCH][ARM] Get mode for rtx costs calculations for SET RTX from destination reg

2014-01-10 Thread Richard Earnshaw
On 09/01/14 17:42, Kyrill Tkachov wrote: > Hi all, > > SET RTXs don't have a mode, so the code to calculate a reg-to-reg set in the > arm > rtx costs function needs to get the mode from one of the registers involved. > We > already did that when the source is a CONST_INT. > > This patch fixes

Re: [PATCH][ARM] Add CRC32 to the feature flags of Cortex-A53, A57

2014-01-10 Thread Richard Earnshaw
On 09/01/14 17:37, Kyrill Tkachov wrote: > Hi all, > > The Cortex-A53 and Cortex-A57 processors support the CRC32 extensions to > ARMv8-a, so we specify that in their definitions in arm-cores.def. > This also updates their big.LITTLE amalgamation and removes the redundant > FL_THUMB_DIV and FL_A

Re: [PATCH][testsuite][ARM] Properly figure -mfloat-abi option for crypto tests

2014-01-10 Thread Richard Earnshaw
On 09/01/14 17:02, Kyrill Tkachov wrote: > Hi all, > > When adding the testsuite options for the crypto tests we need to make sure > that > don't end up adding -mfloat-abi=softfp to a hard-float target like > arm-none-linux-gnueabihf. This patch adds that code to figure out which > -mfpu/-mflo

Re: [PATCH][ARM] Fix arm_init_iwmmxt_builtins to handle only iwmmxt entries

2014-01-10 Thread Richard Earnshaw
On 09/01/14 17:34, Kyrill Tkachov wrote: > Hi all, > > After my CRC32 intrinsics patch that added new entries into the bdesc_2arg > table, the arm_init_iwmmxt_builtins function tries to iterate over them and > blows up, causing an ICE when trying to compile with -mcpu=iwmmxt. > > This patch fix

Re: Resolve pr44194-1.c failure by refining scan-rtl-dump-not pattern

2014-01-10 Thread Renlin Li
Hi Eric, Thank you for your suggestion! I have made the adjustment and test it. Please check the new attachment. Regards, Renlin Li On 10/01/14 10:42, Eric Botcazou wrote: This patch will resolve testsuite/gcc.dg/pr44194-1.c failure observed in aarch64 target by refining the second scan-rtl-du

Re: libsanitizer merge from upstream r196090

2014-01-10 Thread Uros Bizjak
On Fri, Jan 10, 2014 at 10:39 AM, Jakub Jelinek wrote: > On Fri, Jan 10, 2014 at 10:23:59AM +0100, Uros Bizjak wrote: >> I was able to compile sanitizer with r206475 (after Jakub's fixes) and >> run the testsuite. However, on 64bit target, I got some failures in >> asan and several timeouts in tsa

Re: [PATCH i386 AVX-512] Fix conditions for 2 patterns.

2014-01-10 Thread Uros Bizjak
On Fri, Jan 10, 2014 at 11:07 AM, Kirill Yukhin wrote: > Hello, > We've [at least] 2 patterns with wrong conditions. > > Bootstrapped. > > AVX-512 testsuite more complicated. > Few tests are fail, and it seems that not because of > the patch. I am trying to find guilty commit. > > Failing this tes

Re: [PATCH i386 AVX-512] Fix conditions for 2 patterns.

2014-01-10 Thread Kirill Yukhin
On 10 Jan 13:07, Kirill Yukhin wrote: > Few tests are fail, and it seems that not because of > the patch. I am trying to find guilty commit. I filed PR59754 (http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59754) -- Thanks, K

Re: [Patch] Fix PR plugin/59335, plugins not compiling

2014-01-10 Thread Richard Biener
On Thu, Jan 9, 2014 at 11:00 PM, Steve Ellcey wrote: > Some header files needed by plugins are not present in the installed plugin > include directory. This patch adds those headers. I added all the headers > I needed for my plugin plus the ones mentioned in the patch. I did not add > the x86 s

Re: [PATCH, AARCH64] MULTIARCH_DIRNAME breaks multiarch build

2014-01-10 Thread Matthias Klose
Am 10.01.2014 10:49, schrieb Zhenqiang Chen: > On 10 January 2014 17:23, Matthias Klose wrote: >> Am 10.01.2014 09:23, schrieb Zhenqiang Chen: >>> Hi, >>> >>> MULTIARCH_DIRNAME was removed @r196649 since the dir info had been >>> combined in MULTILIB_OSDIRNAMES. >>> >>> But MULTIARCH_DIRNAME was r

Re: Resolve pr44194-1.c failure by refining scan-rtl-dump-not pattern

2014-01-10 Thread Eric Botcazou
> This patch will resolve testsuite/gcc.dg/pr44194-1.c failure observed in > aarch64 target by refining the second scan-rtl-dump-not pattern. > > For aarch64 target, the following barrier is generated which will > exactly match the current pattern > "insn \[^\n\]*set \\(mem". This is undesired. By

Resolve pr44194-1.c failure by refining scan-rtl-dump-not pattern

2014-01-10 Thread Renlin Li
Hi all, This patch will resolve testsuite/gcc.dg/pr44194-1.c failure observed in aarch64 target by refining the second scan-rtl-dump-not pattern. For aarch64 target, the following barrier is generated which will exactly match the current pattern "insn \[^\n\]*set \\(mem". This is undesired. B

[PATCH i386 AVX-512] Fix conditions for 2 patterns.

2014-01-10 Thread Kirill Yukhin
Hello, We've [at least] 2 patterns with wrong conditions. Bootstrapped. AVX-512 testsuite more complicated. Few tests are fail, and it seems that not because of the patch. I am trying to find guilty commit. Failing this tests: avx512f-vpmovsxbd-2.c and other *-extension insns. gcc/ * co

Re: [PATCH, AARCH64] MULTIARCH_DIRNAME breaks multiarch build

2014-01-10 Thread Zhenqiang Chen
On 10 January 2014 17:23, Matthias Klose wrote: > Am 10.01.2014 09:23, schrieb Zhenqiang Chen: >> Hi, >> >> MULTIARCH_DIRNAME was removed @r196649 since the dir info had been >> combined in MULTILIB_OSDIRNAMES. >> >> But MULTIARCH_DIRNAME was re-added @r201164. With this change, the >> final multi

Re: [ARM] Fix incorrect restore of FP registers in nested APCS frame

2014-01-10 Thread Eric Botcazou
> This new test fails when GCC is configured as target > arm-none-linux-gnueabihf and --with-float=hard: > tools/arm-none-linux-gnueabihf/bin/ld: error: ./neon-nested-apcs.exe > uses VFP register arguments, /tmp/cc9LLqES.o does not > tools/arm-none-linux-gnueabihf/bin/ld: failed to merge target spe

Re: libsanitizer merge from upstream r196090

2014-01-10 Thread Jakub Jelinek
On Fri, Jan 10, 2014 at 10:23:59AM +0100, Uros Bizjak wrote: > I was able to compile sanitizer with r206475 (after Jakub's fixes) and > run the testsuite. However, on 64bit target, I got some failures in > asan and several timeouts in tsan [1]. Some of the tsan tests seems to FAIL randomly for qui

Re: [PATCH, AARCH64] MULTIARCH_DIRNAME breaks multiarch build

2014-01-10 Thread Matthias Klose
Am 10.01.2014 09:23, schrieb Zhenqiang Chen: > Hi, > > MULTIARCH_DIRNAME was removed @r196649 since the dir info had been > combined in MULTILIB_OSDIRNAMES. > > But MULTIARCH_DIRNAME was re-added @r201164. With this change, the > final multiarch_dir is combined as > "aarch64-linux-gnu:aarch64-lin

Re: libsanitizer merge from upstream r196090

2014-01-10 Thread Uros Bizjak
On Mon, Dec 2, 2013 at 4:49 PM, Uros Bizjak wrote: >>> Does it support using libbacktrace in GCC? >> >> Not on it's own, but the support in the upstream maintained files >> is there, so hopefully it will be just a matter of follow-up patch >> with configury/Makefile etc. stuff, I'll work on it on

Re: [MIPS, committed] Backport bswap patches to 4.8

2014-01-10 Thread Richard Sandiford
Richard Biener writes: > On Thu, Jan 9, 2014 at 9:06 PM, Richard Sandiford > wrote: >> I got an off-list request to backport the bswap patterns to 4.8. >> They've been in trunk for a while without any problems being reported >> and they should be relatively safe. >> >> Here's what I applied after

Re: RFC Asan instrumentation control

2014-01-10 Thread Maxim Ostapenko
On 01/10/2014 12:36 PM, Jakub Jelinek wrote: On Fri, Jan 10, 2014 at 12:34:49PM +0400, Maxim Ostapenko wrote: Thanks, got it. Is it OK now? Yes, thanks. 2014-01-10 Max Ostapenko * c-c++-common/asan/no-asan-stack.c: New test. Jakub Commited in 206515. -Maxim.

Re: [ARM] Fix incorrect restore of FP registers in nested APCS frame

2014-01-10 Thread Christophe Lyon
Hi, > > 2013-12-22 Eric Botcazou > > * gcc.target/arm/neon-nested-apcs.c: New test. > This new test fails when GCC is configured as target arm-none-linux-gnueabihf and --with-float=hard: tools/arm-none-linux-gnueabihf/bin/ld: error: ./neon-nested-apcs.exe uses VFP register arguments, /tm

Re: [MIPS, committed] Backport bswap patches to 4.8

2014-01-10 Thread Richard Biener
On Thu, Jan 9, 2014 at 9:06 PM, Richard Sandiford wrote: > I got an off-list request to backport the bswap patterns to 4.8. > They've been in trunk for a while without any problems being reported > and they should be relatively safe. > > Here's what I applied after testing on mips64-linux-gnu (wit

Re: [PATCH] MIPS: improve Loongson-2E/2F/3A detection for -march=native

2014-01-10 Thread Richard Sandiford
Huacai Chen writes: > For human-readability, I submit a patch to Linux kernel to display > Loongson-2E/2F/3A in /proc/cpuinfo. But that break -march=native, this > patch fix that. Thanks, applied to trunk, 4.8 and 4.7 with a minor formatting tweak. Richard

Re: [GOMP4] Patch to add option for offloading

2014-01-10 Thread Andrey Turetskiy
Ping On Wed, Dec 18, 2013 at 4:28 PM, Andrey Turetskiy wrote: > Ping > > On Fri, Dec 13, 2013 at 3:09 PM, Andrey Turetskiy > wrote: >> Hi, >> I've added option -foffload-target to specify target and options for >> target compiler for offloading. >> Please, have a look. >> >> -- >> Best regards,

Re: [PING^2][PATCH] -fuse-caller-save - Implement TARGET_FN_OTHER_HARD_REG_USAGE hook for MIPS

2014-01-10 Thread Richard Sandiford
Tom de Vries writes: >> Why not just collect the usage information at >> the end of final rather than at the beginning, so that all splits during >> final have been done? > > If we have a call to a leaf function, the final rtl representation does not > contain calls. The problem does not lie in

Re: RFC Asan instrumentation control

2014-01-10 Thread Jakub Jelinek
On Fri, Jan 10, 2014 at 12:34:49PM +0400, Maxim Ostapenko wrote: > Thanks, got it. Is it OK now? Yes, thanks. > 2014-01-10 Max Ostapenko > > * c-c++-common/asan/no-asan-stack.c: New test. Jakub

Re: RFC Asan instrumentation control

2014-01-10 Thread Maxim Ostapenko
If you want to limit to x86_64-linux only, please do: target { { i?86-*-linux* x86_64-*-linux* } && lp64 } instead. Also, what advantages do you see for trying to assemble the result? If you instead just do dg-do compile, you can drop -save-temps from dg-options and /* { dg-final { cleanup-saved

[PATCH, AARCH64] MULTIARCH_DIRNAME breaks multiarch build

2014-01-10 Thread Zhenqiang Chen
Hi, MULTIARCH_DIRNAME was removed @r196649 since the dir info had been combined in MULTILIB_OSDIRNAMES. But MULTIARCH_DIRNAME was re-added @r201164. With this change, the final multiarch_dir is combined as "aarch64-linux-gnu:aarch64-linux-gnu", which is incorrect and leads to multiarch build fail