Re: The nvptx port [0/11+]

2015-02-18 Thread Thomas Schwinge
Hi! On Mon, 20 Oct 2014 16:17:56 +0200, Bernd Schmidt ber...@codesourcery.com wrote: This is a patch kit that adds the nvptx port to gcc. Committed to trunk in r220781: commit 0f7695734890f93fe58179e36ac2f41bf4147d78 Author: tschwinge tschwinge@138bc75d-0d04-0410-961f-82ee72b054a4 Date: Wed

Re: nvptx-tools and nvptx-newlib (was: The nvptx port [10/11+] Target files)

2015-02-18 Thread Thomas Schwinge
Hi! On Wed, 4 Feb 2015 10:43:14 +0100, Jakub Jelinek ja...@redhat.com wrote: On Mon, Feb 02, 2015 at 04:32:34PM +0100, Thomas Schwinge wrote: Hi! On Tue, 23 Dec 2014 19:49:35 +0100, I wrote: On Mon, 10 Nov 2014 17:19:57 +0100, Bernd Schmidt ber...@codesourcery.com wrote: The

Re: [PATCH, AArch64] Handle SYMBOL_SMALL_TPREL appropriately

2015-02-18 Thread Marcus Shawcroft
On 18 February 2015 at 04:45, Hurugalawadi, Naveen naveen.hurugalaw...@caviumnetworks.com wrote: Hi Marcus, Thanks for the review. OK, but fix the trailing white space in the patch Done. Committed with the modification. Can you prepare a backport into 4.9 ILP32 support is not

Re: [PATCH][C++] PR 65071

2015-02-18 Thread Andrea Azzarone
Ops, forgot the diff. 2015-02-18 9:19 GMT+01:00 Andrea Azzarone azzaro...@gmail.com: Hi all, this patch try to fix PR c++/65071 (ICE on valid, sizeof...() of template template parameter pack in return type). 2015-2-18 Andrea Azzarone azzaro...@gmail.com PR c++/65071 * gcc/cp/parser.c

Re: patch to fix rtl documentation for new floating point comparisons

2015-02-18 Thread Andreas Schwab
Joseph Myers jos...@codesourcery.com writes: For example, on MIPS the C.cond.fmt instruction has a four-bit condition field: In the cond field of the instruction: cond 2..1 specify the nature of the comparison (equals, less than, and so on); cond 0 specifies whether the comparison is

Re: [Haifa Scheduler] Fix latent bug in macro-fusion/instruction grouping

2015-02-18 Thread Maxim Kuvyrkov
On Feb 17, 2015, at 9:43 AM, Jeff Law l...@redhat.com wrote: On 02/11/15 02:20, James Greenhalgh wrote: On Mon, Feb 09, 2015 at 11:16:56PM +, Jeff Law wrote: On 02/06/15 05:24, James Greenhalgh wrote: --- 2015-02-06 James Greenhalgh james.greenha...@arm.com * haifa-sched.c

[PATCH][C++] PR 65071

2015-02-18 Thread Andrea Azzarone
Hi all, this patch try to fix PR c++/65071 (ICE on valid, sizeof...() of template template parameter pack in return type). 2015-2-18 Andrea Azzarone azzaro...@gmail.com PR c++/65071 * gcc/cp/parser.c (cp_parser_sizeof_pack) Also consider template template parameters. Thanks. -- Andrea

nvptx mkoffload: For non-installed testing, look in all COMPILER_PATHs for GCC_INSTALL_NAME (was: nvptx offloading patches [4/n])

