Re: [flag-types.h] don't assume 32-bit ints

2014-10-15 Thread Jakub Jelinek
On Wed, Oct 15, 2014 at 06:16:45PM -0400, DJ Delorie wrote: > > Since flag-types.h is used in target code, it needs to be -Wall-safe > for targets with 16-bit "int". OK? With a ChangeLog entry yes. > --- gcc/flag-types.h (revision 216287) > +++ gcc/flag-types.h (working copy) > @@ -229,17 +22

Re: PR debug/60655, debug loc expressions

2014-10-15 Thread Alan Modra
Ping? https://gcc.gnu.org/ml/gcc-patches/2014-09/msg00704.html -- Alan Modra Australia Development Lab, IBM

[PATCH i386 AVX512] [79/n] Extend expand_mul_widen_hilo.

2014-10-15 Thread Kirill Yukhin
Hello, This patch extends expand_mul_widen_hilo to 512-bit QI,SI,HI modes. Bootstrapped and regtested gcc/ * config/i386/i386.c (ix86_expand_mul_widen_hilo): Handle V32HI, V16SI, V64QI modes. Is it ok for trunk? -- Thanks, K diff --git a/gcc/config/i386/i386.c b/gcc/config/i386

[PATCH i386 AVX512] [78/n] Use blend for inserting.

2014-10-15 Thread Kirill Yukhin
Hello, This patch extends insertion hook. AVX-512* tests on top of patch-set all pass under simulator. gcc/ * config/i386/i386.c (ix86_expand_vector_set): Handle V8DF, V8DI, V16SF, V16SI, V32HI, V64QI modes. -- Thanks, K diff --git a/gcc/config/i386/i386.c b/gcc/config/

Re: __intN patch 3/5: main __int128 -> __intN conversion.

2014-10-15 Thread Markus Trippelsdorf
On 2014.10.15 at 17:00 -0400, DJ Delorie wrote: > > > If you could implement the second option, it would be appreciated. > > Could you please test this for me? It builds as a powerpc-elf > cross-compiler (at least the host half) but I don't have a power > machine here to test on. > > Index: rs6

NRV with address taken

2014-10-15 Thread Marc Glisse
Hello, the attached one-liner passed bootstrap+testsuite (really all languages) on x86_64-linux-gnu (I got an extra pass of unix/-m32: os but I assume that the failure with trunk was random). The current code is a bit weird: we bail out if either result or found is TREE_ADDRESSABLE, but then

Re: [pointer size] more edge cases

2014-10-15 Thread DJ Delorie
> The other changes are OK. Committed those then. Thanks!

Re: Fix pr61848, linux kernel miscompile

2014-10-15 Thread Alan Modra
On Thu, Sep 18, 2014 at 10:02:25PM -0600, Jeff Law wrote: > On 09/16/14 00:03, Alan Modra wrote: > >gcc testsuite additions? I decline. It is too soon. If you had read > >my patch submission you'll see that at some stage gcc was supposed to > >warn on conflicting section attributes, but hasn't d

Re: [PATCH] Add zero-overhead looping for xtensa backend

2014-10-15 Thread Yangfei (Felix)
Hi Sterling, Since the patch is delayed for a long time, I'm kind of pushing it. Sorry for that. Yeah, you are right. We have some performance issue here as GCC may use one more general register in some cases with this patch. Take the following arraysum testcase for example. In dol

Re: [PATCH] Add a new option "-fmerge-bitfields" (patch / doc inside)

2014-10-15 Thread Andrew Pinski
On Tue, Apr 22, 2014 at 3:28 AM, Zoran Jovanovic wrote: > Hello, > Updated doc/invoke.texi by stating that new option is enabled by default at > -O2 and higher. > Also, -fmerge-bitfields added to the list of optimization flags enabled by > default at -O2 and higher. With this patch applied g

Re: gccgo testcase: binary testdata for libgo/go/debug/elf/file_test.go

2014-10-15 Thread Ian Lance Taylor
On Wed, Oct 15, 2014 at 1:41 PM, Lynn A. Boger wrote: > > I'm working on a patch submission for gccgo to add the R_PPC64 relocation > definitions to libgo/go/debug/elf/elf.go. Along with this submission I was > going to update the testcase file_test.go from the same directory so it > would test t

Re: [pointer size] more edge cases

2014-10-15 Thread Joseph S. Myers
On Wed, 15 Oct 2014, DJ Delorie wrote: > > I don't see what the stor-layout.c changes have to do with that > > description, or why they are correct (they look wrong to me; the > > existing addition of BITS_PER_UNIT_LOG + 1 looks logically correct > > for bitsizetype). > > sooo... the type for bit

Re: [pointer size] more edge cases

2014-10-15 Thread DJ Delorie
> I don't see what the stor-layout.c changes have to do with that > description, or why they are correct (they look wrong to me; the > existing addition of BITS_PER_UNIT_LOG + 1 looks logically correct > for bitsizetype). sooo... the type for bitsizetype will *always* be a bigger type than sizety

Re: [Google 4.9] Backport of r210828

2014-10-15 Thread Sterling Augustine
On Thu, Oct 9, 2014 at 10:21 AM, Carrot Wei wrote: > LGTM. > Your description could be more detail, such as which tests on which target. Committed with a much improved message as r216291

Re: [pointer size] more edge cases

