Re: [gofrontend-dev] [PATCH 4/9] Gccgo port to s390[x] -- part I

2014-10-06 Thread Dominik Vogt
On Fri, Oct 03, 2014 at 06:19:46PM -0700, Ian Lance Taylor wrote: On Tue, Sep 9, 2014 at 5:53 AM, Dominik Vogt v...@linux.vnet.ibm.com wrote: This patch fixes the compiler flags in libgo/mksysinfo.sh. In one place, some compiler flags were missing that are consistently used elswhere,

Re: [RFC PATCH] Enable V32HI/V64QI const permutations

2014-10-06 Thread Jakub Jelinek
On Fri, Oct 03, 2014 at 04:39:08PM +0200, Jakub Jelinek wrote: Just to stress the new testcases some more, I've enabled the vec_perm_const{32hi,64qi} patterns. Got several ICEs in expand_vec_perm_broadcast_1, on the final gcc_unreachable () in the function. That function is only called if it

Re: [Fortran, Patch] Implement IMPLICIT NONE

2014-10-06 Thread Marek Polacek
On Thu, Oct 02, 2014 at 12:35:47AM +0200, Tobias Burnus wrote: Tobias Burnus wrote: IMPLICIT NONE (external) While I have implemented is as vendor extension (-std=gnu) Ups, I forgot to include the gcc/fortran/libgfortran.h change in the patch. See updated attachment. Seems that

Re: [Fortran, Patch] Implement IMPLICIT NONE

2014-10-06 Thread Tobias Burnus
Hi Marek, hi all, Tobias Burnus: Seems that gcc/fortran/libgfortran.h part is not committed, after r215914 I get gcc/fortran/decl.c:2960:28: error: âGFC_STD_F2015â was not declared in this scope Committing patches when having a cold doesn't seem to work. I don't have access to the computer

Re: [PATCH] Provide global var location info for asan

2014-10-06 Thread Dodji Seketeli
Hello, Jakub Jelinek ja...@redhat.com writes: 2014-09-24 Jakub Jelinek ja...@redhat.com * ubsan.h (ubsan_get_source_location): New prototype. * ubsan.c (ubsan_source_location_type): New variable. Function renamed to ... (ubsan_get_source_location_type): ... this.

Re: [Fortran, Patch] Implement IMPLICIT NONE

2014-10-06 Thread Marek Polacek
Hi, On Mon, Oct 06, 2014 at 09:20:50AM +0200, Tobias Burnus wrote: Committing patches when having a cold doesn't seem to work. I don't have Having fight with cold last week, I know what you're talking about ;). access to the computer with SVN write permission, thus, I cannot fix it myself

Re: [PATCH 1/2] PR debug/63239 Add DWARF representation for C++11 deleted member function.

2014-10-06 Thread Andreas Schwab
Mark Wielaard m...@redhat.com writes: diff --git a/gcc/testsuite/g++.dg/debug/dwarf2/deleted-member-function.C b/gcc/testsuite/g++.dg/debug/dwarf2/deleted-member-function.C new file mode 100644 index 000..4cc77e6 --- /dev/null +++

Re: [gofrontend-dev] [PATCH 5/9] Gccgo port to s390[x] -- part I

2014-10-06 Thread Dominik Vogt
On Fri, Oct 03, 2014 at 06:25:55PM -0700, Ian Lance Taylor wrote: On Tue, Sep 9, 2014 at 5:56 AM, Dominik Vogt v...@linux.vnet.ibm.com wrote: * libgo/go/debug/elf/file.go (applyRelocationsAMD64): Fix the calculation of some relocations; do not assume that the symbol

Re: [PATCH 1/2] PR debug/63239 Add DWARF representation for C++11 deleted member function.

2014-10-06 Thread Mark Wielaard
On Mon, 2014-10-06 at 09:38 +0200, Andreas Schwab wrote: Mark Wielaard m...@redhat.com writes: diff --git a/gcc/testsuite/g++.dg/debug/dwarf2/deleted-member-function.C b/gcc/testsuite/g++.dg/debug/dwarf2/deleted-member-function.C new file mode 100644 index 000..4cc77e6 ---

Re: [PATCH 1/2] PR debug/63239 Add DWARF representation for C++11 deleted member function.

2014-10-06 Thread Steven Bosscher
On Mon, Oct 6, 2014 at 9:54 AM, Mark Wielaard wrote: Just removing the # prefix (but keeping the space) from scan-assembler-times should work for both our cases. I just don't know why our .s outputs look different. Wild guess: different comment marker in the target's assembly language? Ciao!

[PATCH] gcc.dg/lto/ cleanup

2014-10-06 Thread Marek Polacek
This patch is a cleanup of tests in gcc.dg/lto/ directory. See https://gcc.gnu.org/ml/gcc-patches/2014-09/msg02656.html for more info. I had to use gnu_inline attribute, because primary LTO files can't have dg-options, and using dg-lto-options would reduce testing. Tested on x86_64-linux:

Libjava test failure Was: [PATCH] microblaze: microblaze.md: Use 'SI' instead of 'VOID' for operand 1 of 'call_value_intern'

2014-10-06 Thread Andrew Haley
On 06/10/14 05:08, Chen Gang wrote: After try normal configure, get almost the same result, I guess, our testsuite under Darwin x86_64 is OK. If no any additional reply within a week, I shall continue to try to analyze the libjava Throw_2 issue. Throw_2 is a test specially contrived to

