Re: [v3 PATCH] Reduce the size of variant, it doesn't need an index of type size_t internally.

2017-01-10 Thread Tim Shen via gcc-patches
On Tue, Jan 10, 2017 at 2:19 PM, Ville Voutilainen wrote: > Cleanups based on review; there's no longer any public typedefs added > to variant, > and the test is greatly simpler with much less trickery. Looks good to me. Thanks! -- Regards, Tim Shen

[PATCH v2] Offer suggestions for unrecognized sanitizer options (PR driver/78877)

2017-01-10 Thread David Malcolm
On Tue, 2017-01-10 at 20:49 +0100, Jakub Jelinek wrote: > On Tue, Jan 10, 2017 at 03:02:49PM -0500, David Malcolm wrote: > > +/* Given ARG, an unrecognized sanitizer option, return the best > > + matching sanitizer option, or NULL if there isn't one. */ > > + > > +static const char * > >

Re: [patch mips/gcc] add build-time and runtime options to disable or set madd.fmt type

2017-01-10 Thread Sandra Loosemore
On 01/10/2017 07:24 AM, Yunqiang Su wrote: Hi, folks, any idea about this patch? I can only comment on the documentation parts. diff --git a/gcc/doc/install.texi b/gcc/doc/install.texi index b911d76dd66..e548733537d 100644 --- a/gcc/doc/install.texi +++ b/gcc/doc/install.texi @@ -1338,6

Re: [patch] Fix wrong code for return of small aggregates on big-endian