2015-02-18 Thread Thomas Schwinge
Hi! On Sat, 1 Nov 2014 13:11:29 +0100, Bernd Schmidt ber...@codesourcery.com wrote: [nvptx mkoffload] To support the --enable-offload-targets=nvptx-none=[install directory] configuration option, I committed the following to trunk in r220782 (and filed https://gcc.gnu.org/PR65097): commit

[PATCH] Fix IFN_OBJECT_SIZE expansion (PR sanitizer/65081)

2015-02-18 Thread Marek Polacek
We're lacking the POINTER_DIFF_EXPR, which means that ptr - 1 is in fact ptr + very_big_number. This can result in bogus run-time error when the objsz checking is turned on. Jakub suggested to not to issue the error if (ptr ptr + offset) is true. So this patch attemps to do that, along with

Re: [PATCH] Fix IFN_OBJECT_SIZE expansion (PR sanitizer/65081)

2015-02-18 Thread Jakub Jelinek
On Wed, Feb 18, 2015 at 10:15:03AM +0100, Marek Polacek wrote: We're lacking the POINTER_DIFF_EXPR, which means that ptr - 1 is in fact ptr + very_big_number. This can result in bogus run-time error when the objsz checking is turned on. Jakub suggested to not to issue the error if (ptr ptr

[Patch] Fix android build.

2015-02-18 Thread Ilya Tocar
Hi, On android dlerror returns const char*. Ok for trunk? libgomp/ * target.c (gomp_load_plugin_for_device): Fix type of dlerror return value. (DLSYM_OPT): Ditto. --- libgomp/target.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git

[PATCH][AArch64] Fix wrong-code bug in right-shift SISD patterns

2015-02-18 Thread Kyrill Tkachov
Hi all, This patch fixes a wrong-code bug with the *aarch64_lshr_sisd_or_int_mode3 pattern and its associated splitters. The problem is that for the 2nd alternative it will split a right-shift into a SISD left-shift by the negated amount to be shifted by (the ushl instruction allows such

Re: nvptx-tools and nvptx-newlib (was: The nvptx port [10/11+] Target files)

2015-02-18 Thread Jakub Jelinek
On Wed, Feb 18, 2015 at 09:50:15AM +0100, Thomas Schwinge wrote: What about multilibs, is newlib built for both -m32 and -m64, or just the default option? So far, we have concentrated only on the 64-bit x86_64 configuration; 32-bit has several known issues to be resolved.

Re: nvptx offloading patches [3/n], RFD

2015-02-18 Thread Jakub Jelinek
On Wed, Feb 18, 2015 at 10:12:19AM +0100, Thomas Schwinge wrote: On Tue, 17 Feb 2015 17:40:33 +0100, Jakub Jelinek ja...@redhat.com wrote: On Tue, Feb 17, 2015 at 04:21:06PM +, Joseph Myers wrote: On Tue, 17 Feb 2015, Jakub Jelinek wrote: I have nvptx-newlib symlinked into the gcc

Re: [RFC, PATCH] LTO: IPA inline speed up for large apps (Chrome)

2015-02-18 Thread Martin Liška
On 02/17/2015 07:38 PM, Jan Hubicka wrote: Hi, thanks for working on it. There are 3 basically indpeendent changes in the patch - The patch to make checking in lto_streamer_init ENABLE_CHECKING only that I think can be comitted as obvoius. Hello. Following email contains fix for that,

Re: [PATCH] Use !implicit_section in the recent set_section change (PR ipa/65087)

2015-02-18 Thread Jan Hubicka
On 2015.02.17 at 22:00 +0100, Jan Hubicka wrote: Hi! Markus reported an ICE, that is fixed by following patch, which limits the earlier change to !implicit_section only (which I assume is the user supplied __attribute__((section (. Bootstrapped/regtested on

hashtable optimization

2015-02-18 Thread François Dumont
Hello I am still studying hashtable performances and especially how to reduce overhead compared to tr1 implementation. Most of the overhead is coming from the additional modulo operations required with the new data model. Having a closer look at PR 57885 bench I realized that we can

Re: patch to fix rtl documentation for new floating point comparisons

2015-02-18 Thread Joseph Myers
On Tue, 17 Feb 2015, Kenneth Zadeck wrote: The fp exceptions raise some very tricky issues with respect to gcc and optimization. On many machines, noisy does not mean to throw an exception, it means that you set a bit and then check later. If you try to model this kind of behavior in

Re: nvptx offloading patches [3/n], RFD

2015-02-18 Thread Thomas Schwinge
Hi! On Mon, 16 Feb 2015 22:08:12 +0100, Jakub Jelinek ja...@redhat.com wrote: On Mon, Feb 09, 2015 at 11:20:00AM +0100, Richard Biener wrote: I think (also communicated that on IRC) we should instead try not streaming machine-modes at all but generating them at stream-in time via layout_type

Re: nvptx offloading patches [3/n], RFD

2015-02-18 Thread Thomas Schwinge
Hi! On Tue, 17 Feb 2015 17:40:33 +0100, Jakub Jelinek ja...@redhat.com wrote: On Tue, Feb 17, 2015 at 04:21:06PM +, Joseph Myers wrote: On Tue, 17 Feb 2015, Jakub Jelinek wrote: I have nvptx-newlib symlinked into the gcc tree as newlib, so I expected it would be built in-tree, is

Re: [Patch] Fix android build.

2015-02-18 Thread Jakub Jelinek
On Wed, Feb 18, 2015 at 01:59:25PM +0300, Ilya Tocar wrote: Hi, On android dlerror returns const char*. Looks like POSIX violation. Ok for trunk? libgomp/ * target.c (gomp_load_plugin_for_device): Fix type of dlerror return value. (DLSYM_OPT): Ditto. Therefore, I

Re: nvptx offloading patches [3/n], RFD

2015-02-18 Thread Jakub Jelinek
On Wed, Feb 18, 2015 at 10:12:19AM +0100, Thomas Schwinge wrote: Do you literally have »nvptx-newlib symlinked into the gcc tree as newlib«? If yes, then that should explain the problem: as I wrote in http://news.gmane.org/find-root.php?message_id=%3C87egq8mir1.fsf%40schwinge.name%3E, you

Re: hashtable optimization

2015-02-18 Thread François Dumont
With patch. On 18/02/2015 10:35, François Dumont wrote: Hello I am still studying hashtable performances and especially how to reduce overhead compared to tr1 implementation. Most of the overhead is coming from the additional modulo operations required with the new data model. Having a

Re: nvptx offloading patches [3/n], RFD

2015-02-18 Thread Jakub Jelinek
On Tue, Feb 17, 2015 at 11:00:14AM +0100, Richard Biener wrote: I'm just looking for a way to make this less of a hack (and the LTO IL less target dependent). Not for GCC 5 for which something like your patch is probably ok, but for the future. So, given Ilya's and Thomas' testing, is this

[PATCH] Fix PR62217

2015-02-18 Thread Richard Biener
The following patch extends a heuristic in DOM that avoids propagating copies into IV increments to cover all BIV replacements. This avoids the extra loop copy complete peeling produces and would have also avoided the array bound warning had we not disabled them completely from VRP2.

Re: ping #3: [RFA] Add --with-libz-prefix option in config/zlib.m4

2015-02-18 Thread Mike Frysinger
On 18 Feb 2015 04:56, H.J. Lu wrote: On Wed, Feb 18, 2015 at 4:08 AM, Joel Brobecker brobec...@adacore.com wrote: On Wed, Jan 07, 2015 at 06:45:48PM +0400, Joel Brobecker wrote: This patch enhances config/zlib.m4 to introduce an extra option --with-libz-prefix which allows us to provide the

Re: ping #3: [RFA] Add --with-libz-prefix option in config/zlib.m4

2015-02-18 Thread Joel Sherrill
On 2/18/2015 10:54 AM, Mike Frysinger wrote: On 18 Feb 2015 04:56, H.J. Lu wrote: On Wed, Feb 18, 2015 at 4:08 AM, Joel Brobecker brobec...@adacore.com wrote: On Wed, Jan 07, 2015 at 06:45:48PM +0400, Joel Brobecker wrote: This patch enhances config/zlib.m4 to introduce an extra option

Re: ping #3: [RFA] Add --with-libz-prefix option in config/zlib.m4

2015-02-18 Thread H.J. Lu
On Wed, Feb 18, 2015 at 8:54 AM, Mike Frysinger vap...@gentoo.org wrote: On 18 Feb 2015 04:56, H.J. Lu wrote: On Wed, Feb 18, 2015 at 4:08 AM, Joel Brobecker brobec...@adacore.com wrote: On Wed, Jan 07, 2015 at 06:45:48PM +0400, Joel Brobecker wrote: This patch enhances config/zlib.m4 to

[PATCH, 4.8] Backport Fix debug-insn sensitivity in RA patch to 4.8

2015-02-18 Thread Uros Bizjak
Hello! Richard's patch at [1] is needed to fix a bootstrap failure on alpha-linux-gnu on 4.8 branch. Without the patch, IRA creates different sequences, depending on the presence of -g option. One of the many comparison failures is in expr.c, where -fdebug-compare fails with: --- expr.gkd

Re: [Haifa Scheduler] Fix latent bug in macro-fusion/instruction grouping

2015-02-18 Thread Jeff Law
On 02/18/15 01:03, Maxim Kuvyrkov wrote: The way SCHED_GROUP_P instructions have been handled historically is by combination of two artifacts: (1) removing all dependencies for instructions inside SCHED_GROUP sequence but the one to next insn, and (2) maintaining a fast track for SCHED_GROUP

Re: [PATCH] PR target/65064: Return false for COMMON symbols

2015-02-18 Thread Richard Henderson
On 02/18/2015 05:18 AM, H.J. Lu wrote: PR target/65064 * config/ia64/predicates.md (sdata_symbolic_operand): Return false for common symbols. Ok. r~

Re: [PATCH 6/n] OpenMP 4.0 offloading infrastructure: option handling

2015-02-18 Thread Thomas Schwinge
Hi! On Mon, 13 Oct 2014 14:33:11 +0400, Ilya Verbin iver...@gmail.com wrote: On 13 Oct 12:19, Jakub Jelinek wrote: On Sat, Oct 11, 2014 at 06:49:00PM +0400, Ilya Verbin wrote: 2. -foffload-abi=[lp64|ilp32] This option is supposed to tell mkoffload (and offload compiler) which ABI

[PATCH] Put cleanups of cleanups after cleanups (PR gcov-profile/64634)

2015-02-18 Thread Jakub Jelinek
Hi! Richard's GIMPLE EH rewrite in r151696 regressed following testcase. The problem is that when lowering: [gcov-15.C:14:5] try { [gcov-15.C:18:12] D.2335 = __cxa_allocate_exception (4); [gcov-15.C:18:12] try { [gcov-15.C:18:12] [gcov-15.C:18:12] MEM[(int

Re: [PATCH] Use !implicit_section in the recent set_section change (PR ipa/65087)

2015-02-18 Thread Markus Trippelsdorf
On 2015.02.18 at 10:17 +0100, Jan Hubicka wrote: On 2015.02.17 at 22:00 +0100, Jan Hubicka wrote: Hi! Markus reported an ICE, that is fixed by following patch, which limits the earlier change to !implicit_section only (which I assume is the user supplied

[patch] Fix codecvtchar16_t, char, mbstate_t

2015-02-18 Thread Jonathan Wakely
While working on PR64797 I discovered that the codecvtchar16_t,... specialization was, erm, completely broken when creating UTF-16 surrogate pairs. This fixes it and adds a test, based on the char32_t one I added to the testsuite yesterday. Tested x86_64-linux (little-endian) and powerpc64-linux

Re: [PATCH] PR64959: SFINAE in UDLs

2015-02-18 Thread Alex Velenko
On 13/02/15 22:21, Andrea Azzarone wrote: We can use the same trick used in the other tests. Patch attached. Sorry about that! 2015-02-13 20:45 GMT+01:00 Jakub Jelinek ja...@redhat.com: On Wed, Feb 11, 2015 at 12:26:33AM +0100, Andrea Azzarone wrote: *

Re: [patch] Fix codecvtchar16_t, char, mbstate_t

2015-02-18 Thread Jakub Jelinek
On Wed, Feb 18, 2015 at 07:07:05PM +, Jonathan Wakely wrote: While working on PR64797 I discovered that the codecvtchar16_t,... specialization was, erm, completely broken when creating UTF-16 surrogate pairs. This fixes it and adds a test, based on the char32_t one I added to the

Re: [PATCH] PR64959: SFINAE in UDLs

2015-02-18 Thread Jakub Jelinek
On Wed, Feb 18, 2015 at 06:29:34PM +, Alex Velenko wrote: this patch also fixes issues for arm-none-eabi. Could someone add this patch? ENOPATCH Jakub

Re: ping #3: [RFA] Add --with-libz-prefix option in config/zlib.m4

2015-02-18 Thread Mike Frysinger
On 18 Feb 2015 08:58, H.J. Lu wrote: On Wed, Feb 18, 2015 at 8:54 AM, Mike Frysinger wrote: On 18 Feb 2015 04:56, H.J. Lu wrote: On Wed, Feb 18, 2015 at 4:08 AM, Joel Brobecker wrote: On Wed, Jan 07, 2015 at 06:45:48PM +0400, Joel Brobecker wrote: This patch enhances config/zlib.m4 to

Re: [PATCH 09/36] floatformat.h: Wrap in extern C.

2015-02-18 Thread Jakub Jelinek
On Thu, Feb 12, 2015 at 11:49:01AM +, Pedro Alves wrote: On 02/09/2015 11:49 PM, Pedro Alves wrote: On 02/09/2015 11:35 PM, Andrew Pinski wrote: On Mon, Feb 9, 2015 at 3:20 PM, Pedro Alves pal...@redhat.com wrote: Just like libiberty.h. So that C++ programs, such as GDB when built as

Re: ping #3: [RFA] Add --with-libz-prefix option in config/zlib.m4

2015-02-18 Thread H.J. Lu
On Wed, Feb 18, 2015 at 12:32 PM, Mark Wielaard m...@redhat.com wrote: On Wed, 2015-02-18 at 11:52 -0800, H.J. Lu wrote: On Wed, Feb 18, 2015 at 11:44 AM, Mike Frysinger vap...@gentoo.org wrote: On 18 Feb 2015 08:58, H.J. Lu wrote: On Wed, Feb 18, 2015 at 8:54 AM, Mike Frysinger wrote:

[committed] Add missing cleanup in gfortran.dg/pr37287-1.f90

2015-02-18 Thread Tom de Vries
Hi, After running gfortran tests, I found a pr37287_2.mod in my test directory. Fixed by this patch which adds a missing cleanup-modules. Tested by running the testcase and checking that the file does not occur anymore in the test directory. Committed as obvious. Thanks, - Tom 2015-02-18

[committed][PR65107] Add missing cleanup in gfortran.dg/read_eof_8.f90

2015-02-18 Thread Tom de Vries
Hi, I ran into a failure of gfortran.dg/eof_4.f90, due to the presence of test.dat. The contents of test.dat pointed to read_eof_8.f90, which indeed does not cleanup the test.dat it uses. This patch fixes that. Tested by running the test and checking that test.dat was not present anymore in

Re: [patch] Fix codecvtchar16_t, char, mbstate_t

2015-02-18 Thread Jonathan Wakely
On 18/02/15 19:07 +, Jonathan Wakely wrote: While working on PR64797 I discovered that the codecvtchar16_t,... specialization was, erm, completely broken when creating UTF-16 surrogate pairs. This fixes it and adds a test, based on the char32_t one I added to the testsuite yesterday. Tested

[committed] Add missing cleanup in gfortran.dg/fmt_cache_1.f

2015-02-18 Thread Tom de Vries
Hi, I found a fort.10 file in the test directories, and using the contents tracked it back to fmt_cache_1.f, which creates fort.10 but doesn't remove it. This patch fixes that. Tested by running the test-case and checking that fort.10 doesn't appear anymore in the test directory.

Re: ping #3: [RFA] Add --with-libz-prefix option in config/zlib.m4

2015-02-18 Thread Jakub Jelinek
On Wed, Feb 18, 2015 at 09:32:21PM +0100, Mark Wielaard wrote: Can you elaborate? That doesn't seem like a smart default. And why is is Linux/x86 only? Shouldn't that be something that is done explicitly by a distro configuring binutils after making sure it actually is beneficial

Re: [PATCH] Fix testsuite race on additional_sources

2015-02-18 Thread Jeff Law
On 02/18/15 08:05, Maxim Kuvyrkov wrote: Hi, This testsuite patch fixes race on additional_source testsuite variable. When a test has both dg-additional-sources and dg-do run { target FOO } directives, it may occur that the FOO test will attempt to use additional_sources, which will result

Re: ping #3: [RFA] Add --with-libz-prefix option in config/zlib.m4

2015-02-18 Thread Mark Wielaard
On Wed, 2015-02-18 at 11:52 -0800, H.J. Lu wrote: On Wed, Feb 18, 2015 at 11:44 AM, Mike Frysinger vap...@gentoo.org wrote: On 18 Feb 2015 08:58, H.J. Lu wrote: On Wed, Feb 18, 2015 at 8:54 AM, Mike Frysinger wrote: Why do you want to turn off zlib? On Linux/x86, zlib is required for

[committed] Add missing cleanup in gfortran.dg/finalize_28.f90

2015-02-18 Thread Tom de Vries
Hi, I found finalize_28.f90.003t.original in a gfortran test directory. This patch adds the missing cleanup-tree-dump. Tested by running the test-case and checking that the file does not occur anymore in the test directory. Committed as obvious. Thanks, - Tom 2015-02-18 Tom de Vries

[committed] Add missing cleanup in gfortran.dg/coarray_35a.f90

2015-02-18 Thread Tom de Vries
Hi, After running gfortran tests, I found a global_coarrays.mod file in my test directory. Fixed by this patch which adds a missing cleanup-modules. Tested by running the testcase and checking that the file does not occur anymore in the test directory. Committed as obvious. Thanks, - Tom

Re: [Patch] Add option ftree-stdarg-opt

2015-02-18 Thread Tom de Vries
On 17-02-15 13:26, Richard Biener wrote: On Tue, Feb 17, 2015 at 1:12 PM, Tom de Vries tom_devr...@mentor.com wrote: Hi, this patch adds option ftree-stdarg-opt, which switches pass_stdarg on or off. Pass_stdarg does an optimization on cfun-va_list_gpr/fpr_size, and since it's an

Re: [PATCH] rtl-optimization/64935: Sorting of ready list is different with/without DEBUG_INSNs.

2015-02-18 Thread Jeff Law
On 02/18/15 09:02, Maxim Kuvyrkov wrote: Hi, This patch fixes PR64935, which is triggered when ready list at the start of a basic block is greater than --param=max-sched-ready-insns. Sorting the ready list when it has more than max-sched-ready-insns elements is special in that we want to

Re: ping #3: [RFA] Add --with-libz-prefix option in config/zlib.m4

2015-02-18 Thread Mark Wielaard
On Wed, 2015-02-18 at 12:53 -0800, H.J. Lu wrote: On Wed, Feb 18, 2015 at 12:32 PM, Mark Wielaard m...@redhat.com wrote: That doesn't seem like a smart default. And why is is Linux/x86 only? Shouldn't that be something that is done explicitly by a distro configuring binutils after making

Re: [PATCH] Put cleanups of cleanups after cleanups (PR gcov-profile/64634)

2015-02-18 Thread Jeff Law
On 02/18/15 11:40, Jakub Jelinek wrote: Hi! Richard's GIMPLE EH rewrite in r151696 regressed following testcase. The problem is that when lowering: [gcov-15.C:14:5] try { [gcov-15.C:18:12] D.2335 = __cxa_allocate_exception (4); [gcov-15.C:18:12] try {

Re: ping #3: [RFA] Add --with-libz-prefix option in config/zlib.m4

2015-02-18 Thread H.J. Lu
On Wed, Feb 18, 2015 at 1:40 PM, Mark Wielaard m...@redhat.com wrote: On Wed, 2015-02-18 at 12:53 -0800, H.J. Lu wrote: On Wed, Feb 18, 2015 at 12:32 PM, Mark Wielaard m...@redhat.com wrote: That doesn't seem like a smart default. And why is is Linux/x86 only? Shouldn't that be something

Re: ping #3: [RFA] Add --with-libz-prefix option in config/zlib.m4

2015-02-18 Thread H.J. Lu
On Wed, Feb 18, 2015 at 2:21 PM, Mike Frysinger vap...@gentoo.org wrote: On 18 Feb 2015 13:54, H.J. Lu wrote: On Wed, Feb 18, 2015 at 1:40 PM, Mark Wielaard wrote: On Wed, 2015-02-18 at 12:53 -0800, H.J. Lu wrote: On Wed, Feb 18, 2015 at 12:32 PM, Mark Wielaard wrote: That doesn't seem

Re: [patch] fix PR65048: check that jump-thread paths are still valid

2015-02-18 Thread Sebastian Pop
Jeff Law wrote: These kinds of situations are normally pruned out in mark_threaded_blocks. I added the FSM code generation before calling mark_threaded_blocks. The dumps for the FSM threads are a bit sparse -- they don't show the entire path. That makes it much harder to see what's going

Re: ping #3: [RFA] Add --with-libz-prefix option in config/zlib.m4

2015-02-18 Thread Mark Wielaard
On Wed, Feb 18, 2015 at 01:54:17PM -0800, H.J. Lu wrote: On Wed, Feb 18, 2015 at 1:40 PM, Mark Wielaard m...@redhat.com wrote: On Wed, 2015-02-18 at 12:53 -0800, H.J. Lu wrote: On Wed, Feb 18, 2015 at 12:32 PM, Mark Wielaard m...@redhat.com wrote: That doesn't seem like a smart default.

Re: [RFA] Add --with-libz-prefix option in config/zlib.m4

2015-02-18 Thread Thomas Schwinge
Hi! On Thu, 19 Feb 2015 08:54:46 +0100, I wrote: On Wed, 7 Jan 2015 17:00:59 +0100, Tristan Gingold ging...@adacore.com wrote: On 07 Jan 2015, at 15:45, Joel Brobecker brobec...@adacore.com wrote: This patch enhances config/zlib.m4 to introduce an extra option --with-libz-prefix

Re: [RFA] Add --with-libz-prefix option in config/zlib.m4

2015-02-18 Thread Thomas Schwinge
Hi! On Wed, 7 Jan 2015 17:00:59 +0100, Tristan Gingold ging...@adacore.com wrote: On 07 Jan 2015, at 15:45, Joel Brobecker brobec...@adacore.com wrote: This patch enhances config/zlib.m4 to introduce an extra option --with-libz-prefix which allows us to provide the location of the zlib

Type comparing TLC

2015-02-18 Thread Jan Hubicka
Hi, looking across the ODR violation messages in libreoffice and Chromium I found some false positives and some confused messages. This patch fixes them. In partiuclar - I introduced nasty vtable corruption when breaking out my type merging patches, so we ended up creating separate entries

Re: [PATCH 1/4] Add mkoffload for Intel MIC

2015-02-18 Thread Thomas Schwinge
Hi! On Wed, 22 Oct 2014 22:57:01 +0400, Ilya Verbin iver...@gmail.com wrote: --- /dev/null +++ b/gcc/config/i386/intelmic-mkoffload.c +[...] +#include config.h +#include libgen.h +#include system.h +#include coretypes.h +#include obstack.h +#include intl.h +#include diagnostic.h

Re: [libstdc++/65033] Give alignment info to libatomic

2015-02-18 Thread Jonathan Wakely
On 12/02/15 13:23 -0800, Richard Henderson wrote: When we fixed PR54005, making sure that atomic_is_lock_free returns the same value for all objects of a given type, we probably should have changed the interface so that we would pass size and alignment rather than size and object pointer.

Re: Merge current set of OpenACC changes from gomp-4_0-branch

2015-02-18 Thread Ilya Verbin
On Wed, Feb 04, 2015 at 15:05:45 +, Julian Brown wrote: This (WIP) patch is based on top of a version of your patch that I merged to our internal branch: that's still the easiest way for me to test the PTX backend (with unloading support) at present, and it passes libgomp testing that way.

Re: [PATCH] Use automatic dependencies for mkoffload.o

2015-02-18 Thread Richard Biener
On Wed, 18 Feb 2015, Jakub Jelinek wrote: On Wed, Feb 18, 2015 at 02:56:07PM +0300, Ilya Verbin wrote: On Wed, Feb 18, 2015 at 12:48:21 +0100, Thomas Schwinge wrote: What is the rationale for the insn-modes.h order-only prerequisites for mkoffload.o? Is this simply to get past the build

Re: [wwwdocs] Porting to again

2015-02-18 Thread Marek Polacek
On Wed, Feb 18, 2015 at 01:16:55PM +0100, Jakub Jelinek wrote: On Wed, Feb 18, 2015 at 01:04:30PM +0100, Marek Polacek wrote: --- porting_to.html 10 Feb 2015 11:12:20 - 1.3 +++ porting_to.html 18 Feb 2015 12:01:50 - @@ -24,6 +24,17 @@ manner. Additions and suggestions for

Re: [PATCH][AArch64] Fix wrong-code bug in right-shift SISD patterns

2015-02-18 Thread Kyrill Tkachov
On 18/02/15 12:32, Maxim Kuvyrkov wrote: On Feb 18, 2015, at 2:35 PM, Kyrill Tkachov kyrylo.tkac...@arm.com wrote: Hi all, This patch fixes a wrong-code bug with the *aarch64_lshr_sisd_or_int_mode3 pattern and its associated splitters. The problem is that for the 2nd alternative it will

Re: [PATCH][AArch64] Fix wrong-code bug in right-shift SISD patterns

2015-02-18 Thread Kyrill Tkachov
On 18/02/15 13:46, Maxim Kuvyrkov wrote: On Feb 18, 2015, at 4:42 PM, Kyrill Tkachov kyrylo.tkac...@arm.com wrote: On 18/02/15 12:32, Maxim Kuvyrkov wrote: On Feb 18, 2015, at 2:35 PM, Kyrill Tkachov kyrylo.tkac...@arm.com wrote: Hi all, This patch fixes a wrong-code bug with the

Re: [PATCH 1/4] Add mkoffload for Intel MIC

2015-02-18 Thread Jakub Jelinek
On Wed, Feb 18, 2015 at 12:48:21PM +0100, Thomas Schwinge wrote: --- /dev/null +++ b/gcc/config/i386/t-intelmic @@ -0,0 +1,9 @@ +mkoffload.o: $(srcdir)/config/i386/intelmic-mkoffload.c | insn-modes.h + $(COMPILER) -c $(ALL_COMPILERFLAGS) $(ALL_CPPFLAGS) $(INCLUDES) \ +

Re: [patch, avr] Fix ICE PR64452 pushing eliminated rtxes

2015-02-18 Thread Georg-Johann Lay
Am 02/17/2015 um 03:34 PM schrieb Denis Chertykov: 2015-02-17 14:12 GMT+03:00 Georg-Johann Lay a...@gjlay.de: Byte-wise pushing virtual regs like arg pointer migth result in patterns like (set (mem:QI (post_dec:HI (reg:HI 32 SP))) (subreg:QI (plus:HI (reg:HI 28)

Re: [PATCH][AArch64] Fix wrong-code bug in right-shift SISD patterns

2015-02-18 Thread Maxim Kuvyrkov
On Feb 18, 2015, at 3:32 PM, Maxim Kuvyrkov maxim.kuvyr...@linaro.org wrote: First of all, applauses! I realize how difficult it was to reduce this problem. Your patch looks OK to me, but I can't shake off feeling that it will pessimize cases when d8 is not used afterwards. In

Re: [patch, avr] Fix ICE PR64452 pushing eliminated rtxes

2015-02-18 Thread Denis Chertykov
2015-02-18 14:59 GMT+03:00 Georg-Johann Lay a...@gjlay.de: Am 02/17/2015 um 03:34 PM schrieb Denis Chertykov: 2015-02-17 14:12 GMT+03:00 Georg-Johann Lay a...@gjlay.de: Byte-wise pushing virtual regs like arg pointer migth result in patterns like (set (mem:QI (post_dec:HI (reg:HI 32

Re: [PATCH] PR target/65064: Return false for COMMON symbols

2015-02-18 Thread H.J. Lu
On Sun, Feb 15, 2015 at 06:19:21AM -0800, H.J. Lu wrote: Hi, r220674 exposed a bug in ia64_in_small_data_p. After r220674, COMMON symbols binds locally for executables. But ia64_in_small_data_p returns true for COMMON symbols which are never in small data section. This patch fixes it.

Re: [PATCH] PR target/65064: Return false for COMMON symbols

2015-02-18 Thread H.J. Lu
On Wed, Feb 18, 2015 at 5:18 AM, H.J. Lu hjl.to...@gmail.com wrote: On Sun, Feb 15, 2015 at 06:19:21AM -0800, H.J. Lu wrote: Hi, r220674 exposed a bug in ia64_in_small_data_p. After r220674, COMMON symbols binds locally for executables. But ia64_in_small_data_p returns true for COMMON

Re: [RFC, PATCH] LTO: IPA inline speed up for large apps (Chrome)

2015-02-18 Thread Martin Liška
-checking=release Thread model: posix gcc version 5.0.0 20150218 (experimental) (GCC) marxin@marxinbox:~/Programming/gecko-dev/obj-x86_64-unknown-linux-gnu/toolkit/library perf report marxin@marxinbox:~/Programming/gecko-dev/obj-x86_64-unknown-linux-gnu/toolkit/library perf report --stdio | sed 's

Re: [PATCH] PR rtl-optimization/32219: optimizer causees wrong code in pic/hidden/weak symbol checking

2015-02-18 Thread Alex Velenko
On 13/02/15 05:11, Richard Henderson wrote: On 02/12/2015 08:14 PM, H.J. Lu wrote: I tried the second patch. Results look good on Linux/x86-64. Thanks. My results concurr. I went ahead and installed the patch as posted. r~ 2015-02-12 H.J. Lu hongjiu...@intel.com Richard

[wwwdocs] Porting to again

2015-02-18 Thread Marek Polacek
This is a revised version. I reworded the paragraph dealing with __STDC_VERSION__, made some clarifications wrt %a, and added some text wrt cpp -P issue. Ok? Index: porting_to.html === RCS file:

Re: [wwwdocs] Porting to again

2015-02-18 Thread Jakub Jelinek
On Wed, Feb 18, 2015 at 01:04:30PM +0100, Marek Polacek wrote: --- porting_to.html 10 Feb 2015 11:12:20 - 1.3 +++ porting_to.html 18 Feb 2015 12:01:50 - @@ -24,6 +24,17 @@ manner. Additions and suggestions for improvement are welcome. /p +h2Preprocessor issues/h2 +

[PATCH] Use automatic dependencies for mkoffload.o

2015-02-18 Thread Jakub Jelinek
On Wed, Feb 18, 2015 at 02:56:07PM +0300, Ilya Verbin wrote: On Wed, Feb 18, 2015 at 12:48:21 +0100, Thomas Schwinge wrote: What is the rationale for the insn-modes.h order-only prerequisites for mkoffload.o? Is this simply to get past the build issue which, for example, Jakub also

Re: ping #3: [RFA] Add --with-libz-prefix option in config/zlib.m4

2015-02-18 Thread H.J. Lu
On Wed, Feb 18, 2015 at 4:08 AM, Joel Brobecker brobec...@adacore.com wrote: Yay? Nay? Thank you. On Wed, Jan 07, 2015 at 06:45:48PM +0400, Joel Brobecker wrote: Hello, This patch enhances config/zlib.m4 to introduce an extra option --with-libz-prefix which allows us to provide the

Re: [patch] Warn on undefined loop exit

2015-02-18 Thread Jakub Jelinek
On Thu, Nov 20, 2014 at 05:27:35PM +0100, Richard Biener wrote: On Wed, Nov 19, 2014 at 9:19 PM, Andrew Stubbs a...@codesourcery.com wrote: On 19/11/14 16:39, Marek Polacek wrote: On Wed, Nov 19, 2014 at 04:32:43PM +, Andrew Stubbs wrote: +if (warning_at

Re: [PATCH 1/4] Add mkoffload for Intel MIC

2015-02-18 Thread Ilya Verbin
On Wed, Feb 18, 2015 at 12:48:21 +0100, Thomas Schwinge wrote: What is the rationale for the insn-modes.h order-only prerequisites for mkoffload.o? Is this simply to get past the build issue which, for example, Jakub also reported for the nvptx mkoffload,

Re: nvptx offloading patches [3/n], RFD

2015-02-18 Thread Thomas Schwinge
Hi Jakub! (Will respond to your other questions later.) On Wed, 18 Feb 2015 12:34:38 +0100, Jakub Jelinek ja...@redhat.com wrote: On Wed, Feb 18, 2015 at 10:12:19AM +0100, Thomas Schwinge wrote: Do you literally have »nvptx-newlib symlinked into the gcc tree as newlib«? If yes, then that

Re: nvptx offloading patches [3/n], RFD

2015-02-18 Thread Jakub Jelinek
On Wed, Feb 18, 2015 at 01:09:53PM +0100, Thomas Schwinge wrote: On Wed, 18 Feb 2015 12:34:38 +0100, Jakub Jelinek ja...@redhat.com wrote: On Wed, Feb 18, 2015 at 10:12:19AM +0100, Thomas Schwinge wrote: Do you literally have »nvptx-newlib symlinked into the gcc tree as newlib«? If yes,

Re: [PATCH][AArch64] Fix wrong-code bug in right-shift SISD patterns

2015-02-18 Thread Maxim Kuvyrkov
On Feb 18, 2015, at 4:42 PM, Kyrill Tkachov kyrylo.tkac...@arm.com wrote: On 18/02/15 12:32, Maxim Kuvyrkov wrote: On Feb 18, 2015, at 2:35 PM, Kyrill Tkachov kyrylo.tkac...@arm.com wrote: Hi all, This patch fixes a wrong-code bug with the *aarch64_lshr_sisd_or_int_mode3 pattern and

Re: [RFC, PATCH] LTO: IPA inline speed up for large apps (Chrome)

2015-02-18 Thread Martin Liška
On 02/17/2015 10:03 PM, Jan Hubicka wrote: Hi, this patch should chase away the expensive thunks and aliases walks from most of analysis code. I think only real use left is local_p predicate that needs to stay because i386 expect local flag to match between caller and callee when expanding

[PATCH] rtl-optimization/64935: Sorting of ready list is different with/without DEBUG_INSNs.

2015-02-18 Thread Maxim Kuvyrkov
Hi, This patch fixes PR64935, which is triggered when ready list at the start of a basic block is greater than --param=max-sched-ready-insns. Sorting the ready list when it has more than max-sched-ready-insns elements is special in that we want to sort normal insns even if there are debug

Re: [PATCH][ARM] PR target/64600 Fix another ICE with -mtune=xscale: properly sign-extend mask during constant splitting

2015-02-18 Thread Kyrill Tkachov
Ping. Thanks, Kyrill On 10/02/15 09:25, Kyrill Tkachov wrote: Ping. https://gcc.gnu.org/ml/gcc-patches/2015-02/msg00141.html Thanks, Kyrill On 03/02/15 15:18, Kyrill Tkachov wrote: Hi all, The ICE in this PR occurs when -mtune=xscale triggers a particular path through arm_gen_constant

Re: [PATCH][AArch64] Testcase fix for __ATOMIC_CONSUME

2015-02-18 Thread Alex Velenko
On 12/02/15 18:38, Mike Stump wrote: On Feb 11, 2015, at 12:16 PM, Torvald Riegel trie...@redhat.com wrote: On Mon, 2015-02-09 at 09:10 -0800, Mike Stump wrote: On Feb 9, 2015, at 7:11 AM, Alex Velenko alex.vele...@arm.com wrote: The following patch makes atomic-op-consume.c XFAIL Is this

Re: [PATCH][AArch64] Fix wrong-code bug in right-shift SISD patterns

2015-02-18 Thread Maxim Kuvyrkov
On Feb 18, 2015, at 2:35 PM, Kyrill Tkachov kyrylo.tkac...@arm.com wrote: Hi all, This patch fixes a wrong-code bug with the *aarch64_lshr_sisd_or_int_mode3 pattern and its associated splitters. The problem is that for the 2nd alternative it will split a right-shift into a SISD left-shift

[PATCH] Fix testsuite race on additional_sources

2015-02-18 Thread Maxim Kuvyrkov
Hi, This testsuite patch fixes race on additional_source testsuite variable. When a test has both dg-additional-sources and dg-do run { target FOO } directives, it may occur that the FOO test will attempt to use additional_sources, which will result in failure to compile FOO test. It often

Re: patch to fix rtl documentation for new floating point comparisons

2015-02-18 Thread Kenneth Zadeck
On Feb 18, 2015, at 3:23 AM, Joseph Myers jos...@codesourcery.com wrote: On Tue, 17 Feb 2015, Kenneth Zadeck wrote: The fp exceptions raise some very tricky issues with respect to gcc and optimization. On many machines, noisy does not mean to throw an exception, it means that you

Re: patch to fix rtl documentation for new floating point comparisons

2015-02-18 Thread Joseph Myers
On Wed, 18 Feb 2015, Kenneth Zadeck wrote: I think the norm is dynamic, because that's what was in IEEE 754-1985, with static rounding added more recently on some processors, because of IEEE 754-2008. (There are other variants - IA64 having multiple dynamic rounding mode registers and

Re: ping #3: [RFA] Add --with-libz-prefix option in config/zlib.m4

2015-02-18 Thread Mike Frysinger
On 18 Feb 2015 14:24, H.J. Lu wrote: On Wed, Feb 18, 2015 at 2:21 PM, Mike Frysinger wrote: i think we already have the reports: multiple people don't think it should be (1) x86-specific or (2) required. don't get me wrong -- i think having support like this is great. that doesn't

[PATCH] sem_function::bb_dict_test should take a vecint *

2015-02-18 Thread tbsaunde+gcc
From: Trevor Saunders tbsaunde+...@tbsaunde.org bb_dict_test () ment to operate on the callers vector, not a copy of it. Otherwise it either does nothing or crashes. approved by Honza off list committing to trunk (bootstrapped + regtested x86_64-linux-gnu). Trev gcc/ChangeLog: 2015-02-18

Re: ping #3: [RFA] Add --with-libz-prefix option in config/zlib.m4

2015-02-18 Thread Joel Brobecker
Why do you want to turn off zlib? On Linux/x86, zlib is required for assembler. At least, you should issue an error when --without-libz is used in binutils for Linux/x86 target. I am trying to do the exact opposite, which is to provide an option to compile WITH zlib, but using an install at

[PATCH] Fix for PR c++/60269

2015-02-18 Thread Iyer, Balaji V
Hello Everyone,     Attached, please find a patch that is a fix for PR c++/60269. Tested on x86_64 and have no regression issues. Is this OK for trunk? Thanks, Balaji V. Iyer. +2015-02-18  Balaji V. Iyer  balaji.v.i...@intel.com + +   PR c++/60269 +   * parser.c