Re: [PATCH] SPARC: add mcpu=leon3v7 target

2014-10-06 Thread Eric Botcazou
I have sent an updated patch just now. In case -Aleon does not exist it should select -Av7 instead of -Av8. No patch was attached to this message though. Yes, we have. Just verified that with the FSF. It would be a nice for us to have write access to the sparc parts. OK, but the granularity

RFA: fix mode confusion in caller-save.c:replace_reg_with_saved_mem

2014-10-06 Thread Joern Rennecke
Investigating an ICE while trying to compile libgcc2.c:__udivmoddi4 for a new avr variant with different register set/allocation order, I found replace_reg_with_saved_mem falling over its own nonsense. The instruction: (debug_insn 97 96 98 2 (var_location:SI __x2 (mult:SI (lshiftrt:SI (reg/v:SI

[PATCH] gcc.dg/debug/ cleanup

2014-10-06 Thread Marek Polacek
This patch is a cleanup of tests in gcc.dg/debug/ directory. See https://gcc.gnu.org/ml/gcc-patches/2014-09/msg02656.html for more info. Tested on x86_64-linux: vanilla results == results with this patch == results with this patch and gnu11 as a default. Applying to trunk. 2014-10-06 Marek

[C++ Patch] PR 55250

2014-10-06 Thread Paolo Carlini
Hi, the second half of the bug report is about C++14 variable declarations in constexpr functions, an implementation request which should be already done by Jason's recent commit. The first half is about us not rejecting in C++11 mode type declarations in constexpr functions outside, per

Re: Profile mode maintenance patch

2014-10-06 Thread Jonathan Wakely
On 04/10/14 22:54 +0200, François Dumont wrote: On 23/09/2014 13:27, Jonathan Wakely wrote: Yes, OK for trunk - thanks very much. Hi There was in fact one last test failing, ext/profile/mh.cc, a profile mode specific test. It must have been failing for quite a while since malloc hooks

[PATCH] lto.c: add ARG_UNUSED for an argument

2014-10-06 Thread Martin Liška
Hello. In lto.c, if #ifdef HAVE_WORKING_FORK isn't defined, 'last' argument is unused and in that case GCC cannot bootstrap due to a warning. Ready for trunk? Built on x86_64-linux-gnu. Thanks, Martin gcc/lto/ChangeLog: 2014-10-06 Martin Liska mli...@suse.cz * lto.c (stream_out):

[patch] Fix unaligned access in function returning composite type

2014-10-06 Thread Eric Botcazou
Hi, In Ada, we support all sorts of data structure layout, including on strict- alignment targets, and the compiler must generate glue code if necessary. The problem here is a function returning a structure by invisible reference into a field which is not sufficient aligned because the enclosing

Re: [PATCH] SPARC: add mcpu=leon3v7 target

2014-10-06 Thread Daniel Hellstrom
On 10/06/2014 10:40 AM, Eric Botcazou wrote: I have sent an updated patch just now. In case -Aleon does not exist it should select -Av7 instead of -Av8. No patch was attached to this message though. I posted it with v2 in the subject. I have now attached it. Yes, we have. Just verified that

[PATCH] gcc.dg/ipa/ cleanup

2014-10-06 Thread Marek Polacek
This patch is a cleanup of tests in gcc.dg/ipa/ directory. See https://gcc.gnu.org/ml/gcc-patches/2014-09/msg02656.html for more info. Tested on x86_64-linux: vanilla results == results with this patch == results with this patch and gnu11 as a default. Applying to trunk. 2014-10-06 Marek

[patch] Work harder to find DECL_STRUCT_FUNCTION

2014-10-06 Thread Eric Botcazou
Hi, you can have chains of clone functions in the callgraph but can_inline_edge_p stops at the first clone when it is looking for DECL_STRUCT_FUNCTION, which can fool the following conditions in the predicate. Tested on x86_64-suse-linux, OK for the mainline? 2014-10-06 Eric Botcazou

[Java PATCH] Generate declarations in jvgenmain.c

2014-10-06 Thread Marek Polacek
Java testsuite breaks with -std=gnu11 as a default and/or with -Wimplicit-function-declaration on, since the jvgenmain.c program that generates a C file containing 'main' function which calls either 'JvRunMainName' or 'JvRunMain' does not generate forward declarations for these functions. The

Re: [PATCH RFC]Pair load store instructions using a generic scheduling fusion pass

2014-10-06 Thread Bin.Cheng
On Wed, Oct 1, 2014 at 5:06 AM, Mike Stump mikest...@comcast.net wrote: On Sep 30, 2014, at 2:22 AM, Bin Cheng bin.ch...@arm.com wrote: Then I decided to take one step forward to introduce a generic instruction fusion infrastructure in GCC, because in essence, load/store pair is nothing

Fix trace numbering in .dwarf2 dump file

2014-10-06 Thread Eric Botcazou
The numbering of traces in the .dwarf2 dump file is broken, for example: Creating trace 0 : start at note 1 Creating trace 1 : start at note 51 Creating trace 2 : start at code_label 16 Creating trace 3 : start at code_label 23 Creating trace 4 : start at code_label 38 Creating trace 5 : start at

[patch] Turn 1 lra_assert into 1 gcc_assert

2014-10-06 Thread Eric Botcazou
Hi, we have run into a case on the ARM where LRA silently generates wrong code in release mode, while reload (-mno-lra) would stop with the usual spill failure. Tested on x86_64-suse-linux, OK for the mainline? 2014-10-06 Eric Botcazou ebotca...@adacore.com * lra-assigns.c

[patch] Fix internal error in LTO mode on Ada code

2014-10-06 Thread Eric Botcazou
Hi, LTO quickly breaks on Ada code involving floating-point comparisons because the Ada compiler defaults to -fnon-call-exceptions -fno-trapping-math (on most platforms) and only -fnon-call-exceptions is restored on the LTO side, so you can end up with the wrong number of edges on FP

[patch] Fix miscompilation of gnat1 in LTO bootstrap

2014-10-06 Thread Eric Botcazou
Hi, this is a regression from GCC 4.8.x: gnat1 (uintp.adb:UI_Lt) is miscompiled during a LTO boostrap when Init_Operand is inlined into it. A store is wrongly deleted by GIMPLE DSE because of a missed aliasing relationship between 2 array types with TYPE_NONALIASED_COMPONENT set, one being a

[PATCH] libgomp.c/ cleanup

2014-10-06 Thread Marek Polacek
This patch is a cleanup of tests in libgomp.c/ directory. See https://gcc.gnu.org/ml/gcc-patches/2014-09/msg02656.html for more info. Tested on x86_64-linux: vanilla results == results with this patch == results with this patch and gnu11 as a default. Applying to trunk. 2014-10-06 Marek

[patch] Fix ICE during LTRANS on big Ada application

2014-10-06 Thread Eric Botcazou
Hi, we have run into an ICE in LTO mode on a big Ada application with our 4.9- based compiler, whose error message is: xx.constprop.15627/1865799 (xxx.constprop) @0x76835bd0 Type: function definition analyzed Visibility: used_from_other_partition public

Re: [PATCH] libgomp.c/ cleanup

2014-10-06 Thread Jakub Jelinek
On Mon, Oct 06, 2014 at 12:18:33PM +0200, Marek Polacek wrote: --- gcc/libgomp/testsuite/libgomp.c/affinity-1.c +++ gcc/libgomp/testsuite/libgomp.c/affinity-1.c @@ -31,6 +31,7 @@ #ifdef DO_FORK #include signal.h +pid_t waitpid (pid_t, int *, int); I'd #include sys/wait.h here instead.

Re: [gomp4] OpenACC wait directive

2014-10-06 Thread Ilmir Usmanov
Hi Cesar! Thank you for the update! On 05.10.2014 02:32, Cesar Philippidis wrote: On 10/03/2014 07:34 AM, Cesar Philippidis wrote: On 09/24/2014 12:18 AM, Ilmir Usmanov wrote: Hi Cesar! Thank you for the patch! On 24.09.2014 02:29, Cesar Philippidis wrote: This patch adds support for the

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

2014-10-06 Thread Yury Gribov
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 architectures. New patch which checks that -fasan-shadow-offset is only enabled for

[SPARC] Remove an unused predicate

2014-10-06 Thread Eric Botcazou
Tested on SPARC/Solaris, applied on the mainline. 2014-10-06 Eric Botcazou ebotca...@adacore.com * config/sparc/predicates.md (int_register_operand): Delete. -- Eric BotcazouIndex: config/sparc/predicates.md === ---

Re: [PATCH,i386] Fix adxintrin on mingw.

2014-10-06 Thread Ilya Tocar
On 03 Oct 07:53, H.J. Lu wrote: On Fri, Oct 3, 2014 at 6:46 AM, Ilya Tocar tocarip.in...@gmail.com wrote: On 02 Oct 07:41, H.J. Lu wrote: On Thu, Oct 2, 2014 at 7:29 AM, Ilya Tocar tocarip.in...@gmail.com wrote: The same is true for x32. Can you add a testcase to show it fails on x32

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

2014-10-06 Thread Yury Gribov
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 architectures. New patch which checks that

Re: [PATCH] lto.c: add ARG_UNUSED for an argument

2014-10-06 Thread Richard Biener
On Mon, Oct 6, 2014 at 11:36 AM, Martin Liška mli...@suse.cz wrote: Hello. In lto.c, if #ifdef HAVE_WORKING_FORK isn't defined, 'last' argument is unused and in that case GCC cannot bootstrap due to a warning. Ready for trunk? Built on x86_64-linux-gnu. Ok. Thanks, Richard. Thanks,

Re: [patch] Fix unaligned access in function returning composite type

2014-10-06 Thread Richard Biener
On Mon, Oct 6, 2014 at 11:40 AM, Eric Botcazou ebotca...@adacore.com wrote: Hi, In Ada, we support all sorts of data structure layout, including on strict- alignment targets, and the compiler must generate glue code if necessary. The problem here is a function returning a structure by

Re: [patch] Work harder to find DECL_STRUCT_FUNCTION

2014-10-06 Thread Richard Biener
On Mon, Oct 6, 2014 at 11:52 AM, Eric Botcazou ebotca...@adacore.com wrote: Hi, you can have chains of clone functions in the callgraph but can_inline_edge_p stops at the first clone when it is looking for DECL_STRUCT_FUNCTION, which can fool the following conditions in the predicate.

Re: [PATCH RFC]Pair load store instructions using a generic scheduling fusion pass

2014-10-06 Thread Richard Biener
On Mon, Oct 6, 2014 at 11:57 AM, Bin.Cheng amker.ch...@gmail.com wrote: On Wed, Oct 1, 2014 at 5:06 AM, Mike Stump mikest...@comcast.net wrote: On Sep 30, 2014, at 2:22 AM, Bin Cheng bin.ch...@arm.com wrote: Then I decided to take one step forward to introduce a generic instruction fusion

Re: [patch] Turn 1 lra_assert into 1 gcc_assert

2014-10-06 Thread Richard Biener
On Mon, Oct 6, 2014 at 12:02 PM, Eric Botcazou ebotca...@adacore.com wrote: Hi, we have run into a case on the ARM where LRA silently generates wrong code in release mode, while reload (-mno-lra) would stop with the usual spill failure. Tested on x86_64-suse-linux, OK for the mainline? The

Re: [patch] Fix internal error in LTO mode on Ada code

2014-10-06 Thread Richard Biener
On Mon, Oct 6, 2014 at 12:10 PM, Eric Botcazou ebotca...@adacore.com wrote: Hi, LTO quickly breaks on Ada code involving floating-point comparisons because the Ada compiler defaults to -fnon-call-exceptions -fno-trapping-math (on most platforms) and only -fnon-call-exceptions is restored on

Re: [patch] Fix miscompilation of gnat1 in LTO bootstrap

2014-10-06 Thread Richard Biener
On Mon, Oct 6, 2014 at 12:16 PM, Eric Botcazou ebotca...@adacore.com wrote: Hi, this is a regression from GCC 4.8.x: gnat1 (uintp.adb:UI_Lt) is miscompiled during a LTO boostrap when Init_Operand is inlined into it. A store is wrongly deleted by GIMPLE DSE because of a missed aliasing

[PATCH] gcc.dg/ cleanup

2014-10-06 Thread Marek Polacek
This patch is a cleanup of the rest of the tests in gcc.dg/ directory that haven't been fixed in previous patches. See https://gcc.gnu.org/ml/gcc-patches/2014-09/msg02656.html for more info. Tested on x86_64-linux: vanilla results == results with this patch == results with this patch and gnu11 as

Re: [PATCH] libgomp.c/ cleanup

2014-10-06 Thread Marek Polacek
On Mon, Oct 06, 2014 at 12:46:51PM +0200, Jakub Jelinek wrote: On Mon, Oct 06, 2014 at 12:18:33PM +0200, Marek Polacek wrote: --- gcc/libgomp/testsuite/libgomp.c/affinity-1.c +++ gcc/libgomp/testsuite/libgomp.c/affinity-1.c @@ -31,6 +31,7 @@ #ifdef DO_FORK #include signal.h

Re: [PATCH,i386] Fix adxintrin on mingw.

2014-10-06 Thread Uros Bizjak
On Thu, Oct 2, 2014 at 4:29 PM, Ilya Tocar tocarip.in...@gmail.com wrote: Hi, sizeof (long) == 4 on windows, so we should use long long as param type. Patch below does it. Ok for trunk? 2014-10-02 Ilya Tocar ilya.to...@intel.com * config/i386/adxintrin.h (_subborrow_u64): Use

Re: [gomp4] Offload option handling

2014-10-06 Thread Bernd Schmidt
On 10/03/2014 04:31 PM, Andrey Turetskiy wrote: I've applied your option patch on our offload branch (w/o '-ftarget-options' switch yet) and it seems to be working fine. However the patch looks a bit unfinished: @@ -440,7 +554,11 @@ access_check (const char *name, int mode static char*

[PATCH] libitm.c/ cleanup

2014-10-06 Thread Marek Polacek
This patch is a cleanup of tests in libitm.c/ directory. See https://gcc.gnu.org/ml/gcc-patches/2014-09/msg02656.html for more info. Only two tests needed fixing this time around. Tested on x86_64-linux: vanilla results == results with this patch == results with this patch and gnu11 as a

[patch] Add std::experimental::apply()

2014-10-06 Thread Jonathan Wakely
From the Fundamentals TS. Tested x86_64-linux, committed to trunk. commit df1e59560b6723da563f3aae8d95dcfe2781173b Author: Jonathan Wakely jwak...@redhat.com Date: Fri Oct 3 17:51:31 2014 +0100 * include/Makefile.am: Add new header. * include/Makefile.in: Regenerate. *

Re: [RFC PATCH] Enable V32HI/V64QI const permutations

2014-10-06 Thread Kirill Yukhin
Hello Jakub, On 03 Oct 16:39, Jakub Jelinek wrote: --- gcc/config/i386/sse.md.jj 2014-09-26 10:33:18.0 +0200 +++ gcc/config/i386/sse.md2014-10-03 15:03:44.170446452 +0200 @@ -10386,7 +10386,8 @@ (define_mode_iterator VEC_PERM_CONST (V8SI TARGET_AVX) (V4DI TARGET_AVX)

[PATCH i386 AVX512] [63/n] Add vpshufb, perm autogen.

2014-10-06 Thread Kirill Yukhin
Hello, This patch extends permutations for AVX-512*. Comments are welcome! Bootstrapped. AVX-512* tests on top of patch-set all pass under simulator. Is it ok for trunk? gcc/ * config/i386/i386.c (ix86_expand_vec_perm_vpermi2): Handle V64QImode, V8HImode, V16HImode,

Re: RFA: Merge definitions of get_some_local_dynamic_name

2014-10-06 Thread Rainer Orth
Hi Richard, Rainer Orth r...@cebitec.uni-bielefeld.de writes: Hi Richard, It seems the new get_some_local_dynamic_name implementation in function.c lost the non-NULL check the sparc.c version had. I'm currently testing the following patch: Could you do a: call debug_rtx (...) on the

[PATCH] gcc.target/i386/ cleanup

2014-10-06 Thread Marek Polacek
This patch is a cleanup of tests in gcc.target/i386/ directory. See https://gcc.gnu.org/ml/gcc-patches/2014-09/msg02656.html for more info. Tested on x86_64-linux: vanilla results == results with this patch == results with this patch and gnu11 as a default. Applying to trunk. 2014-10-06 Marek

[PATCH] misc cleanup

2014-10-06 Thread Marek Polacek
This patch fixes some miscellaneous stuff in the testsuite. See https://gcc.gnu.org/ml/gcc-patches/2014-09/msg02656.html for more info. Tested on x86_64-linux: vanilla results == results with this patch == results with this patch and gnu11 as a default. Applying to trunk. 2014-10-06 Marek

Re: Libjava test failure Was: [PATCH] microblaze: microblaze.md: Use 'SI' instead of 'VOID' for operand 1 of 'call_value_intern'

2014-10-06 Thread Chen Gang
On 10/6/14 16:37, Andrew Haley wrote: On 06/10/14 05:08, Chen Gang wrote: After try normal configure, get almost the same result, I guess, our testsuite under Darwin x86_64 is OK. If no any additional reply within a week, I shall continue to try to analyze the libjava Throw_2 issue.

Re: [C++ Patch] PR 55250

2014-10-06 Thread Jason Merrill
On 10/06/2014 04:56 AM, Paolo Carlini wrote: +! is_typedef_decl (var) +! TYPE_DECL_ALIAS_P (var)) Maybe DECL_IMPLICIT_TYPEDEF_P (var) instead of the above? OK either way. Jason

Re: Libjava test failure Was: [PATCH] microblaze: microblaze.md: Use 'SI' instead of 'VOID' for operand 1 of 'call_value_intern'

2014-10-06 Thread Andrew Haley
On 10/06/2014 02:53 PM, Chen Gang wrote: On 10/6/14 16:37, Andrew Haley wrote: On 06/10/14 05:08, Chen Gang wrote: After try normal configure, get almost the same result, I guess, our testsuite under Darwin x86_64 is OK. If no any additional reply within a week, I shall continue to try to

Re: [RFC PATCH] Enable V32HI/V64QI const permutations

2014-10-06 Thread Ilya Tocar
On 06 Oct 09:08, Jakub Jelinek wrote: On Fri, Oct 03, 2014 at 04:39:08PM +0200, Jakub Jelinek wrote: Just to stress the new testcases some more, I've enabled the vec_perm_const{32hi,64qi} patterns. Got several ICEs in expand_vec_perm_broadcast_1, on the final gcc_unreachable () in the

Re: [PATCH i386 AVX512] [63/n] Add vpshufb, perm autogen.

2014-10-06 Thread Jakub Jelinek
On Mon, Oct 06, 2014 at 04:55:28PM +0400, Kirill Yukhin wrote: --- a/gcc/config/i386/i386.c +++ b/gcc/config/i386/i386.c @@ -21364,20 +21364,113 @@ ix86_expand_vec_perm_vpermi2 (rtx target, rtx op0, rtx mask, rtx op1) enum machine_mode mode = GET_MODE (op0); switch (mode) { +

Re: Libjava test failure Was: [PATCH] microblaze: microblaze.md: Use 'SI' instead of 'VOID' for operand 1 of 'call_value_intern'

2014-10-06 Thread Chen Gang
On 10/6/14 21:54, Andrew Haley wrote: On 10/06/2014 02:53 PM, Chen Gang wrote: On 10/6/14 16:37, Andrew Haley wrote: On 06/10/14 05:08, Chen Gang wrote: After try normal configure, get almost the same result, I guess, our testsuite under Darwin x86_64 is OK. If no any additional reply

Re: [gofrontend-dev] [PATCH 5/9] Gccgo port to s390[x] -- part I

2014-10-06 Thread Ian Lance Taylor
On Mon, Oct 6, 2014 at 12:42 AM, Dominik Vogt v...@linux.vnet.ibm.com wrote: On s390[x] the symbol value of a section symbol is definitely not zero. Is true even in an object file? I agree that in an executable a section symbol will have a non-zero value, but that case doesn't arise since an

Re: Libjava test failure Was: [PATCH] microblaze: microblaze.md: Use 'SI' instead of 'VOID' for operand 1 of 'call_value_intern'

2014-10-06 Thread Andrew Haley
On 10/06/2014 03:27 PM, Chen Gang wrote: On 10/6/14 21:54, Andrew Haley wrote: On 10/06/2014 02:53 PM, Chen Gang wrote: On 10/6/14 16:37, Andrew Haley wrote: On 06/10/14 05:08, Chen Gang wrote: After try normal configure, get almost the same result, I guess, our testsuite under Darwin x86_64

Re: [PATCH, rs6000] Document issues with permutes for analyze_swaps

2014-10-06 Thread David Edelsohn
On Sun, Oct 5, 2014 at 6:06 PM, Bill Schmidt wschm...@linux.vnet.ibm.com wrote: Hi, I spent some time thinking about handling vperm instructions in the analyze_swaps pass, and convinced myself that it isn't necessarily wise to do so. At the least it will require adding a cost model to the

Re: Libjava test failure Was: [PATCH] microblaze: microblaze.md: Use 'SI' instead of 'VOID' for operand 1 of 'call_value_intern'

2014-10-06 Thread Chen Gang
On 10/6/14 22:28, Andrew Haley wrote: On 10/06/2014 03:27 PM, Chen Gang wrote: On 10/6/14 21:54, Andrew Haley wrote: On 10/06/2014 02:53 PM, Chen Gang wrote: On 10/6/14 16:37, Andrew Haley wrote: On 06/10/14 05:08, Chen Gang wrote: After try normal configure, get almost the same result, I

Re: Libjava test failure Was: [PATCH] microblaze: microblaze.md: Use 'SI' instead of 'VOID' for operand 1 of 'call_value_intern'

2014-10-06 Thread Andrew Haley
On 10/06/2014 04:00 PM, Chen Gang wrote: On 10/6/14 22:28, Andrew Haley wrote: On 10/06/2014 03:27 PM, Chen Gang wrote: On 10/6/14 21:54, Andrew Haley wrote: On 10/06/2014 02:53 PM, Chen Gang wrote: On 10/6/14 16:37, Andrew Haley wrote: On 06/10/14 05:08, Chen Gang wrote: After try normal

Re: Libjava test failure Was: [PATCH] microblaze: microblaze.md: Use 'SI' instead of 'VOID' for operand 1 of 'call_value_intern'

2014-10-06 Thread Chen Gang
On 10/6/14 22:59, Andrew Haley wrote: On 10/06/2014 04:00 PM, Chen Gang wrote: On 10/6/14 22:28, Andrew Haley wrote: [...] The testsuite isn't breaking: it's telling you something useful. Sorry it is breaked, at present, I temporarily skipped it and then can let make check finish, the

Re: [RFC PATCH] Enable V32HI/V64QI const permutations

2014-10-06 Thread Jakub Jelinek
On Mon, Oct 06, 2014 at 06:09:07PM +0400, Ilya Tocar wrote: Speaking of -mavx512{bw,vl,f}, there apparently is a full 2 operand shuffle for V32HI, V16S[IF], V8D[IF], so the only one instruction full 2 operand shuffle we are missing is V64QI, right? What would be best worst case sequence

RE: [RFC: Patch, PR 60102] [4.9/4.10 Regression] powerpc fp-bit ices@dwf_regno

2014-10-06 Thread rohitarul...@freescale.com
From: Dharmakan Rohit-B30502 Sent: Monday, September 29, 2014 3:54 PM From: Ulrich Weigand [mailto:uweig...@de.ibm.com] Maciej W. Rozycki wrote: On Mon, 4 Aug 2014, Edmar wrote: Committed on trunk, revision 213596 Committed on 4.9 branch, revision 213597 This change

[PATCH 5/n] OpenMP 4.0 offloading infrastructure: libgomp

2014-10-06 Thread Ilya Verbin
Hello, This patch adds plugin support to libgomp, as well as memory mapping and interaction with target devices through plugin's interface. Bootstrapped and regtested on top of patch 4. Is it OK for trunk? Thanks, -- Ilya 2014-10-06 Jakub Jelinek ja...@redhat.com Ilya Verbin

Re: [PATCH v2] libstdc++: Add hexfloat/defaultfloat io manipulators.

2014-10-06 Thread Jonathan Wakely
On 28/03/14 08:56 +0900, Luke Allardyce wrote: It looks like the new standard also requires the precision to be ignored for hexfloat For conversion from a floating-point type, if floatfield != (ios_base::fixed | ios_base:: scientific), str.precision() is specified as precision in the

[PATCH] Indirect-call topn targets profiler (instrumentation)

2014-10-06 Thread Rong Xu
Hi, I ported the indirect-call topn target profile from google branch. It implements the algorithm described in Value profiling and optimization by Brad Calder and Peter Feller. This patch is about the instrumentation. When --param=indir-call-topn-profile=1 is specified, we will use

[PATCH] Indirect-call topn targets profiler (runtime)

2014-10-06 Thread Rong Xu
Hi, I ported the indirect-call topn target profile from google branch. It implements the algorithm described in Value profiling and optimization by Brad Calder and Peter Feller. This patch is the runtime support. Tested with gcc regression tests and the instrumentation patch (in a separated

Re: [PATCH] SPARC: add mcpu=leon3v7 target

2014-10-06 Thread Eric Botcazou
I posted it with v2 in the subject. I have now attached it. There is a pasto in the v2: @@ -1764,6 +1772,12 @@ extern int sparc_indent_opcode; #define AS_LEON_FLAG -Av8 #endif +#ifdef HAVE_AS_LEONV7 +#define AS_LEONV7_FLAG -Aleon +#else +#define AS_LEONV7_FLAG -Av7 +#endif + /* We use gcc

Re: [PATCH RFC]Pair load store instructions using a generic scheduling fusion pass

2014-10-06 Thread Mike Stump
On Oct 6, 2014, at 4:32 AM, Richard Biener richard.guent...@gmail.com wrote: On Mon, Oct 6, 2014 at 11:57 AM, Bin.Cheng amker.ch...@gmail.com wrote: How many merging opportunities does sched2 undo again? ISTR it has the tendency of pushing stores down and loads up. So, the pass works by

Re: Libjava test failure Was: [PATCH] microblaze: microblaze.md: Use 'SI' instead of 'VOID' for operand 1 of 'call_value_intern'

2014-10-06 Thread Mike Stump
On Oct 6, 2014, at 8:36 AM, Chen Gang gang.chen.5...@gmail.com wrote: For me, make -k check is suitable for one sub-system (e.g. for cross building, and mainly focus on gcc), but not for global check (full non-cross building check): In our world, there is no sub-system, so, talk of such is

Re: [PATCH 0/14+2][Vectorizer] Made reductions endianness-neutral, fixes PR/61114

2014-10-06 Thread Alan Lawrence
Ok, so unless there are objections, I plan to commit patches 1, 2, 4, 5, and 6, which have been previously approved, in that sequence. (Of those, all bar patch 2 are AArch64 only.) I think this is better than maintaining an ever-expanding patch series. Then I'll get to work on migrating all

[jit] Fix make check-jit

2014-10-06 Thread David Malcolm
Committed to branch dmalcolm/jit: gcc/jit/ChangeLog.jit: * Make-lang.in (lang_checks): Append check-jit. * docs/internals/index.rst (Running the test suite): Use check-jit rather than check-parallel-jit. --- gcc/jit/Make-lang.in | 2 ++

Re: RFA: Merge definitions of get_some_local_dynamic_name

2014-10-06 Thread Richard Sandiford
Rainer Orth r...@cebitec.uni-bielefeld.de writes: Hi Richard, Rainer Orth r...@cebitec.uni-bielefeld.de writes: Hi Richard, It seems the new get_some_local_dynamic_name implementation in function.c lost the non-NULL check the sparc.c version had. I'm currently testing the following patch:

Re: [PATCH] Indirect-call topn targets profiler (instrumentation)

2014-10-06 Thread Jan Hubicka
Rong, Would be possible to use topn profiler to get resonale histograms for switch expansion, too? In that case it may make sense to have value version of it as well. Otherwise the patch seems OK. I would implement it myself by introducing separate variables holding the topn profiler

Re: [PATCH] Indirect-call topn targets profiler (instrumentation)

2014-10-06 Thread Xinliang David Li
On Mon, Oct 6, 2014 at 11:22 AM, Jan Hubicka hubi...@ucw.cz wrote: Rong, Would be possible to use topn profiler to get resonale histograms for switch expansion, too? In that case it may make sense to have value version of it as well. The underlying topn_profiler can be used the same way as

Re: RFA: fix mode confusion in caller-save.c:replace_reg_with_saved_mem

2014-10-06 Thread Jeff Law
On 10/06/14 02:51, Joern Rennecke wrote: Investigating an ICE while trying to compile libgcc2.c:__udivmoddi4 for a new avr variant with different register set/allocation order, I found replace_reg_with_saved_mem falling over its own nonsense. The instruction: (debug_insn 97 96 98 2

Re: [PATCH] Indirect-call topn targets profiler (instrumentation)

2014-10-06 Thread Jan Hubicka
On Mon, Oct 6, 2014 at 11:22 AM, Jan Hubicka hubi...@ucw.cz wrote: Rong, Would be possible to use topn profiler to get resonale histograms for switch expansion, too? In that case it may make sense to have value version of it as well. The underlying topn_profiler can be used the same

[jit] Merger of trunk r215958 into jit branch

2014-10-06 Thread David Malcolm
I've merged today's svn trunk into the git branch dmalcolm/jit, pushing it as 81d39bf76e3ffd4e62c307cc8d22df18572eec19 Specifically, this is from trunk r215958 aka e012cdc775868e9922f5fef9068a764546876d93 bringing in all changes since 215803 aka 68874c5d021dd6b8b740ffd23960e46e333270b5

Re: [PATCH] Indirect-call topn targets profiler (instrumentation)

2014-10-06 Thread Rong Xu
On Mon, Oct 6, 2014 at 12:21 PM, Jan Hubicka hubi...@ucw.cz wrote: On Mon, Oct 6, 2014 at 11:22 AM, Jan Hubicka hubi...@ucw.cz wrote: Rong, Would be possible to use topn profiler to get resonale histograms for switch expansion, too? In that case it may make sense to have value version

Re: [PATCH] Indirect-call topn targets profiler (instrumentation)

2014-10-06 Thread Jan Hubicka
Yes. That's doable. I'm just not sure if top 2 entries are enough. I can see from your example that it will be useful for BBs having multiple case stmt. I suppose we want real histogram profiler here. I.e. something that is given value ranges and fill in the counts depending on value

Re: [PATCH] Indirect-call topn targets profiler (instrumentation)

2014-10-06 Thread Xinliang David Li
On Mon, Oct 6, 2014 at 1:31 PM, Jan Hubicka hubi...@ucw.cz wrote: Yes. That's doable. I'm just not sure if top 2 entries are enough. I can see from your example that it will be useful for BBs having multiple case stmt. I suppose we want real histogram profiler here. I.e. something that is

Re: [patch] Add -static-libquadmath option

2014-10-06 Thread Joseph S. Myers
Since -static-libquadmath introduces LGPL requirements on redistributing the resulting binaries (that you provide source or relinkable object files to allow relinking with modified versions of libquadmath) that don't otherwise generally apply simply through using GCC to build a program even if

Re: [C PATCH] Use error_operand_p more

2014-10-06 Thread Joseph S. Myers
On Sun, 5 Oct 2014, Marek Polacek wrote: It occured to me that we should probably use error_operand_p in the C FE where appropriate. Following change is meant only as a little cleanup. Bootstrapped/regtested on x86_64-linux, ok for trunk? 2014-10-04 Marek Polacek pola...@redhat.com

Re: [PATCH x86_64] Optimize access to globals in -fpie -pie builds with copy relocations

2014-10-06 Thread Sriraman Tallam
Ping. On Mon, Sep 29, 2014 at 10:57 AM, Sriraman Tallam tmsri...@google.com wrote: Ping. On Fri, Sep 19, 2014 at 2:11 PM, Sriraman Tallam tmsri...@google.com wrote: Hi Richard, I also ran the gcc testsuite with RUNTESTFLAGS=--tool_opts=-mcopyrelocs to check for issues. The only test that

Re: [patch] Add -static-libquadmath option

2014-10-06 Thread Steve Kargl
On Mon, Oct 06, 2014 at 08:38:14PM +, Joseph S. Myers wrote: Since -static-libquadmath introduces LGPL requirements on redistributing the resulting binaries (that you provide source or relinkable object files to allow relinking with modified versions of libquadmath) that don't otherwise

Re: [Java PATCH] Generate declarations in jvgenmain.c

2014-10-06 Thread Mark Wielaard
On Mon, Oct 06, 2014 at 11:54:00AM +0200, Marek Polacek wrote: Java testsuite breaks with -std=gnu11 as a default and/or with -Wimplicit-function-declaration on, since the jvgenmain.c program that generates a C file containing 'main' function which calls either 'JvRunMainName' or 'JvRunMain'

Re: sort_heap complexity guarantee

2014-10-06 Thread François Dumont
On 05/10/2014 22:54, Marc Glisse wrote: On Sun, 5 Oct 2014, François Dumont wrote: I took a look at PR 61217 regarding pop_heap complexity guarantee. Looks like we have no test to check complexity of our algos so I start writing some starting with the heap operations. I found no issue

Re: sort_heap complexity guarantee

2014-10-06 Thread Daniel Krügler
2014-10-06 23:00 GMT+02:00 François Dumont frs.dum...@gmail.com: On 05/10/2014 22:54, Marc Glisse wrote: On Sun, 5 Oct 2014, François Dumont wrote: I took a look at PR 61217 regarding pop_heap complexity guarantee. Looks like we have no test to check complexity of our algos so I start

Re: [patch] Add -static-libquadmath option

2014-10-06 Thread Joseph S. Myers
On Mon, 6 Oct 2014, Steve Kargl wrote: On Mon, Oct 06, 2014 at 08:38:14PM +, Joseph S. Myers wrote: Since -static-libquadmath introduces LGPL requirements on redistributing the resulting binaries (that you provide source or relinkable object files to allow relinking with modified

RE: [Patch, MIPS] Cleanup mips header files.

2014-10-06 Thread Matthew Fortune
Hi Steve, You're the lucky recipient of my first review so apologies for being slow and cautious... I tried to find a reason why the files were originally separated like this and I can't see anything obvious. I assume you also found no reason. Presumably the separation was just to avoid

[google/gcc-4_9] Add gcc driver option -no-pie

2014-10-06 Thread Sriraman Tallam
Hi, A build tool we are using adds -pie by default and I am adding -no-pie to override this when necessary. Is this patch alright? Would this be suitable for trunk? Thanks Sri Add a negative for option -pie: -no-pie builds a position dependent executable. Note that -no-pie also negates a

Re: [patch] Flatten function.h take 2

2014-10-06 Thread Andrew MacLeod
Ping... anyone want to take a look? Andrew On 09/29/2014 01:23 PM, Andrew MacLeod wrote: On 09/16/2014 05:23 PM, Andrew MacLeod wrote: On 09/16/2014 05:12 PM, Joseph S. Myers wrote: On Tue, 16 Sep 2014, Andrew MacLeod wrote: I did an include file reduction on all the language/*.[ch] and

Re: [google/gcc-4_9] Add gcc driver option -no-pie

2014-10-06 Thread Joseph S. Myers
If adding a new option, you need to document it in invoke.texi. -- Joseph S. Myers jos...@codesourcery.com

Re: Libjava test failure Was: [PATCH] microblaze: microblaze.md: Use 'SI' instead of 'VOID' for operand 1 of 'call_value_intern'

2014-10-06 Thread Chen Gang
On 10/7/14 1:29, Mike Stump wrote: On Oct 6, 2014, at 8:36 AM, Chen Gang gang.chen.5...@gmail.com wrote: For me, make -k check is suitable for one sub-system (e.g. for cross building, and mainly focus on gcc), but not for global check (full non-cross building check): In our world, there is

Re: [google/gcc-4_9] Add gcc driver option -no-pie

2014-10-06 Thread Sriraman Tallam
On Mon, Oct 6, 2014 at 3:22 PM, Joseph S. Myers jos...@codesourcery.com wrote: If adding a new option, you need to document it in invoke.texi. Patch updated. Thanks Sri -- Joseph S. Myers jos...@codesourcery.com Add a negative for option -pie: -no-pie builds a position dependent

  1   2   >