2017-01-10 Thread Eric Botcazou
> I have noticed new failures after this commit (r244249). > g++.dg/opt/call3.C fails at execution on armeb targets > g++.dg/opt/call2.C fails at execution on aarch64_be It turns out that there is already a big-endian adjustment a few lines above: /* If the value has a record type and an

Re: [PATCH 9a] Add "__RTL" to C frontend

2017-01-10 Thread Joseph Myers
On Mon, 9 Jan 2017, David Malcolm wrote: > This part of the patch adds the ability to tag a function with > "__RTL", analogous to the "__GIMPLE" tag. > > gcc/c-family/ChangeLog: > * c-common.c (c_common_reswords): Add "__RTL". > * c-common.h (enum rid): Add RID_RTL. > >

[PATCH] DOn't miscompile programs with -flto -fno-merge-constants (PR middle-end/50199)

2017-01-10 Thread Jakub Jelinek
Hi! LTO partitioning can actually take appart uses of STRING_CSTs or other constants and put those into multiple partitions. When -fno-merge-constants is in effect, it means those constants aren't merged by the linker and e.g. following testcase fails. I fail to see what -flto

[committed] add C99 runtime requirement to a couple of tests

2017-01-10 Thread Martin Sebor
The three bugs below point test failures in a couple of the printf tests due to them exercising C99 features not present in the default runtime on the targets (the h and hh length modifiers on HP-UX 11.11 and the a format specifier in Newlib). As suggested in PR 78959, the attached patch

[PATCH] Fix as flags for -c xx.s -g0 -g (PR driver/49726)

2017-01-10 Thread Jakub Jelinek
Hi! ASM_DEBUG_SPEC uses usually %{g:%{!g0:--gdwarf2}} or something similar. In the past, it used to be %{g:--gdwarf2}. Both have problems (and thus this is also a regression). The problem with the current ASM_DEBUG_SPEC is that if one uses -g0 -g, which normally means that -g overrides -g0, we

[PATCH] Optimize n + 1 for automatic n array (PR c++/71537)

2017-01-10 Thread Jakub Jelinek
Hi! This patch allows to fold n + 1 != 0 into true for automatic array n. We already handle it for variables in the symbol table (if not weak), but automatic vars are never in the symbol table. Bootstrapped/regtested on x86_64-linux and i686-linux, ok for trunk? Or shall I add the function

[C++ PATCH] Fix ICE on invalid (PR c++/78341)

2017-01-10 Thread Jakub Jelinek
Hi! As mentioned in the PR, cp_parser_parse_definitely may fail even when alignas_expr actually is meaningful, e.g. when the error is due to the missing closing paren. Bootstrapped/regtested on x86_64-linux and i686-linux, ok for trunk? 2017-01-10 Jakub Jelinek PR

[PATCH] Avoid generating code when writing PCH (PR c++/72813)

2017-01-10 Thread Jakub Jelinek
Hi! The comments in both the C and C++ FEs say that after writing PCH file when --output-pch= is used, we don't want to do anything else and the routines return to the caller early, especially for C++ FE skipping lots of needed handling for code generation. But, nothing is signalled to the

Re: [v3 PATCH] Reduce the size of variant, it doesn't need an index of type size_t internally.

2017-01-10 Thread Ville Voutilainen
Cleanups based on review; there's no longer any public typedefs added to variant, and the test is greatly simpler with much less trickery. 2017-01-11 Ville Voutilainen Reduce the size of variant, it doesn't need an index of type size_t internally. *

[PATCH] adding missing LTO to some warning options (PR 78606)

2017-01-10 Thread Martin Sebor
The -Walloca-larger-than, -Wformat-length, and -Wformat-truncation options do not mention LTO among the supported languages and so are disabled when -flto is used, causing false negatives. The attached patch adds the missing LTO to the three options. This makes -Walloca-larger-than work with

[committed] Fix issues with unrepresentable column numbers (PR c++/77949)

2017-01-10 Thread David Malcolm
PR c++/77949 identifies an ICE when the C++ frontend attempts to emit a fix-it hint inserting a missing semicolon at column 4097 of a source file. This column value exceeds LINE_MAP_MAX_COLUMN_NUMBER and hence isn't representable using a location_t. Attempting to do so leads to these problems,

libgo patch committed: copy more scheduler code from Go 1.7 runtime

2017-01-10 Thread Ian Lance Taylor
I looked at a diff of libgo/go/runtime/proc.go between Go 1.7 and gccgo, and copied over all the easy stuff. Bootstrapped and ran Go testsuite on x86_64-pc-linux-gnu. Committed to mainline. Ian Index: gcc/go/gofrontend/MERGE ===

Re: [PATCH] [msp430] Sync msp430_mcu_data with devices.csv

2017-01-10 Thread DJ Delorie
Committed for you, thanks!

[PATCH] Add new test

2017-01-10 Thread Jeff Law
pr77766 is a duplicate of pr78856. This just adds the test for pr77766 to the testsuite. Committing to the trunk. Jeff commit 2729592d26c48c6c2add94f4ee98bd4328eacf85 Author: law Date: Tue Jan 10 20:55:59 2017 + PR

[doc, committed] fix overfull hboxes in GCC manual

2017-01-10 Thread Sandra Loosemore
I've checked in this patch to address various overfull hbox warnings in the PDF version of the GCC manual. These changes are intended to be completely boring and content-free. ;-) There's still one case, in the ARM -mtune description, that needs more thought -- I'm considering changing all

[LRA] Fix PR rtl-optimization/79032

2017-01-10 Thread Eric Botcazou
hat of MODE, instead of the alignment of the original memory reference. 2017-01-10 Eric Botcazou <ebotca...@adacore.com> * gcc.c-torture/execute/20170110-1.c: New test. -- Eric Botcazou/* PR rtl-optimization/79032 */ extern void abort (void); struct S { short a;

[PATCH, rs6000] Fix PR79044 (ICE in swap optimization)

2017-01-10 Thread Bill Schmidt
Hi, PR79044 reports a situation where swap optimization ICEs in GCC 6 and in trunk. The problem is that swap optimization doesn't properly recognize that element-reversing loads and stores (e.g., lxvw4x) cannot be treated as "swappable" instructions. These arise from the __builtin_vec_xl and

Contents of PO file 'cpplib-7.1-b20170101.da.po'

2017-01-10 Thread Translation Project Robot
cpplib-7.1-b20170101.da.po.gz Description: Binary data The Translation Project robot, in the name of your translation coordinator.

New Danish PO file for 'cpplib' (version 7.1-b20170101)

2017-01-10 Thread Translation Project Robot
Hello, gentle maintainer. This is a message from the Translation Project robot. A revised PO file for textual domain 'cpplib' has been submitted by the Danish team of translators. The file is available at: http://translationproject.org/latest/cpplib/da.po (This file,

Re: [PATCH] Offer suggestions for unrecognized sanitizer options (PR driver/78877)

2017-01-10 Thread Jakub Jelinek
On Tue, Jan 10, 2017 at 03:02:49PM -0500, David Malcolm wrote: > +/* Given ARG, an unrecognized sanitizer option, return the best > + matching sanitizer option, or NULL if there isn't one. */ > + > +static const char * > +get_closest_sanitizer_option (const string_fragment ) > +{ > +

Re: Go patch committed: drop size arguments to hash/equal functions

2017-01-10 Thread Rainer Orth
Hi Ian, > Drop the size arguments for the hash/equal functions stored in type > descriptors. Types know what size they are. To make this work, > generate hash/equal functions for types that can use an identity > comparison but are not a standard size and alignment. > > Drop the multiplications

[PATCH] Offer suggestions for unrecognized sanitizer options (PR driver/78877)

2017-01-10 Thread David Malcolm
PR driver/78877 notes that we offer a suggestion for "-fsanitize-addres" (with a hyphen): xgcc: error: unrecognized command line option '-fsanitize-addres'; did you mean '-fsanitize=address'? but not for "-fsanitize=addres" (with an equals sign): xgcc: error: unrecognized argument to

Re: [PATCH][PR tree-optimization/78856] Invalidate cached iteration information when threading across multiple loop headers

2017-01-10 Thread Jeff Law
On 01/04/2017 05:25 AM, Richard Biener wrote: On Wed, Jan 4, 2017 at 6:31 AM, Jeff Law wrote: So as noted in the BZ comments the jump threading code has code that detects when a jump threading path wants to cross multiple loop headers and truncates the jump threading path in

C++ PATCH for C++17 NB comment FI20 (parenthesized initialization with decomposition)

2017-01-10 Thread Jason Merrill
The FI20 comment on the decomposition declarations proposal complained that the syntax unnecessarily excluded parenthesized initialization. This patch implements the resolution. Tested x86_64-pc-linux-gnu, applying to trunk. commit 749ec367e50b356a40fd41a3daae10d9d948062b Author: Jason Merrill

Re: C++ PATCH to implement C++17 variadic using

2017-01-10 Thread Jason Merrill
On Mon, Jan 9, 2017 at 4:50 PM, Jason Merrill wrote: > The last C++17 feature was pretty trivial to implement, as expected. ...and the feature-test macro. commit 9926adaa233a28474a12d16601a4f5a1204efbee Author: Jason Merrill Date: Mon Jan 9 17:37:21 2017

[PING][Aarch64][PATCH] Fix gcc.dg/zero_bits_compound-2.c for aarch64

2017-01-10 Thread Michael Collison
Ping. Link to original patch: https://gcc.gnu.org/ml/gcc-patches/2016-12/msg00400.html

Re: [PATCH] PR77528 add default constructors for container adaptors

2017-01-10 Thread Tim Song
On Tue, Jan 10, 2017 at 12:33 PM, Jonathan Wakely wrote: > The standard says that the container adaptors have a constructor with > a default argument, which serves as a default constructor. That > involves default-constructing the underlying sequence as the default > argument

Re: [PATCH] Add support for Fuchsia (OS)

2017-01-10 Thread Josh Conner via gcc-patches
On 1/10/17 7:54 AM, Richard Earnshaw (lists) wrote: On 12/12/16 21:31, Josh Conner via gcc-patches wrote: On 12/10/16 3:26 AM, Richard Earnshaw wrote: On 08/12/16 22:55, Josh Conner wrote: +arm*-*-fuchsia*) + tm_file="${tm_file} fuchsia.h arm/fuchsia-elf.h glibc-stdint.h" +

[PATCH] Adjust effective target selector to fix C++17 FAIL

2017-01-10 Thread Jonathan Wakely
This test using std::set_unexpected fails when run with -std=gnu++17 because throw() is no longer a dynamic exception specification and so the unexpected handler no longer gets called. This tweaks the target selector so it's only run for C++11 and C++14, where unexpected handlers are usable (it

[PATCH] PR77528 add default constructors for container adaptors

2017-01-10 Thread Jonathan Wakely
The standard says that the container adaptors have a constructor with a default argument, which serves as a default constructor. That involves default-constructing the underlying sequence as the default argument and then move-constructing the member variable from that argument. Because std::deque

[PATCH][AArch64] Improve Cortex-A53 scheduling of int/fp transfers

2017-01-10 Thread Wilco Dijkstra
My previous change to the Cortex-A53 scheduler resulted in a 13% regression on a proprietary benchmark. This turned out to be due to non-optimal scheduling of int to float conversions. This patch separates int to FP transfers from int to float conversions based on experiments to determine the

Re: [Ping~][AArch64] Add commandline support for -march=armv8.3-a

2017-01-10 Thread James Greenhalgh
On Fri, Jan 06, 2017 at 11:33:39AM +, Jiong Wang wrote: > On 11/11/16 18:22, Jiong Wang wrote: > >This patch add command line support for ARMv8.3-A through new architecture: > > > > -march=armv8.3-a > > > >ARMv8.3-A implies all default features of ARMv8.2-A and meanwhile it includes > >the

Re: [PATCH] Enable SGX intrinsics

2017-01-10 Thread Andrew Senkevich
On Fri, Dec 30, 2016 at 03:37:14PM +0100, Uros Bizjak wrote: >> As suggested in [1], you should write multi-line enums like: >> >> enum foo >> { >> a = ... >> b = ... >> } > > Sure. Plus it depends on if users of the APIs should just write the operands > on their own as numbers, or as

Re: [PATCH] Add support for Fuchsia (OS)

2017-01-10 Thread Richard Earnshaw (lists)
On 12/12/16 21:31, Josh Conner via gcc-patches wrote: > On 12/10/16 3:26 AM, Richard Earnshaw wrote: >> On 08/12/16 22:55, Josh Conner wrote: >>> +arm*-*-fuchsia*) >>> + tm_file="${tm_file} fuchsia.h arm/fuchsia-elf.h glibc-stdint.h" >>> + tmake_file="${tmake_file} arm/t-bpabi" >>> +