2014-10-15 Thread Joseph S. Myers
On Wed, 15 Oct 2014, DJ Delorie wrote: > A few more cases where pointers were assumed to be whole bytes. > Ok? I don't see what the stor-layout.c changes have to do with that description, or why they are correct (they look wrong to me; the existing addition of BITS_PER_UNIT_LOG + 1 looks logica

Re: [GOOGLE] recalculate dominance before update_ssa in AutoFDO pass

2014-10-15 Thread Xinliang David Li
ok. David On Wed, Oct 15, 2014 at 4:00 PM, Dehao Chen wrote: > It's destroyed by gimple_value_profile_transformations. > > Patch updated to move the update_ssa early. > > Index: gcc/auto-profile.c > === > --- gcc/auto-profile.c (rev

Re: [GOOGLE] recalculate dominance before update_ssa in AutoFDO pass

2014-10-15 Thread Dehao Chen
It's destroyed by gimple_value_profile_transformations. Patch updated to move the update_ssa early. Index: gcc/auto-profile.c === --- gcc/auto-profile.c (revision 216278) +++ gcc/auto-profile.c (working copy) @@ -1674,7 +1674,14 @@ a

[pointer size] more edge cases

2014-10-15 Thread DJ Delorie
A few more cases where pointers were assumed to be whole bytes. Ok? Index: gcc/c-family/c-cppbuiltin.c === --- gcc/c-family/c-cppbuiltin.c (revision 216287) +++ gcc/c-family/c-cppbuiltin.c (working copy) @@ -668,13 +668,13 @@ cpp_ato

[flag-types.h] don't assume 32-bit ints

2014-10-15 Thread DJ Delorie
Since flag-types.h is used in target code, it needs to be -Wall-safe for targets with 16-bit "int". OK? Index: gcc/flag-types.h === --- gcc/flag-types.h(revision 216287) +++ gcc/flag-types.h(working copy) @@ -229,17 +229,17

[lto] don't assume pointer size

2014-10-15 Thread DJ Delorie
In the event that pointer sizes aren't powers of two, choose a more suitable alignment than (unsigned)(-1), which results in HUGE file sizes. Ok? Index: gcc/lto/lto-object.c === --- gcc/lto/lto-object.c(revision 216287) +++

Re: [C PATCH] Clamp down "incomplete type" error (PR c/63543)

2014-10-15 Thread Joseph S. Myers
On Wed, 15 Oct 2014, Jeff Law wrote: > On 10/15/14 15:46, Joseph S. Myers wrote: > > On Wed, 15 Oct 2014, Marek Polacek wrote: > > > > > We've got a complaint that the "dereferencing pointer to incomplete > > > type" error is printed for all occurrences of the incomplete type, > > > which is too

Re: [C PATCH] Clamp down "incomplete type" error (PR c/63543)

2014-10-15 Thread Jakub Jelinek
On Wed, Oct 15, 2014 at 03:48:20PM -0600, Jeff Law wrote: > On 10/15/14 15:46, Joseph S. Myers wrote: > >On Wed, 15 Oct 2014, Marek Polacek wrote: > > > >>We've got a complaint that the "dereferencing pointer to incomplete > >>type" error is printed for all occurrences of the incomplete type, > >>w

Re: [C PATCH] Clamp down "incomplete type" error (PR c/63543)

2014-10-15 Thread Jeff Law
On 10/15/14 15:46, Joseph S. Myers wrote: On Wed, 15 Oct 2014, Marek Polacek wrote: We've got a complaint that the "dereferencing pointer to incomplete type" error is printed for all occurrences of the incomplete type, which is too verbose. Also it'd be nicer to print the type as well. This pa

Re: [C PATCH] Clamp down "incomplete type" error (PR c/63543)

2014-10-15 Thread Joseph S. Myers
On Wed, 15 Oct 2014, Marek Polacek wrote: > We've got a complaint that the "dereferencing pointer to incomplete > type" error is printed for all occurrences of the incomplete type, > which is too verbose. Also it'd be nicer to print the type as well. > This patch fixes this; if we find an incompl

Re: [PATCH 2/5] gcc: configure and Makefile changes needed by jit

2014-10-15 Thread David Malcolm
On Wed, 2014-10-15 at 14:51 -0600, Jeff Law wrote: > On 10/15/14 12:25, David Malcolm wrote: > > On Wed, 2014-10-15 at 11:36 -0600, Jeff Law wrote: > >> On 10/13/14 11:45, David Malcolm wrote: > >>> gcc/ChangeLog: > >>> * configure.ac (gcc_version): Expose this value for use via > >>> AC_SUBST,

[v3] Update a testcase vs c++/53000

2014-10-15 Thread Paolo Carlini
Hi, contributed by Daniel. Thanks! Paolo. / 2014-10-15 Daniel Krugler * testsuite/20_util/common_type/requirements/sfinae_friendly_1.cc: Enable tests predicated on PR c++/53000. Index: testsuite/20_util/common_type/requirements/sfinae_friendly_1.cc =

Re: [C PATCH] Clamp down "incomplete type" error (PR c/63543)

2014-10-15 Thread Jeff Law
On 10/15/14 11:22, Marek Polacek wrote: We've got a complaint that the "dereferencing pointer to incomplete type" error is printed for all occurrences of the incomplete type, which is too verbose. Also it'd be nicer to print the type as well. This patch fixes this; if we find an incomplete type,

Re: __intN patch 3/5: main __int128 -> __intN conversion.