[PATCH] Reload global options when strict aliasing is dropped (PR ipa/79043).

2017-01-10 Thread Martin Liška
As mentioned in the PR, we currently do not properly reload global optimization options when we drop strict-aliasing flag on a function that equals to cfun. Patch can bootstrap on ppc64le-redhat-linux and survives regression tests. Ready to be installed? Martin >From

Re: [PATCH 2/2] IPA ICF: make algorithm stable to survive -fcompare-debug

2017-01-10 Thread Martin Liška
On 01/10/2017 02:56 PM, Richard Biener wrote: On Mon, Jan 9, 2017 at 4:05 PM, Martin Liška wrote: Second part of the patch does sorting of final congruence classes, it's groups and items included in the groups according DECL_UID. Both patches can bootstrap together on

Re: [patch mips/gcc] add build-time and runtime options to disable or set madd.fmt type

2017-01-10 Thread Yunqiang Su
Hi, folks, any idea about this patch? > 在 2016年12月24日,20:53,Yunqiang Su 写道: > > By this patch, I add >build-time options: > —with-madd4=unfused/fused/no >runtime options: > -mmadd4=unfused/fused/no > > to disable or set madd.fmt type. > > For

Re: Fix compilation errors with libstdc++v3 for AVR target and allow --enable-libstdcxx

2017-01-10 Thread Jonathan Wakely
On 23/12/16 13:18 -0200, Felipe Magno de Almeida wrote: On Fri, Dec 16, 2016 at 10:45 AM, Jonathan Wakely wrote: On 15/12/16 21:41 -0300, Felipe Magno de Almeida wrote: Good point. Do you want me to update the patch with __men or go your way with the wrapper? Hello

[PATCH] combine ignoring a check

2017-01-10 Thread Nathan Sidwell
This patch addresses an issue Nicolai found while browsing combine.c. He noticed some checking code that looked like: for (i = 0; i < NUM_ARGS; i++) if (!cond) break; for (i = 0; i < NUM_ARGS; i++) if (!other_cond) break; if (i == NUM_ARGS) do_the_thing; the

Re: [PATCH 2/2] IPA ICF: make algorithm stable to survive -fcompare-debug

2017-01-10 Thread Richard Biener
On Mon, Jan 9, 2017 at 4:05 PM, Martin Liška wrote: > Second part of the patch does sorting of final congruence classes, it's groups > and items included in the groups according DECL_UID. > > Both patches can bootstrap together on ppc64le-redhat-linux and survive > regression

Re: [PATCH 9d] Don't call delete_tree_ssa for __RTL functions

2017-01-10 Thread Richard Biener
On Tue, Jan 10, 2017 at 3:38 AM, David Malcolm wrote: > gcc/ChangeLog: > * final.c (rest_of_clean_state): Don't call delete_tree_ssa for > __RTL functions. Heh, so you are lucky that nothing looks at this. MEM_EXPRs can contain SSA names these days (for