2014-10-15 Thread DJ Delorie
> If you could implement the second option, it would be appreciated. Could you please test this for me? It builds as a powerpc-elf cross-compiler (at least the host half) but I don't have a power machine here to test on. Index: rs6000-c.c

Re: [PATCH 2/5] gcc: configure and Makefile changes needed by jit

2014-10-15 Thread Jeff Law
On 10/15/14 12:25, David Malcolm wrote: On Wed, 2014-10-15 at 11:36 -0600, Jeff Law wrote: On 10/13/14 11:45, David Malcolm wrote: gcc/ChangeLog: * configure.ac (gcc_version): Expose this value for use via AC_SUBST, since the jit code needs it within the new file libgccj

Re: [PATCH 08/10] Documentation for the JIT library (Re: Patches 5-10 of jit merger)

2014-10-15 Thread Jeff Law
On 10/15/14 10:56, David Malcolm wrote: On Tue, 2014-10-14 at 11:09 -0400, David Malcolm wrote: On Mon, 2014-10-13 at 13:45 -0400, David Malcolm wrote: I'd like to merge the JIT branch into trunk: https://gcc.gnu.org/wiki/JIT This is "v2" since it incorporates fixes for the various issues i

gccgo testcase: binary testdata for libgo/go/debug/elf/file_test.go

2014-10-15 Thread Lynn A. Boger
Hi, I'm working on a patch submission for gccgo to add the R_PPC64 relocation definitions to libgo/go/debug/elf/elf.go. Along with this submission I was going to update the testcase file_test.go from the same directory so it would test the new defines for ppc64 relocation. This test current

Re: [Bug libstdc++/63500] [4.9/5 Regression] bug in debug version of std::make_move_iterator?

2014-10-15 Thread Jonathan Wakely
On 15/10/14 22:06 +0200, François Dumont wrote: On 15/10/2014 13:10, Jonathan Wakely wrote: I find this much easier to read: #if __cplusplus < 201103L typedef _Is_contiguous_sequence<_Sequence> __tag; #else using __lvalref = std::is_lvalue_reference< typename std::iterator_traits

Re: [Bug libstdc++/63500] [4.9/5 Regression] bug in debug version of std::make_move_iterator?

2014-10-15 Thread François Dumont
On 15/10/2014 13:10, Jonathan Wakely wrote: I find this much easier to read: #if __cplusplus < 201103L typedef _Is_contiguous_sequence<_Sequence> __tag; #else using __lvalref = std::is_lvalue_reference< typename std::iterator_traits<_InputIterator>::reference>; using __con

Re: [patch] Fix ICE during LTRANS on big Ada application

2014-10-15 Thread Jan Hubicka
> Jan, any opinion on the following patch? > > > 2014-10-06 Eric Botcazou > > > > * ipa-inline-transform.c (can_remove_node_now_p_1): Return false for a > > master clone that has non-inline clones. > > https://gcc.gnu.org/ml/gcc-patches/2014-10/msg00436.html > > Thanks in advance. Y