Re: [PATCH 9b] Don't assume that copy tables were initialized

2017-01-10 Thread Richard Biener
On Tue, Jan 10, 2017 at 3:38 AM, David Malcolm wrote: > gcc/ChangeLog: > * cfg.c (original_copy_tables_initialized_p): New function. > * cfg.h (original_copy_tables_initialized_p): New decl. > * cfgrtl.c (relink_block_chain): Guard the call to >

Re: [PATCH] Tree-level fix for PR 69526

2017-01-10 Thread Robin Dapp
Perhaps I'm still missing how some cases are handled or not handled, sorry for the noise. > I'm not sure there is anything to "interpret" -- the operation is unsigned > and overflow is when the operation may wrap around zero. There might > be clever ways of re-writing the expression to >

Re: [PATCH] Do not error when -E provided (PR pch/78970).

2017-01-10 Thread Jakub Jelinek
On Tue, Jan 10, 2017 at 02:32:07PM +0100, Martin Liška wrote: > On 01/09/2017 04:12 PM, Jakub Jelinek wrote: > > Was ? > > > > Ok with that change. > > > > Jakub > > Thanks for review, installed as r244227. > > I've finished testing for both active branches, may I install the patch? Ok.

Re: [PATCH] Do not error when -E provided (PR pch/78970).

2017-01-10 Thread Martin Liška
On 01/09/2017 04:12 PM, Jakub Jelinek wrote: Was ? Ok with that change. Jakub Thanks for review, installed as r244227. I've finished testing for both active branches, may I install the patch? Martin

Re: [gomp4] remove GOVD_USE_DEVPTR

2017-01-10 Thread Thomas Schwinge
Hi! On Mon, 7 Nov 2016 12:45:43 -0800, Cesar Philippidis wrote: > It looks like gomp-4_0-branch was using the GOVD_USE_DEVPTR attribute [...] > I've applied this patch remove GOVD_USE_DEVPTR from gomp4. I'm seeing: [...]/gcc/gimplify.c: In function 'int

Re: [gomp4] remove GOVD_USE_DEVPTR

2017-01-10 Thread Thomas Schwinge
Hi! On Mon, 7 Nov 2016 12:45:43 -0800, Cesar Philippidis wrote: > It looks like gomp-4_0-branch was using the GOVD_USE_DEVPTR attribute [...] > I've applied this patch remove GOVD_USE_DEVPTR from gomp4. > 2016-11-07 Cesar Philippidis > >

Re: [PATCH v2] PR sanitizer/78992: Fix sigaction definition on 32-bit sparc

2017-01-10 Thread Jakub Jelinek
On Tue, Jan 10, 2017 at 01:19:32PM +, James Clarke wrote: > libsanitizer: > PR sanitizer/78992 > * sanitizer_common/sanitizer_platform_limits_posix.h > (struct __sanitizer_sigaction): Cherry-pick upstream r291561. Ok, thanks. > --- >

[PATCH v2] PR sanitizer/78992: Fix sigaction definition on 32-bit sparc

2017-01-10 Thread James Clarke
libsanitizer: PR sanitizer/78992 * sanitizer_common/sanitizer_platform_limits_posix.h (struct __sanitizer_sigaction): Cherry-pick upstream r291561. --- libsanitizer/sanitizer_common/sanitizer_platform_limits_posix.h | 7 +-- 1 file changed, 5 insertions(+), 2

Re: [PR tree-optimization/78024] Clear basic block flags before using BB_VISITED for OpenACC loops processing

2017-01-10 Thread Thomas Schwinge
Hi! On Wed, 19 Oct 2016 12:28:39 +0200, I wrote: > [PR tree-optimization/78024] Clear basic block flags before using > BB_VISITED for OpenACC loops processing To fix the same problem, committed the following to gcc-6-branch in r244264: commit 82f25a52380b366a99af8045fb615d79d4ff94b6

Re: [PATCH][x86_64] Enable AVX512 VPOPCNTD/VPOPCNTQ instructions

2017-01-10 Thread Kirill Yukhin
On 10 Jan 15:00, Andrew Senkevich wrote: > 2017-01-10 13:31 GMT+03:00 Uros Bizjak : > > On Tue, Jan 10, 2017 at 11:21 AM, Andrew Senkevich > > wrote: > >> 2017-01-10 13:04 GMT+03:00 Kirill Yukhin : > >>> Hi, > >>> In

Re: Pretty printers for versioned namespace

2017-01-10 Thread Jonathan Wakely
On 09/01/17 21:25 +0100, François Dumont wrote: On 04/01/2017 13:52, Jonathan Wakely wrote: On 24/12/16 14:47 +0100, François Dumont wrote: I'd prefer not to have to use the regex matches in libstdc++.exp as they complicate things. For the two examples above, the whatis results are bad even

Re: [patch] Fix wrong code for return of small aggregates on big-endian

2017-01-10 Thread Eric Botcazou
> They pass before the patch (I only checked armeb). I think that's not true for aarch64_be though, since the patch doesn't change code generation for this target. But I'll fix that too. -- Eric Botcazou

Re: [PTX] fix worker propagation ICE

2017-01-10 Thread Thomas Schwinge
Hi! On Wed, 3 Aug 2016 13:30:10 -0400, Nathan Sidwell wrote: > The PTX backend could ice when generating a state propagation sequence > entering > partitioned execution. [...] To fix the same problem, applying the same changes, committed the following to gcc-6-branch in

Re: [Ada] Fix PR ada/78845

2017-01-10 Thread Arnaud Charlet
> >> 2016-12-21 Simon Wright >> > > >> > >>PR ada/78845 > >>* a-ngcoar.adb (Inverse): call Unit_Matrix with First_1 set to > >>A'First(2) > >> and vice versa. > >>* a-ngrear.adb (Inverse): likewise. > > Can the patch be

Re: [PATCH][x86_64] Enable AVX512 VPOPCNTD/VPOPCNTQ instructions

2017-01-10 Thread Andrew Senkevich
2017-01-10 13:31 GMT+03:00 Uros Bizjak : > On Tue, Jan 10, 2017 at 11:21 AM, Andrew Senkevich > wrote: >> 2017-01-10 13:04 GMT+03:00 Kirill Yukhin : >>> Hi, >>> In addition to Uroš's inputs: diff --git

Re: [Ada] Fix PR ada/78845

2017-01-10 Thread Simon Wright
On 21 Dec 2016, at 14:52, Arnaud Charlet wrote: > >>> Yes, please resend an updated patch. >> >> The function Ada.Numerics.Generic_Real_Arrays.Inverse is required >> (ARM G.3.1(72)) to >> return a matrix with the bounds of the dimension indices swapped, i.e. >>

Re: [patch] Fix wrong code for return of small aggregates on big-endian

2017-01-10 Thread Christophe Lyon
On 10 January 2017 at 11:26, Eric Botcazou wrote: >> They pass before the patch (I only checked armeb). > > Thanks, I see what's going on, but can you post the configure line of armeb? > Sure, it is: --target=armeb-none-linux-gnueabihf --with-float=hard --with-mode=arm

Re: [PATCH] Fix IPA CP where it forgot to add a reference in cgraph

2017-01-10 Thread Martin Liška
PING^1 On 12/20/2016 03:55 PM, Martin Liška wrote: On 12/20/2016 11:06 AM, Martin Jambor wrote: ...this test should be for ADDR_EXPR here. Or you could switch the IPA_REF_* constants the other way round which I bet is going to have the same effect in practice, but personally, I'd test for

[PATCH][ARM] [gcc] Add __artificial__ attribute to all NEON intrinsics

2017-01-10 Thread Tamar Christina
Hi all, This patch adds the __artificial__ and __gnu_inline__ attributes to the intrinsics in arm_neon.h so that costs are associated to the user function during profiling and during debugging the intrinsics are hidden in trace. A similar patch was already applied to Aarch64. The artificial

Re: [PATCH][x86_64] Enable AVX512 VPOPCNTD/VPOPCNTQ instructions

2017-01-10 Thread Uros Bizjak
On Tue, Jan 10, 2017 at 11:21 AM, Andrew Senkevich wrote: > 2017-01-10 13:04 GMT+03:00 Kirill Yukhin : >> Hi, >> In addition to Uroš's inputs: >>> diff --git a/gcc/config/i386/avx512vpopcntdqintrin.h >>>

Re: [patch] Fix wrong code for return of small aggregates on big-endian

2017-01-10 Thread Eric Botcazou
> They pass before the patch (I only checked armeb). Thanks, I see what's going on, but can you post the configure line of armeb? -- Eric Botcazou