Re: [PATCH 07/10] Testsuite for the JIT (Re: Patches 5-10 of jit merger

2014-10-15 Thread Mike Stump
On Oct 15, 2014, at 10:49 AM, Jeff Law wrote: > I just did some very rough sanity checking -- the details of the testsuite > I'm totally leaving in your hands :-) Likewise, I’m likely to leave it totally in your hands. If you have any specific questions or concerns, asking would be the way to

Re: [PATCH] Add zero-overhead looping for xtensa backend

2014-10-15 Thread augustine.sterl...@gmail.com
On Tue, Oct 14, 2014 at 8:39 AM, Felix Yang wrote: > PING? > Cheers, > Felix Felix, This isn't my day job, 24-hour pings are unproductive. You shouldn't need to worry about the trip count register getting spilled. It makes no difference whatsoever to how the loop operates--the trip count is dea

Re: [PATCH 1/5] libiberty: Expose choose_tmpdir, and fix constness of return type

2014-10-15 Thread DJ Delorie
As long as you commit to gcc first, or at the same time as other commits, you'll be fine. If you commit elsewhere first or only, it may be reverted by a script.

Re: [PATCH 1/5] libiberty: Expose choose_tmpdir, and fix constness of return type

2014-10-15 Thread David Malcolm
On Wed, 2014-10-15 at 11:33 -0600, Jeff Law wrote: > On 10/13/14 11:45, David Malcolm wrote: > > The jit needs to create a temporary directory, so I need to expose > > libiberty's choose_tmpdir as a public entrypoint. > > > > include/ChangeLog: > > * libiberty.h (choose_tmpdir): New prototype.

Re: [PATCH 2/5] gcc: configure and Makefile changes needed by jit

2014-10-15 Thread Joseph S. Myers
On Wed, 15 Oct 2014, David Malcolm wrote: > As for the "bindir" in site.exp, Joseph asked me when the library > invokes a driver to convert from .s to .so to: > > On Tue, 2014-09-23 at 23:27 +, Joseph S. Myers wrote: > > * use the $(target_noncanonical)-gcc-$(version) name for the > > driver

Re: [PATCH 2/5] gcc: configure and Makefile changes needed by jit

2014-10-15 Thread David Malcolm
On Wed, 2014-10-15 at 11:36 -0600, Jeff Law wrote: > On 10/13/14 11:45, David Malcolm wrote: > > gcc/ChangeLog: > > * configure.ac (gcc_version): Expose this value for use via > > AC_SUBST, since the jit code needs it within the new file > > libgccjit.pc.in. > > (doc_build_sys): New

Re: [v3] Minimally exercise the other alias_decls in

2014-10-15 Thread Paolo Carlini
Hi, On 10/15/2014 11:44 AM, Jonathan Wakely wrote: We might also want to consider a global s/gnu++0x/gnu++11/ some day. The below takes care of 17-19, the rest over the next days... Paolo. // 2014-10-15 Paolo Carlini * testsuite/17_intro/freestanding.cc: Prefer -st

[PATCH, i386]: Fix PR 59432, sync/atomic FAILs on 32bit x86 systems without .cfi directives

2014-10-15 Thread Uros Bizjak
Hello! Now that %ebx is no more fixed, we can remove all PIC related complications in atomic_compare_and_swap_doubleword pattern. The immediate consequence is, that we avoid "hidden" xchgs that clobbered unwinding state. Earlier fix by Ian [1] partly solved this issue using various .cfi directive

Re: [patch] Fix ICE during LTRANS on big Ada application

2014-10-15 Thread Eric Botcazou
Jan, any opinion on the following patch? > 2014-10-06 Eric Botcazou > > * ipa-inline-transform.c (can_remove_node_now_p_1): Return false for a > master clone that has non-inline clones. https://gcc.gnu.org/ml/gcc-patches/2014-10/msg00436.html Thanks in advance. -- Eric Botcazou

Re: [PATCH x86, java, PR63536] Fix java bootstrap for -mtune=intel/slm

2014-10-15 Thread Evgeny Stupachenko
Committed to trunk. Author: iverbin Date: Wed Oct 15 17:49:27 2014 New Revision: 216280 URL: https://gcc.gnu.org/viewcvs?rev=216280&root=gcc&view=rev Log: PR target/63536 gcc/java/ * lang.c (java_print_error_function): Add check on NULL function context. Modified: trunk/gcc/java/ChangeLog

Re: [PATCH, Pointer Bounds Checker 14/x] Passes [1/n] Expand interfaces

2014-10-15 Thread Ilya Enkovich
2014-10-14 1:11 GMT+04:00 Jeff Law : > On 10/08/14 12:50, Ilya Enkovich wrote: >> >> Hi, >> >> This patch starts a series which is a result of split of Pointer Bounds >> Checker patch #14 (Pointer Bounds Checker passes). >> >> This patch introduces compiler flags used by checker and helper function

Re: [PATCH 10/10] ChangeLog files (Re: Patches 5-10 of jit merger

2014-10-15 Thread Jeff Law
On 10/14/14 09:30, David Malcolm wrote: On Tue, 2014-10-14 at 11:09 -0400, David Malcolm wrote: On Mon, 2014-10-13 at 13:45 -0400, David Malcolm wrote: I'd like to merge the JIT branch into trunk: https://gcc.gnu.org/wiki/JIT This is "v2" since it incorporates fixes for the various issues i

Re: [PATCH 09/10] Prebuilt texinfo documentation for the JIT library (Re: Patches 5-10 of jit merger)

2014-10-15 Thread Jeff Law
On 10/15/14 10:59, David Malcolm wrote: On Tue, 2014-10-14 at 11:09 -0400, David Malcolm wrote: On Mon, 2014-10-13 at 13:45 -0400, David Malcolm wrote: I'd like to merge the JIT branch into trunk: https://gcc.gnu.org/wiki/JIT This is "v2" since it incorporates fixes for the various issues i

Re: [GOOGLE] recalculate dominance before update_ssa in AutoFDO pass

2014-10-15 Thread Xinliang David Li
Is it destroyed by value profile transformations? Can you move the dominance recomputing code closer to where it gets invalidated? David On Wed, Oct 15, 2014 at 10:37 AM, Dehao Chen wrote: > This patch recalculates dominance info before update_ssa call in > AutoFDO. This fixes bug when dominance

Re: [PATCH 07/10] Testsuite for the JIT (Re: Patches 5-10 of jit merger

2014-10-15 Thread Jeff Law
On 10/14/14 09:18, David Malcolm wrote: On Tue, 2014-10-14 at 11:09 -0400, David Malcolm wrote: On Mon, 2014-10-13 at 13:45 -0400, David Malcolm wrote: I'd like to merge the JIT branch into trunk: https://gcc.gnu.org/wiki/JIT This is "v2" since it incorporates fixes for the various issues i

Re: [v3] Minimally exercise the other alias_decls in

2014-10-15 Thread Paolo Carlini
On 10/15/2014 07:35 PM, Mike Stump wrote: On Oct 15, 2014, at 8:41 AM, Paolo Carlini wrote: ... the below replaces -std=c++0x and -std=c++11 with -std=gnu++11. Seems odd to go from c++11 -> gnu++11. Why? Because all the new testcases we are committing use -std=gnu++11, which is right, becau

Re: [v3] Minimally exercise the other alias_decls in

2014-10-15 Thread Mike Stump
On Oct 15, 2014, at 2:45 AM, Jonathan Wakely wrote: > On 15/10/14 10:44 +0100, Jonathan Wakely wrote: >> We might also want to consider a global s/gnu++0x/gnu++11/ some day. > > One reason for doing that replacement is that I keep copying existing > tests and then having to change it, and I'm laz

Re: [PATCH 05/10] JIT-related changes outside of jit subdir

2014-10-15 Thread Jeff Law
On 10/14/14 09:10, David Malcolm wrote: ChangeLog: * MAINTAINERS (Various Maintainers): Add myself as jit maintainer. contrib/ChangeLog: * jit-coverage-report.py: New file: a script to print crude code-coverage information for the libgccjit API. gcc/ChangeLog: *

C++ PATCH for c++/63528 (variadic variable template)

2014-10-15 Thread Jason Merrill
We were never calling coerce_template_parms for arguments to a variable template, which meant we weren't packing arguments into an argument pack, so tsubst_pack_expansion gave up. Tested x86_64-pc-linux-gnu, applying to trunk. commit 7e029731a4b77a1c54982e9922ff09219c46cc4b Author: Jason Merril

[GOOGLE] recalculate dominance before update_ssa in AutoFDO pass

2014-10-15 Thread Dehao Chen
This patch recalculates dominance info before update_ssa call in AutoFDO. This fixes bug when dominance info is out-of-date and causes segfaults during update_ssa. Bootstrapped and regression test on-going. OK for google-4_9 branch? Thanks, Dehao Index: gcc/auto-profile.c ==

Re: [PATCH 2/5] gcc: configure and Makefile changes needed by jit

2014-10-15 Thread Jeff Law
On 10/13/14 11:45, David Malcolm wrote: gcc/ChangeLog: * configure.ac (gcc_version): Expose this value for use via AC_SUBST, since the jit code needs it within the new file libgccjit.pc.in. (doc_build_sys): New variable, set to "sphinx" if sphinx is install

Re: [v3] Minimally exercise the other alias_decls in

2014-10-15 Thread Mike Stump
On Oct 15, 2014, at 8:41 AM, Paolo Carlini wrote: > ... the below replaces -std=c++0x and -std=c++11 with -std=gnu++11. Seems odd to go from c++11 -> gnu++11. Why?

Re: [v3] Minimally exercise the other alias_decls in

2014-10-15 Thread Mike Stump
On Oct 15, 2014, at 3:30 AM, Paolo Carlini wrote: > On 10/15/2014 11:50 AM, Paolo Carlini wrote: >>> We might also want to consider a global s/gnu++0x/gnu++11/ some day. >> And s/gnu++1y/gnu++14 of course ;) > I'm going to commit the below. Yes please, thanks.

Re: [PATCH 1/5] libiberty: Expose choose_tmpdir, and fix constness of return type

2014-10-15 Thread Jeff Law
On 10/13/14 11:45, David Malcolm wrote: The jit needs to create a temporary directory, so I need to expose libiberty's choose_tmpdir as a public entrypoint. include/ChangeLog: * libiberty.h (choose_tmpdir): New prototype. libiberty/ChangeLog: * choose-temp.c (choose_tmpdir): Rem

Re: [PATCH 1/2] Revert PR49721's patch

2014-10-15 Thread Jeff Law
On 10/13/14 16:35, Andrew Pinski wrote: On Fri, Aug 8, 2014 at 8:51 PM, Andrew Pinski wrote: OK? When the second patch is approved? Ping? Nevermind my comments, both were approved in the last day or two. jeff

Re: [PATCH 1/2] Revert PR49721's patch

2014-10-15 Thread Andrew Pinski
On Wed, Oct 15, 2014 at 10:29 AM, Jeff Law wrote: > On 10/13/14 16:35, Andrew Pinski wrote: >> >> On Fri, Aug 8, 2014 at 8:51 PM, Andrew Pinski wrote: >>> >>> OK? When the second patch is approved? >> >> >> Ping? >> >>> >>> Thanks, >>> Andrew Pinski >>> >>> ChangeLog: >>> Revert: >>>

Re: [PATCH 1/2] Revert PR49721's patch

2014-10-15 Thread Jeff Law
On 10/13/14 16:35, Andrew Pinski wrote: On Fri, Aug 8, 2014 at 8:51 PM, Andrew Pinski wrote: OK? When the second patch is approved? Ping? Thanks, Andrew Pinski ChangeLog: Revert: 2011-08-19 H.J. Lu PR middle-end/49721 * explow.c (convert_memory_addr

[C PATCH] Clamp down "incomplete type" error (PR c/63543)

2014-10-15 Thread Marek Polacek
We've got a complaint that the "dereferencing pointer to incomplete type" error is printed for all occurrences of the incomplete type, which is too verbose. Also it'd be nicer to print the type as well. This patch fixes this; if we find an incomplete type, mark it with error node, then we don't pr

Re: [PATCH PING]Improve induction variable elimination

2014-10-15 Thread Mike Stump
On Oct 15, 2014, at 6:36 AM, Richard Biener wrote: > + wide_int max_wi = wi::max_value (TYPE_PRECISION (type), UNSIGNED); > + max_val = wi::to_widest (wide_int_to_tree (type, max_wi)); > > ick - there must be a better way to "extend" max_wi to "infinite" > precision. Mike? widest_in

Re: New rematerialization sub-pass in LRA

2014-10-15 Thread Michael Meissner
On Tue, Oct 14, 2014 at 07:28:45PM -0400, Vladimir Makarov wrote: > On 2014-10-14 4:17 PM, Peter Bergner wrote: > >On Fri, 2014-10-10 at 11:02 -0400, Vladimir Makarov wrote: > >>Here is a new rematerialization sub-pass of LRA. > > > >When Mike and I build with this patch along with the patch that >

Re: [PATCH][0/n] Merge from match-and-simplify

2014-10-15 Thread Jakub Jelinek
On Wed, Oct 15, 2014 at 03:00:57PM +0200, Richard Biener wrote: > To re-iterate here (as I expect most people will only look > at [0/n] patches ;)), the question is whether we are fine > with making fold-const (thus fold_{unary,binary,ternary}) > not handle some cases it handles currently. I'd say

Re: [PATCH 3/5] IPA ICF pass

2014-10-15 Thread Martin Liška
On 10/14/2014 06:04 PM, Jan Hubicka wrote: diff --git a/gcc/cgraph.h b/gcc/cgraph.h index fb41b01..2de98b4 100644 --- a/gcc/cgraph.h +++ b/gcc/cgraph.h @@ -172,6 +172,12 @@ public: /* Dump referring in list to FILE. */ void dump_referring (FILE *); + /* Get number of references for thi

[PATCH 09/10] Prebuilt texinfo documentation for the JIT library (Re: Patches 5-10 of jit merger)

2014-10-15 Thread David Malcolm
On Tue, 2014-10-14 at 11:09 -0400, David Malcolm wrote: > On Mon, 2014-10-13 at 13:45 -0400, David Malcolm wrote: > > I'd like to merge the JIT branch into trunk: > > https://gcc.gnu.org/wiki/JIT > > > > This is "v2" since it incorporates fixes for the various issues > > identified by Joseph in

Fix PR lto/62026 on mainline

2014-10-15 Thread Jan Hubicka
Hi, this is mainline version of the fix for thunk creation I comitted to gcc-4.9 branch week ago. Bootstrapped/regtested x86_64-linux, comitted. Honza PR lto/62026 * lto/pr62026.C: New testcase. * cgraphclones.c (duplicate_thunk_for_node): Get body to have args t

Re: [PATCH 4/n] OpenMP 4.0 offloading infrastructure: lto-wrapper

2014-10-15 Thread Cary Coutant
> My preference would be to add the | SECTION_EXCLUDE unconditionally, and > instead guard the > if (flags & SECTION_EXCLUDE) > *f++ = 'e'; > in varasm.c (default_elf_asm_named_section). The only other user of > SECTION_EXCLUDE seems to be -gsplit-dwarf right now, Cary, is such a change > ok

[PATCH 08/10] Documentation for the JIT library (Re: Patches 5-10 of jit merger)

2014-10-15 Thread David Malcolm
On Tue, 2014-10-14 at 11:09 -0400, David Malcolm wrote: > On Mon, 2014-10-13 at 13:45 -0400, David Malcolm wrote: > > I'd like to merge the JIT branch into trunk: > > https://gcc.gnu.org/wiki/JIT > > > > This is "v2" since it incorporates fixes for the various issues > > identified by Joseph in

Re: [PATCH x86, java, PR63536] Fix java bootstrap for -mtune=intel/slm

2014-10-15 Thread Jakub Jelinek
On Wed, Oct 15, 2014 at 05:57:22PM +0100, Andrew Haley wrote: > On 10/15/2014 05:54 PM, Evgeny Stupachenko wrote: > > The patch fixes java i686 bootstrap for -mtune=intel/slm. > > > > Recent changes triggered java to write a note on compilation for a > > function without context. > > > > make che

Re: [arm][patch] fix arm_neon_ok check on !arm_arch7

2014-10-15 Thread Andrew Stubbs
On 15/10/14 17:34, Jiong Wang wrote: On 23/09/14 16:22, Stubbs, Andrew wrote: Maybe the original patch is better? Or maybe it should reconfigure the FPU instead of erroring out? But reconfigure it to what? Andrew, are you still working on this? a bunch of tests on my local environment

Re: [PATCH x86, java, PR63536] Fix java bootstrap for -mtune=intel/slm

2014-10-15 Thread Andrew Haley
On 10/15/2014 05:54 PM, Evgeny Stupachenko wrote: > The patch fixes java i686 bootstrap for -mtune=intel/slm. > > Recent changes triggered java to write a note on compilation for a > function without context. > > make check in progress > > Is it ok? I guess so, but I don't understand how any fu

[PATCH x86, java, PR63536] Fix java bootstrap for -mtune=intel/slm

2014-10-15 Thread Evgeny Stupachenko
Hi, The patch fixes java i686 bootstrap for -mtune=intel/slm. Recent changes triggered java to write a note on compilation for a function without context. make check in progress Is it ok? ChangeLog: 2014-10-15 Evgeny Stupachenko PR target/63536 * java/lang.c (java_print_er

Re: [PATCHv3][Kasan][PING^3] Allow to override Asan shadow offset from command line

2014-10-15 Thread Jakub Jelinek
On Wed, Oct 15, 2014 at 08:17:39PM +0400, Yury Gribov wrote: > +static unsigned HOST_WIDE_INT asan_shadow_offset_value; > +static bool asan_shadow_offset_computed; > + > +/* Sets shadow offset to value in string VAL. */ > + > +bool > +set_asan_shadow_offset (const char *val) > +{ > + char *endp;

Re: [arm][patch] fix arm_neon_ok check on !arm_arch7

2014-10-15 Thread Jiong Wang
On 23/09/14 16:22, Stubbs, Andrew wrote: Maybe the original patch is better? Or maybe it should reconfigure the FPU instead of erroring out? But reconfigure it to what? Andrew, are you still working on this? a bunch of tests on my local environment failed because of the reason James m

Re: New rematerialization sub-pass in LRA

2014-10-15 Thread Vladimir Makarov
On 2014-10-14 12:01 PM, Wilco Dijkstra wrote: Vladimir Makarov wrote: On SPECINT2k performance is ~0.5% worse (5.5% regression on perlbmk), and SPECFP is ~0.2% faster. Thanks for reporting this. It is important for me as I have no aarch64 machine for benchmarking. Perlbmk performance degradat

Re: [PATCH][0/n] Merge from match-and-simplify

2014-10-15 Thread Kyrill Tkachov
On 15/10/14 14:00, Richard Biener wrote: Any comments and reviews welcome (I don't think that my maintainership covers enough to simply check this in without approval). Hi Richard, The match-and-simplify branch bootstrapped successfully on aarch64-none-linux-gnu FWIW. Thanks, Kyrill

[RFC, PATCH]: Introduction of callgraph annotation class

2014-10-15 Thread Martin Liška
Hello. Following patch introduces a new class called callgraph_annotation. Idea behind the patch is to provide a generic interface one can use to register custom info related to a cgraph_node. As you know, symbol_table provides hooks for creation, deletion and duplication of a cgraph_node. If

Re: [PATCH i386 AVX512] [75/n] Update vec_init.

2014-10-15 Thread Jakub Jelinek
On Thu, Oct 09, 2014 at 04:13:25PM +0400, Kirill Yukhin wrote: > --- a/gcc/config/i386/i386.c > +++ b/gcc/config/i386/i386.c > @@ -39791,6 +39791,8 @@ ix86_expand_vector_init_duplicate (bool mmx_ok, enum > machine_mode mode, > case V8SFmode: > case V8SImode: > case V2DFmode: > +

[gomp4] OpenACC / C++

2014-10-15 Thread James Norris
Hi! This patch adds OpenACC support to C++ in the gomp4 branch. OK to apply? Thanks! Jim ChangeLog entries... = gcc/ChangeLog.gomp 2014-10-15 James Norris * builtin-types.def (BT_FN_VOID_INT_PTR_INT): New type. * oacc-builtins.def (BUILT_IN_GOACC_WAIT): New built

[RFC] Compile-time gimple-checking (again)

2014-10-15 Thread David Malcolm
Back in March I posted an 89-patch kit to expand and make use of the gimple statement class hierarchy to move much of the type-checking of statement accessors to be at compile-time rather than run-time: https://gcc.gnu.org/ml/gcc-patches/2014-04/msg01148.html I'd like to get these patches into

[PATCHv3][Kasan][PING^3] Allow to override Asan shadow offset from command line

2014-10-15 Thread Yury Gribov
On 10/06/2014 03:17 PM, Yury Gribov wrote: On 10/06/2014 03:06 PM, Yury Gribov wrote: On 09/29/2014 09:21 PM, Yury Gribov wrote: Hi all, Kasan developers has asked for an option to override offset of Asan shadow memory region. This should simplify experimenting with memory layouts on 64-bit a

Re: [PATCH, Pointer Bounds Checker 29/x] Debug info

2014-10-15 Thread Jeff Law
On 10/15/14 09:54, Ilya Enkovich wrote: I looked into sdbout.c code and didn't find any places requiring fixes. It seems it just ignores cases when it doesn't know what to output. Perfect. I really don't want to invest developer time into significantly improving sdbout or dbxout. If sdbout i

Re: update address taken: don't drop clobbers

2014-10-15 Thread Jeff Law
On 10/15/14 08:35, Marc Glisse wrote: Would that extra pass be acceptable? Otherwise, what do you think should be responsible for cleaning up the dead assignments? Does anyone have an opinion on which side needs to be improved? As a reminder: - we have a va_list with its address taken by va_

Re: [PATCH, Pointer Bounds Checker 29/x] Debug info

2014-10-15 Thread Ilya Enkovich
2014-06-18 0:56 GMT+04:00 Jeff Law : > On 06/11/14 02:50, Ilya Enkovich wrote: >> >> Hi, >> >> This patch skips all bounds during debug info generation. >> >> Bootstrapped and tested on linux-x86_64. >> >> Thanks, >> Ilya >> -- >> gcc/ >> >> 2014-06-11 Ilya Enkovich >> >> * dbxout.c (dbx

patch to fix PR63448

2014-10-15 Thread Vladimir Makarov
The following patch solves PR63448. The problem is described on https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63448 The patch is safe and was bootstrapped and tested on x86/x86-64. Committed as rev. 216270 to the trunk and as rev. 216271 to gcc-4.9 branch. 2014-10-15 Vladimir Makarov

libgomp testsuite: (not) using a specific driver for C++, Fortran?

2014-10-15 Thread Thomas Schwinge
Hi! No matter whether it's C, C++, or Fortran source code, the libgomp testsuite always uses (for build-tree testing) gcc/xgcc, or (for installed testing) GCC_UNDER_TEST. It doesn't make use of GXX_UNDER_TEST, GFORTRAN_UNDER_TEST. To support the latter two languages' needs, some -l[...] flags ar

Re: [PATCH] Better tolerance of incoming profile insanities in jump threading

2014-10-15 Thread Jeff Law
On 10/14/14 23:09, Teresa Johnson wrote: The below patch fixes the overflow detection when recomputing probabilities after jump threading, in case of incoming profile insanities. It detects more cases where the computation will overflow not only the max probability but the max int and possibly wr

Re: [PATCH 7/n] OpenMP 4.0 offloading infrastructure: testsuite

2014-10-15 Thread Jakub Jelinek
On Wed, Oct 15, 2014 at 07:28:37PM +0400, Ilya Verbin wrote: > On 15 Oct 17:05, Jakub Jelinek wrote: > > > This patch adds all examples with '#pragma omp target' from [1] to libgomp > > > testsuite. Without an accelerator or emulator, these tests are > > > UNSUPPORTED. > > > > Why? Most of the

Re: [PATCH 7/n] OpenMP 4.0 offloading infrastructure: testsuite

2014-10-15 Thread Ilya Verbin
On 15 Oct 17:05, Jakub Jelinek wrote: > > This patch adds all examples with '#pragma omp target' from [1] to libgomp > > testsuite. Without an accelerator or emulator, these tests are UNSUPPORTED. > > Why? Most of the tests should work just fine with host fallback. > Yeah, from Examples 4.0.1 I'

[ping] account for register spans in expand_builtin_init_dwarf_reg_sizes

2014-10-15 Thread Olivier Hainque
Hello, ping for https://gcc.gnu.org/ml/gcc-patches/2014-09/msg02625.html a patch proposal to fix exception propagation on spe powerpc targets. The idea is to improve expand_builtin_init_dwarf_reg_sizes to account for register spans as other functions in dwarf2cfi.c do, by extracting the processi

Re: [PATCH 7/n] OpenMP 4.0 offloading infrastructure: testsuite

2014-10-15 Thread Jakub Jelinek
On Wed, Oct 15, 2014 at 06:57:52PM +0400, Ilya Verbin wrote: > Hello, > > This patch adds all examples with '#pragma omp target' from [1] to libgomp > testsuite. Without an accelerator or emulator, these tests are UNSUPPORTED. Why? Most of the tests should work just fine with host fallback. Yea

[PATCH 7/n] OpenMP 4.0 offloading infrastructure: testsuite

2014-10-15 Thread Ilya Verbin
Hello, This patch adds all examples with '#pragma omp target' from [1] to libgomp testsuite. Without an accelerator or emulator, these tests are UNSUPPORTED. Bootstrapped and regtested on i686-linux and x86_64-linux. Is it OK for trunk? [1] http://openmp.org/mp-documents/OpenMP_Examples_4.0.1.p

Re: [PATCH][AARCH64][03]Add ACLE 2.0 predefined macros: Define __ARM_SIZEOF_MINIMAL_ENUM macro

2014-10-15 Thread Richard Earnshaw
On 15/10/14 14:44, Renlin Li wrote: > Hi all, > > This patch is part of a series of patches that implement ACLE 2.0 > predefined macros support for AArch64. > > aarch64-none-elf target has been tested on the model, no regression. > > Is it Okay to commit? > > Kind regards! > Renlin Li > > gcc

Re: [PATCH][AARCH64][02]Add ACLE 2.0 predefined macros: More macros which are unconditional

2014-10-15 Thread Richard Earnshaw
On 15/10/14 14:39, Renlin Li wrote: > Hi all, > > This patch is part of a series of patches that implement ACLE 2.0 > predefined macros support for AArch64. > > aarch64-none-elf target has been tested on the model, no regression. > > Is it Okay to commit? > > Kind regards! > Renlin Li > > gcc

[PATCH] PR lto/61048 Write/read option -fsanitize to/from object files

2014-10-15 Thread Ilya Palachev
Hi all, The attached patch fixes PR lto/61048. The basic idea is to write option -fsanitize to existing ELF section .gnu.lto_.opts in object files and then read it in lto-wrapper. On 15.10.2014 12:46, Richard Biener wrote: You need to handle them in lto-opts.c and output them to the existing

Re: [PATCH][AARCH64][01]Add ACLE 2.0 predefined marcos: Add __ARM_BIG_ENDIAN marco

2014-10-15 Thread Richard Earnshaw
On 15/10/14 14:35, Renlin Li wrote: > Hi all, > > This patch is part of a series of patches that implement ACLE 2.0 > predefined macro support for AArch64. > > aarch64-none-elf target has been tested on the model, no regression. > > Is it Okay to commit? > > Kind regards! > Renlin Li > > gcc/

Re: [PATCH 4/n] OpenMP 4.0 offloading infrastructure: lto-wrapper

2014-10-15 Thread Jakub Jelinek
On Wed, Oct 15, 2014 at 06:18:56PM +0400, Ilya Verbin wrote: > On 14 Oct 11:40, Jakub Jelinek wrote: > > My preference would be to add the | SECTION_EXCLUDE unconditionally, and > > instead guard the > > if (flags & SECTION_EXCLUDE) > > *f++ = 'e'; > > in varasm.c (default_elf_asm_named_secti

Re: [PATCH 1/2] [AARCH64,NEON] Add patterns + builtins for vld[234](q?)_lane_* intrinsics

2014-10-15 Thread Charles Baylis
On 9 October 2014 16:03, Tejas Belagod wrote: >> >> +(define_insn "vec_load_lanesoi_lane" > > > Best to prepend "aarch64_" the pattern name, IMHO, else it looks like a > standard pattern name(eg. vec_load_lanes) at first glance. > > Otherwise, LGTM(but I can't approve it). Thanks for this patch.

  1   2   >