Re: [PATCH][x86_64] Enable AVX512 VPOPCNTD/VPOPCNTQ instructions

2017-01-10 Thread Andrew Senkevich
2017-01-10 13:04 GMT+03:00 Kirill Yukhin : > Hi, > In addition to Uroš's inputs: >> diff --git a/gcc/config/i386/avx512vpopcntdqintrin.h >> b/gcc/config/i386/avx512vpopcntdqintrin.h >> new file mode 100644 >> index 000..28305f6 >> --- /dev/null >> +++

Re: [patch] Fix wrong code for return of small aggregates on big-endian

2017-01-10 Thread Christophe Lyon
On 10 January 2017 at 09:58, Eric Botcazou wrote: >> I have noticed new failures after this commit (r244249). >> g++.dg/opt/call3.C fails at execution on armeb targets >> g++.dg/opt/call2.C fails at execution on aarch64_be > > They are new testcases: can you find out

[PATCH] Fix PR79034

2017-01-10 Thread Richard Biener
I am testing the following patch to fix another case where call shrink-wrapping fails to update PHIs properly. The previous fix missed the case where the destination contained a degenerate PHI thus we can simply propagate those out. Bootstrap and regtest running on x86_64-unknown-linux-gnu (for

Re: [PATCH][x86_64] Enable AVX512 VPOPCNTD/VPOPCNTQ instructions

2017-01-10 Thread Kirill Yukhin
Hi, In addition to Uroš's inputs: > diff --git a/gcc/config/i386/avx512vpopcntdqintrin.h > b/gcc/config/i386/avx512vpopcntdqintrin.h > new file mode 100644 > index 000..28305f6 > --- /dev/null > +++ b/gcc/config/i386/avx512vpopcntdqintrin.h > @@ -0,0 +1,90 @@ > +/* Copyright (C) 2016 Free

last call predictor tweek

2017-01-10 Thread Jan Hubicka
Hi, this is the last change to make call predictor match the statistical data. Honza PR middle-end/77484 * predict.def (PRED_CALL): Set to 67. Index: predict.def === --- predict.def (revision 244259) +++ predict.def

[PATCH] nds32: Backport PR70668 solution from mainline

2017-01-10 Thread Chung-Ju Wu
Hi, all, https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70668 https://gcc.gnu.org/ml/gcc-patches/2016-04/msg01862.html Thanks Stefan Reinauer to create this PR and thanks Segher Boessenkool to fix the issue. Now I backport the solution to gcc-6 and gcc-5 branches. Committed as revision r244257

Re: [patch] Fix wrong code for return of small aggregates on big-endian

2017-01-10 Thread Eric Botcazou
> I have noticed new failures after this commit (r244249). > g++.dg/opt/call3.C fails at execution on armeb targets > g++.dg/opt/call2.C fails at execution on aarch64_be They are new testcases: can you find out whether they pass before the patch? -- Eric Botcazou

[gomp4] Support multi-dimensional pointer based arrays in OpenACC data clauses

2017-01-10 Thread Chung-Lin Tang
This patch implements support for dynamically allocated multi-dimensional arrays in OpenACC data clauses. To illustrate, these kinds of arrays now work: int **a; float *f[100]; double ***d; #pragma acc parallel copy (a[0:100][x:y], f[10:20][0:30]) copyout(d[x:y][x:y][x:y]) { ... } The

Fwd: [Patch] PR71017 - libgcc/config/i386/cpuinfo.c:346:17: runtime error: left shift of 1 by 31 places cannot be represented in type 'int'

2017-01-10 Thread Uros Bizjak
On Mon, Jan 9, 2017 at 11:05 PM, Dominique d'Humières wrote: > > > Le 9 janv. 2017 à 20:37, Uros Bizjak a écrit : > > > > Hello! > > > >> The following patch fixes errors of the kind > >> > >> libgcc/config/i386/cpuinfo.c:260:17: runtime error: left shift

Re: [patch] Fix wrong code for return of small aggregates on big-endian

2017-01-10 Thread Christophe Lyon
On 9 January 2017 at 12:14, Richard Biener wrote: > On Mon, Jan 9, 2017 at 11:43 AM, Eric Botcazou wrote: >> Hi, >> >> this is a regression present on all active branches for big-endian targets >> returning small aggregate types in registers