Re: [PATCH][testsuite] Mark as UNSUPPORTED tests that don't fit into tiny memory model

2014-12-18 Thread Rainer Orth
Andreas Schwab sch...@linux-m68k.org writes: domi...@lps.ens.fr (Dominique Dhumieres) writes: Here it is. Committed as r218662 to get things going again. The same change should be done for libitm and libatomic. And boehm-gc. Right, fixed as follows and installed on mainline after testing

Re: [PATCH][testsuite] Mark as UNSUPPORTED tests that don't fit into tiny memory model

2014-12-18 Thread Kyrill Tkachov
On 18/12/14 09:43, Rainer Orth wrote: Andreas Schwab sch...@linux-m68k.org writes: domi...@lps.ens.fr (Dominique Dhumieres) writes: Here it is. Committed as r218662 to get things going again. The same change should be done for libitm and libatomic. And boehm-gc. Right, fixed as follows

Re: [PATCH][testsuite] Mark as UNSUPPORTED tests that don't fit into tiny memory model

2014-12-18 Thread Rainer Orth
Hi Kyrill, Sorry about this. I had grepped for includes of gcc-defs.exp in lib*, which missed boehm-gc. Thanks for fixing this, I was away earlier this week. I'll make sure to check the libraries explicitly if I do any testsuite refactoring work in the future great, thanks. I know all too

[testsuite, i386] Change mpx effective-target test into link test

2014-12-18 Thread Rainer Orth
Currently, the new gcc.dg/lto/chkp-privatize test FAILs on Solaris/x86 with gas and ld for 64-bit: UNRESOLVED: gcc.dg/lto/chkp-privatize c_lto_chkp-privatize_0.o-c_lto_chkp-privatize_1.o execute -fPIC -flto -flto-partition=max -fcheck-pointer-bounds -mmpx FAIL: gcc.dg/lto/chkp-privatize

[patch 1/2][ARM]: New CPU support for Marvell Whitney

2014-12-18 Thread Xingxing Pan
Hi, This patch contains Marvell Whitney core's pipeline description. Test on arm-linux-gnueabi and no new regression are found. Is it OK for trunk? Regards, Xingxing 2014-12-18 Xingxing Pan xxing...@marvell.com * config/arm/arm-cores.def: Add new core marvell-whitney.

Re: [PATCH 4/4] OpenMP 4.0 offloading to Intel MIC: non-fallback testing

2014-12-18 Thread Jakub Jelinek
On Wed, Dec 17, 2014 at 11:48:01PM +0100, Thomas Schwinge wrote: I have another suggestion: on gomp-4_0-branch, we had already started using a libgomp-test-support.exp file for a similar purpose. I now changed your code on gomp-4_0-branch in r218845 as follows (though, not very much tested).

Re: OpenACC middle end changes

2014-12-18 Thread Thomas Schwinge
Hi! On Thu, 13 Nov 2014 19:09:49 +0100, Jakub Jelinek ja...@redhat.com wrote: On Thu, Nov 13, 2014 at 05:59:11PM +0100, Thomas Schwinge wrote: --- gcc/builtins.c +++ gcc/builtins.c +/* Expand OpenACC acc_on_device. + + This has to happen late (that is, not in early folding;

Re: OpenACC middle end changes

2014-12-18 Thread Thomas Schwinge
Hi Jakub! Many thanks for the review comments! The very most have been addresed, here are just a few comments. If you feel strongly/differently about any, I'll address those, too. On Thu, 13 Nov 2014 19:09:49 +0100, Jakub Jelinek ja...@redhat.com wrote: On Thu, Nov 13, 2014 at 05:59:11PM

Re: [C++ Patch] PR 60955

2014-12-18 Thread Paolo Carlini
Hi, On 12/17/2014 09:37 PM, Jason Merrill wrote: I'm uncomfortable with setting TREE_NO_WARNING on a decl just because we don't want a warning for one particular use of it. How about suppressing warnings across the call to build_static_cast? Sure. The below uses the

Re: [testsuite, i386] Change mpx effective-target test into link test

2014-12-18 Thread Uros Bizjak
On Thu, Dec 18, 2014 at 11:12 AM, Rainer Orth r...@cebitec.uni-bielefeld.de wrote: Currently, the new gcc.dg/lto/chkp-privatize test FAILs on Solaris/x86 with gas and ld for 64-bit: UNRESOLVED: gcc.dg/lto/chkp-privatize c_lto_chkp-privatize_0.o-c_lto_chkp-privatize_1.o execute -fPIC -flto

Re: OpenACC middle end changes

2014-12-18 Thread Jakub Jelinek
On Thu, Dec 18, 2014 at 11:46:00AM +0100, Thomas Schwinge wrote: just rtx v1 = GEN_INT (...); rtx v2 = GEN_INT (...); machine_mode mode = TYPE_MODE (TREE_TYPE (arg)); rtx ret = gen_reg_rtx (TYPE_MODE (integer_type_node)); emit_move_insn (ret, const0_rtx); rtx_code_label

Re: OpenACC middle end changes

2014-12-18 Thread Jakub Jelinek
On Thu, Dec 18, 2014 at 12:07:01PM +0100, Thomas Schwinge wrote: - case GF_OMP_FOR_KIND_SIMD: - kind = simd; - break; - case GF_OMP_FOR_KIND_CILKSIMD: - kind = cilksimd; - break; case GF_OMP_FOR_KIND_DISTRIBUTE: kind = distribute; break; case

Re: [PATCH][AArch64] Implement vsqrt_f64 intrinsic

2014-12-18 Thread Kyrill Tkachov
On 17/12/14 00:04, Joseph Myers wrote: On Mon, 15 Dec 2014, James Greenhalgh wrote: @@ -22792,6 +22792,12 @@ vsqrtq_f32 (float32x4_t a) return __builtin_aarch64_sqrtv4sf (a); } +__extension__ static __inline float64x1_t __attribute__ ((__always_inline__)) +vsqrt_f64 (float64x1_t a)

Re: OpenACC middle end changes

2014-12-18 Thread Jakub Jelinek
On Thu, Dec 18, 2014 at 12:38:53PM +0100, Jakub Jelinek wrote: So, is what is on the gomp-4_0-branch now all that you'd like to merge to trunk now? Has it been tested on nvptx? I guess we should test it with XeonPhi offloading too to make sure it doesn't break. And then you or together with

[PATCH] Fix -fsanitize=float-cast-overflow with C FE (PR sanitizer/64344)

2014-12-18 Thread Jakub Jelinek
On Tue, Dec 16, 2014 at 06:58:47PM +, Joseph Myers wrote: On Fri, 12 Dec 2014, Jakub Jelinek wrote: Hi! -fsanitize=float-cast-overflow sanitization is done in convert.c and calls there save_expr. Unfortunately, save_expr is a no-go for the C FE, we need c_save_expr, but as

Re: OpenACC middle end changes

2014-12-18 Thread Jakub Jelinek
On Thu, Dec 18, 2014 at 01:02:22PM +0100, Jakub Jelinek wrote: On Thu, Dec 18, 2014 at 12:38:53PM +0100, Jakub Jelinek wrote: So, is what is on the gomp-4_0-branch now all that you'd like to merge to trunk now? Has it been tested on nvptx? I guess we should test it with XeonPhi offloading

Re: OpenACC middle end changes

2014-12-18 Thread Thomas Schwinge
Hi Jakub! On Thu, 18 Dec 2014 13:15:38 +0100, Jakub Jelinek ja...@redhat.com wrote: On Thu, Dec 18, 2014 at 01:02:22PM +0100, Jakub Jelinek wrote: On Thu, Dec 18, 2014 at 12:38:53PM +0100, Jakub Jelinek wrote: So, is what is on the gomp-4_0-branch now all that you'd like to merge to

Re: OpenACC middle end changes

2014-12-18 Thread Jakub Jelinek
On Thu, Dec 18, 2014 at 01:24:20PM +0100, Thomas Schwinge wrote: Hi Jakub! On Thu, 18 Dec 2014 13:15:38 +0100, Jakub Jelinek ja...@redhat.com wrote: On Thu, Dec 18, 2014 at 01:02:22PM +0100, Jakub Jelinek wrote: On Thu, Dec 18, 2014 at 12:38:53PM +0100, Jakub Jelinek wrote: So, is

Re: OpenACC middle end changes

2014-12-18 Thread Jakub Jelinek
On Thu, Dec 18, 2014 at 01:31:45PM +0100, Jakub Jelinek wrote: --- gcc/config.gcc +++ gcc/config.gcc @@ -2906,6 +2906,7 @@ esac case ${target} in *-intelmic-* | *-intelmicemul-*) tmake_file=${tmake_file} i386/t-intelmic + tm_file=${tm_file} i386/intelmic-offload.h ;;

[PATCH] X86-64: Add -mskip-rax-setup

2014-12-18 Thread H.J. Lu
The Linux kernel never passes floating point arguments around, vararg functions or not. Hence no vector registers are ever used when calling a vararg function. But gcc still dutifully emits an xor %eax,%eax before each and every call of a vararg function. Since no callee use that for anything,

Re: OpenACC middle end changes

2014-12-18 Thread Thomas Schwinge
Hi Jakub! On Thu, 18 Dec 2014 12:33:11 +0100, Jakub Jelinek ja...@redhat.com wrote: On Thu, Dec 18, 2014 at 11:46:00AM +0100, Thomas Schwinge wrote: just rtx v1 = GEN_INT (...); rtx v2 = GEN_INT (...); machine_mode mode = TYPE_MODE (TREE_TYPE (arg)); rtx ret = gen_reg_rtx

Re: OpenACC middle end changes

2014-12-18 Thread Thomas Schwinge
Hi Jakub! On Thu, 18 Dec 2014 13:36:16 +0100, Jakub Jelinek ja...@redhat.com wrote: On Thu, Dec 18, 2014 at 01:31:45PM +0100, Jakub Jelinek wrote: --- gcc/config.gcc +++ gcc/config.gcc @@ -2906,6 +2906,7 @@ esac case ${target} in *-intelmic-* | *-intelmicemul-*)

Re: [PATCH] X86-64: Add -mskip-rax-setup

2014-12-18 Thread Uros Bizjak
On Thu, Dec 18, 2014 at 2:11 PM, H.J. Lu hongjiu...@intel.com wrote: The Linux kernel never passes floating point arguments around, vararg functions or not. Hence no vector registers are ever used when calling a vararg function. But gcc still dutifully emits an xor %eax,%eax before each and

Re: [PATCH][AArch64] Implement vsqrt_f64 intrinsic

2014-12-18 Thread Joseph Myers
On Thu, 18 Dec 2014, Kyrill Tkachov wrote: I see that there are some intrinsics implemented in terms of __builtin_fabsf, presumable they can be at 'risk' too of having a library call emitted? The semantics of fabsf/fabs/fabsl are to clear the sign bit, never raise any exceptions (even for

[PATCH] Fix for PR64330

2014-12-18 Thread Martin Liška
Hello. Following patch is fix for PR64330, is pre-approved by Honza and I'm going to install it. No regression was seen on x86_64-linux-pc. Thanks, Martin From 6f9ed17a2897be6a26e2c64f8e581771c216e41b Mon Sep 17 00:00:00 2001 From: mliska mli...@suse.cz Date: Wed, 17 Dec 2014 19:11:48 +0100

Re: [PATCH] IPA ICF: refactoring + fix for PR ipa/63569

2014-12-18 Thread Martin Liška
On 12/17/2014 04:23 PM, Richard Biener wrote: On Wed, Dec 17, 2014 at 12:17 PM, Martin Liška mli...@suse.cz wrote: On 12/11/2014 01:37 PM, Richard Biener wrote: On Wed, Dec 10, 2014 at 1:18 PM, Martin Liška mli...@suse.cz wrote: Hello. As suggested by Richard, I split compare_operand

Re: [PATCH 2/3] Extended if-conversion

2014-12-18 Thread Yuri Rumyantsev
of BB just after label if FLAG_FORCE_VECTORIZE is true. (tree_if_conversion): Add initialization of FLAG_FORCE_VECTORIZE which is copy of inner or outer loop field force_vectorize. patch.20141218 Description: Binary data

Pragma parsing (was: [PATCH] OpenACC for C++ front end)

2014-12-18 Thread Thomas Schwinge
Hi! On Thu, 13 Nov 2014 14:02:37 +0100, Jakub Jelinek ja...@redhat.com wrote: void init_pragma (void) { + if (flag_openacc) +{ + const int n_oacc_pragmas + = sizeof (oacc_pragmas) / sizeof (*oacc_pragmas); + int i; + + for (i = 0; i n_oacc_pragmas;

Re: [PATCH] X86-64: Add -mskip-rax-setup

2014-12-18 Thread Uros Bizjak
On Thu, Dec 18, 2014 at 2:24 PM, Uros Bizjak ubiz...@gmail.com wrote: On Thu, Dec 18, 2014 at 2:11 PM, H.J. Lu hongjiu...@intel.com wrote: The Linux kernel never passes floating point arguments around, vararg functions or not. Hence no vector registers are ever used when calling a vararg

Re: [PATCH] X86-64: Add -mskip-rax-setup

2014-12-18 Thread Jakub Jelinek
On Thu, Dec 18, 2014 at 02:24:06PM +0100, Uros Bizjak wrote: It removed 14309 redundant xor %eax,%eax instructions and saved about 27KB. I am currently running the new kernel without any problem. OK for trunk? How about skipping RAX setup unconditionally for !TARGET_SSE? Please see

Re: [PATCH] X86-64: Add -mskip-rax-setup

2014-12-18 Thread H.J. Lu
On Thu, Dec 18, 2014 at 5:51 AM, Jakub Jelinek ja...@redhat.com wrote: On Thu, Dec 18, 2014 at 02:24:06PM +0100, Uros Bizjak wrote: It removed 14309 redundant xor %eax,%eax instructions and saved about 27KB. I am currently running the new kernel without any problem. OK for trunk? How

Re: [PATCH] X86-64: Add -mskip-rax-setup

2014-12-18 Thread Uros Bizjak
On Thu, Dec 18, 2014 at 2:49 PM, Uros Bizjak ubiz...@gmail.com wrote: The Linux kernel never passes floating point arguments around, vararg functions or not. Hence no vector registers are ever used when calling a vararg function. But gcc still dutifully emits an xor %eax,%eax before each and

Re: [PATCH] X86-64: Add -mskip-rax-setup

2014-12-18 Thread H.J. Lu
On Thu, Dec 18, 2014 at 6:03 AM, Uros Bizjak ubiz...@gmail.com wrote: On Thu, Dec 18, 2014 at 2:49 PM, Uros Bizjak ubiz...@gmail.com wrote: The Linux kernel never passes floating point arguments around, vararg functions or not. Hence no vector registers are ever used when calling a vararg

Re: [PATCH] X86-64: Add -mskip-rax-setup

2014-12-18 Thread Uros Bizjak
On Thu, Dec 18, 2014 at 3:09 PM, H.J. Lu hjl.to...@gmail.com wrote: The Linux kernel never passes floating point arguments around, vararg functions or not. Hence no vector registers are ever used when calling a vararg function. But gcc still dutifully emits an xor %eax,%eax before each and

Re: [C++ Patch] PR 60955

2014-12-18 Thread Jason Merrill
On 12/18/2014 06:17 AM, Paolo Carlini wrote: Sure. The below uses the c_inhibit_evaluation_warnings mechanism and passes testing. I wondered if in such cases we could alternately use the warning_sentinel mechanism (moved to cp-tree.h of course) ? That would make sense to me. Jason

Re: OpenACC middle end changes

2014-12-18 Thread Jakub Jelinek
On Thu, Dec 18, 2014 at 12:07:01PM +0100, Thomas Schwinge wrote: Many thanks for the review comments! The very most have been addresed, here are just a few comments. If you feel strongly/differently about any, I'll address those, too. So, with your latest change both compilers build: mkdir

Re: Pragma parsing (was: [PATCH] OpenACC for C++ front end)

2014-12-18 Thread Jakub Jelinek
On Thu, Dec 18, 2014 at 02:48:32PM +0100, Thomas Schwinge wrote: Like for CILK, I'd strongly prefer if for the clauses that are specific to OpenACC only you'd use PRAGMA_OACC_CLAUSE_* instead, and put them after the PRAGMA_CILK_* enum values. If you want to have PRAGMA_OACC_CLAUSE_ aliases

Re: [PATCH, ARM] attribute target (thumb,arm) [6/6] - [7/7]

2014-12-18 Thread Christian Bruel
Hello Ramana, I don't know if you have started to look at it, but the attribute support fails after upgrading. This patch aims to catch up on the changes around the fipa_ra -masm-syntax-unified options since the initial posting. They were not tested/supported with the attribute, and of

libgomp offloading testing (was: OpenACC middle end changes)

2014-12-18 Thread Thomas Schwinge
Hi Jakub! On Thu, 18 Dec 2014 15:20:42 +0100, Jakub Jelinek ja...@redhat.com wrote: So, with your latest change both compilers build: mkdir objmic; cd objmic ../configure --build=x86_64-intelmicemul-linux-gnu --host=x86_64-intelmicemul-linux-gnu --target=x86_64-intelmicemul-linux-gnu

[patch c++]: Fix PR/61198: Crash when selecting specializations through aliases.

2014-12-18 Thread Kai Tietz
Hi, this patch adds handling of aliases within templates and tries to resolve specialization for them. ChangeLog 2014-12-18 Kai Tietz kti...@redhat.com PR c++/61198 * pt.c (retrieve_specialization): Handle using. Tested on x86_64-w64-mingw32. Ok for apply? Regards, Kai ChangeLog

Re: [PATCH 4/4] OpenMP 4.0 offloading to Intel MIC: non-fallback testing

2014-12-18 Thread Thomas Schwinge
Hi! On Thu, 30 Oct 2014 14:40:01 +0300, Ilya Verbin iver...@gmail.com wrote: This patch allows to run non-fallback 'make check-target-libgomp'. It passes to the host compiler additional -B options with the paths to the offload compilers, since non-installed host compiler doesn't know

Re: [PATCH][ARM] Implement TARGET_SCHED_MACRO_FUSION_PAIR_P

2014-12-18 Thread Kyrill Tkachov
Ping. Thanks, Kyrill On 11/12/14 15:06, Kyrill Tkachov wrote: Ping. https://gcc.gnu.org/ml/gcc-patches/2014-12/msg00340.html Thanks, Kyrill On 04/12/14 09:19, Kyrill Tkachov wrote: On 02/12/14 22:58, Ramana Radhakrishnan wrote: On Tue, Nov 11, 2014 at 11:55 AM, Kyrill Tkachov

RE: [PATCH] Disable -fuse-caller-save when -pg is active

2014-12-18 Thread Radovan Obradovic
Patch has been tested with DejaGnu gcc test suite on mips32r2 cross compiler and bootstrapped and tested on x86_64 native compiler. Radovan From: Jeff Law [l...@redhat.com] Sent: Monday, November 17, 2014 1:18 PM To: Radovan Obradovic;

Re: OpenACC middle end changes

2014-12-18 Thread Thomas Schwinge
Hi Jakub! On Thu, 13 Nov 2014 19:09:49 +0100, Jakub Jelinek ja...@redhat.com wrote: --- gcc/builtins.c +++ gcc/builtins.c +static rtx +expand_builtin_acc_on_device (tree exp, rtx target ATTRIBUTE_UNUSED) +{ + if (!validate_arglist (exp, INTEGER_TYPE, VOID_TYPE)) +return

Re: [C++ Patch] PR 60955

2014-12-18 Thread Paolo Carlini
Hi, On 12/18/2014 03:20 PM, Jason Merrill wrote: On 12/18/2014 06:17 AM, Paolo Carlini wrote: Sure. The below uses the c_inhibit_evaluation_warnings mechanism and passes testing. I wondered if in such cases we could alternately use the warning_sentinel mechanism (moved to cp-tree.h of course)

Re: [PATCH 2/2] [C++] pr31397 - implement -Wsuggest-override

2014-12-18 Thread Trevor Saunders
On Wed, Dec 17, 2014 at 04:21:31PM -0500, Jason Merrill wrote: On 11/27/2014 01:28 AM, tsaund...@mozilla.com wrote: + if (warn_override DECL_VIRTUAL_P (decl) !DECL_OVERRIDE_P (decl) Why check DECL_VIRTUAL_P here? I think it was to avoid warning when decl is hiding a non virtual method

Re: [PATCH] PR 62173, re-shuffle insns for RTL loop invariant hoisting

2014-12-18 Thread Jiong Wang
On 17/12/14 15:54, Richard Biener wrote: On Mon, Dec 15, 2014 at 4:29 PM, Jiong Wang jiong.w...@arm.com wrote: On 15/12/14 15:28, Jiong Wang wrote: On 04/12/14 19:32, Jiong Wang wrote: On 04/12/14 11:07, Richard Biener wrote: On Thu, Dec 4, 2014 at 12:07 PM, Richard Biener

Re: [PATCH] X86-64: Add -mskip-rax-setup

2014-12-18 Thread H. Peter Anvin
On 12/18/2014 06:12 AM, Uros Bizjak wrote: # temporary until string.h is fixed KBUILD_CFLAGS += -ffreestanding Yes, it looks to me that new option is the way to go. Is this an OK? In principle, I'm OK with the patch approach, but let's wait for eventual comments from Linux people.

Aproved patches

2014-12-18 Thread Ricardo Sardano Guanciale
Please, can somebody help me? Where do I find GCC approved/accepted patch? Is there some patch directory in repository, if is please where? Thanks and best regards, Ricardo

Re: [PATCH][ARM][doc] Remove mention of Advanced RISC Machines

2014-12-18 Thread Kyrill Tkachov
Ping. https://gcc.gnu.org/ml/gcc-patches/2014-12/msg00947.html Thanks, Kyrill On 10/12/14 16:50, Kyrill Tkachov wrote: Hi all, The company ARM is not Advanced RISC Machines anymore and anyway the ARM architecture is distinct from the company. This patch adjusts the documentation accordingly.

Re: [PATCH] X86-64: Add -mskip-rax-setup

2014-12-18 Thread H.J. Lu
On Thu, Dec 18, 2014 at 9:08 AM, H. Peter Anvin h...@zytor.com wrote: On 12/18/2014 06:12 AM, Uros Bizjak wrote: # temporary until string.h is fixed KBUILD_CFLAGS += -ffreestanding Yes, it looks to me that new option is the way to go. Is this an OK? In principle, I'm OK with the patch

Re: [C++ Patch] PR 60955

2014-12-18 Thread Jason Merrill
On 12/18/2014 11:31 AM, Paolo Carlini wrote: + warning_sentinel s (extra_warnings); Let's add a comment about which warning we're avoiding here. OK with that change. Jason

Re: [PATCH] X86-64: Add -mskip-rax-setup

2014-12-18 Thread Uros Bizjak
On Thu, Dec 18, 2014 at 6:08 PM, H. Peter Anvin h...@zytor.com wrote: On 12/18/2014 06:12 AM, Uros Bizjak wrote: # temporary until string.h is fixed KBUILD_CFLAGS += -ffreestanding Yes, it looks to me that new option is the way to go. Is this an OK? In principle, I'm OK with the patch

Re: [patch c++]: Fix PR/61198: Crash when selecting specializations through aliases.

2014-12-18 Thread Jason Merrill
On 12/18/2014 10:10 AM, Kai Tietz wrote: + if (TMPL_ARGS_DEPTH (args) TMPL_PARMS_DEPTH (tmpl_parms)) +args = get_innermost_template_args +(args, TMPL_PARMS_DEPTH (tmpl_parms)); It seems unlikely to be correct to arbitrarily strip off outer template args. Where did the

Re: Aproved patches

2014-12-18 Thread Jakub Jelinek
On Thu, Dec 18, 2014 at 03:13:32PM -0200, Ricardo Sardano Guanciale wrote: Please, can somebody help me? Where do I find GCC approved/accepted patch? Is there some patch directory in repository, if is please where? Approved patches are usually just committed to the SVN repository, there is no

Re: [PATCH][ARM][doc] Remove mention of Advanced RISC Machines

2014-12-18 Thread James Greenhalgh
On Thu, Dec 18, 2014 at 05:18:18PM +, Kyrill Tkachov wrote: Ping. https://gcc.gnu.org/ml/gcc-patches/2014-12/msg00947.html This patch almost certainly falls under the obvious rule: will the person who objects to my work the most be able to find a fault with my fix? ARM hasn't stood

Re: [PATCH] IPA ICF: refactoring + fix for PR ipa/63569

2014-12-18 Thread Martin Liška
On 12/17/2014 04:23 PM, Richard Biener wrote: On Wed, Dec 17, 2014 at 12:17 PM, Martin Liška mli...@suse.cz wrote: On 12/11/2014 01:37 PM, Richard Biener wrote: On Wed, Dec 10, 2014 at 1:18 PM, Martin Liška mli...@suse.cz wrote: Hello. As suggested by Richard, I split compare_operand

Re: [PATCH 4/4] OpenMP 4.0 offloading to Intel MIC: non-fallback testing

2014-12-18 Thread Ilya Verbin
Hi, 2014-12-18 16:27 GMT+01:00 Thomas Schwinge tho...@codesourcery.com: Hi! On Thu, 30 Oct 2014 14:40:01 +0300, Ilya Verbin iver...@gmail.com wrote: This patch allows to run non-fallback 'make check-target-libgomp'. It passes to the host compiler additional -B options with the paths to

Re: [PATCH] X86-64: Add -mskip-rax-setup

2014-12-18 Thread H.J. Lu
On Thu, Dec 18, 2014 at 9:23 AM, Uros Bizjak ubiz...@gmail.com wrote: On Thu, Dec 18, 2014 at 6:08 PM, H. Peter Anvin h...@zytor.com wrote: On 12/18/2014 06:12 AM, Uros Bizjak wrote: # temporary until string.h is fixed KBUILD_CFLAGS += -ffreestanding Yes, it looks to me that new option is

Re: [PATCH] X86-64: Add -mskip-rax-setup

2014-12-18 Thread H. Peter Anvin
On 12/18/2014 09:43 AM, H.J. Lu wrote: Peter, please feel free to use my kernel patch or create a different one. Great, thanks! -hpa

Re: [PATCH 4/4] OpenMP 4.0 offloading to Intel MIC: non-fallback testing

2014-12-18 Thread Jakub Jelinek
On Thu, Dec 18, 2014 at 06:41:18PM +0100, Ilya Verbin wrote: What we're doing in OpenACC offloading testing (gomp-4_0-branch), is in the libgomp.oacc-c/c.exp (etc.) file cycle through all the available offloading devices, and then in there -- I think -- such options should be set, that are

Re: [C++ Patch] PR 60955

2014-12-18 Thread Paolo Carlini
Hi, On 12/18/2014 06:21 PM, Jason Merrill wrote: On 12/18/2014 11:31 AM, Paolo Carlini wrote: + warning_sentinel s (extra_warnings); Let's add a comment about which warning we're avoiding here. OK with that change. Thanks. I'm attaching what I just committed. This is a regression, I

Re: Pragma parsing

2014-12-18 Thread Thomas Schwinge
Hi Jakub! On Thu, 18 Dec 2014 15:29:42 +0100, Jakub Jelinek ja...@redhat.com wrote: On Thu, Dec 18, 2014 at 02:48:32PM +0100, Thomas Schwinge wrote: Like for CILK, I'd strongly prefer if for the clauses that are specific to OpenACC only you'd use PRAGMA_OACC_CLAUSE_* instead, and put

Re: Pragma parsing

2014-12-18 Thread Jakub Jelinek
On Thu, Dec 18, 2014 at 07:00:09PM +0100, Thomas Schwinge wrote: OK, so there is this limit. But, I fail to understand how merely moving the OpenACC-only PRAGMA_*_CLAUSE_* to the end of enum pragma_omp_clause will help overcome that? Or have I now completely confused myself, and I'm not even

Re: [patch c++]: Fix PR/61198: Crash when selecting specializations through aliases.

2014-12-18 Thread Kai Tietz
2014-12-18 18:26 GMT+01:00 Jason Merrill ja...@redhat.com: On 12/18/2014 10:10 AM, Kai Tietz wrote: + if (TMPL_ARGS_DEPTH (args) TMPL_PARMS_DEPTH (tmpl_parms)) +args = get_innermost_template_args +(args, TMPL_PARMS_DEPTH (tmpl_parms)); It seems unlikely to be correct to

Including a file from include/ in gcc/*.h (was: [gomp4] Use include/gomp-constants.h more actively)

2014-12-18 Thread Thomas Schwinge
Hi! On Wed, 17 Dec 2014 23:26:53 +0100, I wrote: Committed to gomp-4_0-branch in r218840: commit febcd8dfdb10fa80edff0880973d1915ca2fef74 Author: tschwinge tschwinge@138bc75d-0d04-0410-961f-82ee72b054a4 Date: Wed Dec 17 22:26:24 2014 + Use include/gomp-constants.h more

Re: Including a file from include/ in gcc/*.h (was: [gomp4] Use include/gomp-constants.h more actively)

2014-12-18 Thread Jakub Jelinek
On Thu, Dec 18, 2014 at 07:25:03PM +0100, Thomas Schwinge wrote: Hi! On Wed, 17 Dec 2014 23:26:53 +0100, I wrote: Committed to gomp-4_0-branch in r218840: commit febcd8dfdb10fa80edff0880973d1915ca2fef74 Author: tschwinge tschwinge@138bc75d-0d04-0410-961f-82ee72b054a4 Date: Wed Dec

Re: [PATCH] X86-64: Add -mskip-rax-setup

2014-12-18 Thread Rasmus Villemoes
On Thu, Dec 18 2014, H. Peter Anvin h...@zytor.com wrote: On 12/18/2014 09:43 AM, H.J. Lu wrote: Peter, please feel free to use my kernel patch or create a different one. Great, thanks! Thanks for accepting this idea, and especially to H.J. for already having done all the work. Minor

Re: [PATCH] X86-64: Add -mskip-rax-setup

2014-12-18 Thread H. Peter Anvin
On 12/18/2014 10:37 AM, Rasmus Villemoes wrote: Minor thing: If it's not too late, I'd appreciate a 'Suggested-by' or similar mention in the kernel change log. I think we can get that. -hpa

Re: [RFC, PATCH, fortran] PR fortran/60255 Deferred character length

2014-12-18 Thread Andre Vehreschild
Hi all, here is my next try on proposing a patch for the issue in pr60255. It took me quite some time to understand the intricacies with handling variables associated in a select type. I think I got most of the issues fixed now: - Added generation of _len component for each unlimited polymorphic

Re: [PATCH 0/9] Enable LRA on SH

2014-12-18 Thread Vladimir Makarov
On 2014-12-17 7:56 PM, Kaz Kojima wrote: This patch series is to make SH target use LRA and is discussed in the PR target/55212: [SH] Switch to LRA https://gcc.gnu.org/bugzilla/show_bug.cgi?id=55212 I think there are issues which should be solved before defaulting LRA on SH and the helps from

Re: [PATCH 0/9] Enable LRA on SH

2014-12-18 Thread Oleg Endo
On Thu, 2014-12-18 at 13:42 -0500, Vladimir Makarov wrote: Thank you very much for your work on porting LRA to SH! I know it is quite a challenge to do any RA work for this target. The R0-ness (both, GP reg r0 and FP reg fr0) of the ISA taxes the patience sometimes. Too often, I've been

Re: [PATCH] X86-64: Add -mskip-rax-setup

2014-12-18 Thread Mike Stump
On Dec 18, 2014, at 9:43 AM, H.J. Lu hjl.to...@gmail.com wrote: + arguments passed in vector registers. This can be used to + optimize Linux kernel./li English, to optimize _the_ Linux kernel?

Re: [debug-early] reuse variable DIEs and fix their context

2014-12-18 Thread Aldy Hernandez
Hi Jason. It's embarrassing that I just got to this now. I hope you don't repay the favor and take as long responding to me :(. On 09/12/14 08:15, Jason Merrill wrote: On 09/11/2014 08:51 PM, Aldy Hernandez wrote: /* Generate hidden aliases for Java. */ - if (candidates) + if

Re: [PATCH 0/9] Enable LRA on SH

2014-12-18 Thread Mike Stump
On Dec 18, 2014, at 10:42 AM, Vladimir Makarov vmaka...@redhat.com wrote: Thank you very much for your work on porting LRA to SH! I’ve ported my target to LRA, and I have not found any LRA bits I needed to change.

Re: [PATCH] X86-64: Add -mskip-rax-setup

2014-12-18 Thread H.J. Lu
On Thu, Dec 18, 2014 at 11:22 AM, Mike Stump mikest...@comcast.net wrote: On Dec 18, 2014, at 9:43 AM, H.J. Lu hjl.to...@gmail.com wrote: + arguments passed in vector registers. This can be used to + optimize Linux kernel./li English, to optimize _the_ Linux kernel? Updated. -- H.J. ---

Re: [PATCH 2/4] New data structure for cgraph_summary introduced.

2014-12-18 Thread Jan Hubicka
2014-12-08 Martin Liska mli...@suse.cz * cgraph.h (symbol_table::allocate_cgraph_symbol): Summary UID is filled up. * symbol-summary.h: New file. * gengtype.c (open_base_files): Add symbol-summary.h. * toplev.c (general_init): Call constructor of

Re: [PATCH 3/4] First usage of cgraph_summary in ipa-prop pass.

2014-12-18 Thread Jan Hubicka
gcc/lto/ChangeLog: 2014-12-08 Martin Liska mli...@suse.cz * lto-partition.c: Include of symbol-summary.h is added. * lto-symtab.c: Likewise. * lto.c: Likewise. I must say I am not friend of flattening. Hope this will be resolved for 5.1 gcc/ChangeLog:

Re: [PATCH 4/4] Data structure is used for inline_summary struct.

2014-12-18 Thread Jan Hubicka
2014-12-08 Martin Liska mli...@suse.cz * lto-partition.c (add_symbol_to_partition_1): New inline_summary_d is used. (undo_partition): Likewise. (lto_balanced_map): Likewise. gcc/ChangeLog: 2014-12-08 Martin Liska mli...@suse.cz * cgraphunit.c

[debug-early] C++: moving more things before the compilation proper

2014-12-18 Thread Aldy Hernandez
Hi Jason. As promised, I am moving most everything out of cxx_post_compilation_parsing_cleanups() which is the hook that will be called after the compilation proper has run (symtab-finalize_compilation_unit()). Apart from the Java hidden alias support which I just asked you about, and the

[PATCH, rs6000] PR59708, Fix operand ordering for 128-bit andc/orc

2014-12-18 Thread Pat Haugen
The following patch fixes testsuite failures builtin-arith-overflow-[10,11].c. In the case where rs6000_split_logical() was being called the operands were swapped such that the wrong operand was being complemented. Bootstrap/tested on powerpc64-linux with no new regressions. Ok for trunk

patch to solve PR64291

2014-12-18 Thread Vladimir Makarov
The following patch solves https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64291 It is a bug in a new rematerialization subpass of LRA. The patch was bootstrapped on x86/x86-64. Committed as rev. 218874. 2014-12-18 Vladimir Makarov vmaka...@redhat.com PR rtl-optimization/64291

Re: [RFC PATCH 3/9] Add TARGET_CANNOT_SUBSTITUTE_MEM_EQUIV target macro

2014-12-18 Thread Vladimir Makarov
On 2014-12-17 8:00 PM, Kaz Kojima wrote: This was discussed in PR55212 https://gcc.gnu.org/bugzilla/show_bug.cgi?id=55212#c25 https://gcc.gnu.org/bugzilla/show_bug.cgi?id=55212#c58 https://gcc.gnu.org/bugzilla/show_bug.cgi?id=55212#c69 and is to fix another ICEs in assign_by_spills. This patch

Re: [RFC PATCH 2/9] LRA: Swap base_term and index_term for some case

2014-12-18 Thread Vladimir Makarov
On 2014-12-17 7:59 PM, Kaz Kojima wrote: This was discussed in PR55212 https://gcc.gnu.org/bugzilla/show_bug.cgi?id=55212#c52 https://gcc.gnu.org/bugzilla/show_bug.cgi?id=55212#c53 https://gcc.gnu.org/bugzilla/show_bug.cgi?id=55212#c55 and is to fix ICE in assign_by_spills. In that case,

Re: [PATCH] PR 62173, re-shuffle insns for RTL loop invariant hoisting

2014-12-18 Thread Jiong Wang
2014-12-18 17:00 GMT+00:00 Jiong Wang jiong.w...@arm.com: ok for trunk? gcc/ PR62173 loop-invariant.c.c (expensive_addr): New hash_table. (need_expensive_addr_check_p): New bool. (find_exits): Rename to find_exists_and_reshuffle. Support re-shuffle instructions for

Re: [PATCH 0/9] Enable LRA on SH

2014-12-18 Thread Vladimir Makarov
On 2014-12-18 2:26 PM, Mike Stump wrote: On Dec 18, 2014, at 10:42 AM, Vladimir Makarov vmaka...@redhat.com wrote: Thank you very much for your work on porting LRA to SH! I’ve ported my target to LRA, and I have not found any LRA bits I needed to change. Thanks, Mike. When I did a few

Re: [RFC PATCH 4/9] Add TARGET_LEGITIMIZE_ADDRESS_DISPLACEMENT target macro

2014-12-18 Thread Vladimir Makarov
On 2014-12-17 8:00 PM, Kaz Kojima wrote: This patch adds a target macro discussed in PR55212 https://gcc.gnu.org/bugzilla/show_bug.cgi?id=55212#c76 https://gcc.gnu.org/bugzilla/show_bug.cgi?id=55212#c78 is to avoid bad codes on SH. SH has very limited base+displacement addressing and it looks

Re: [RFC PATCH 1/9] LRA: Take account implicit usage of pseudo reg in mem arg

2014-12-18 Thread Vladimir Makarov
On 2014-12-17 7:57 PM, Kaz Kojima wrote: This patch is discussed in PR55212 https://gcc.gnu.org/bugzilla/show_bug.cgi?id=55212#c47 https://gcc.gnu.org/bugzilla/show_bug.cgi?id=55212#c48 https://gcc.gnu.org/bugzilla/show_bug.cgi?id=55212#c49 and is to avoid segfault in remove_pseudos. In the

Re: [PATCH] Fix -fsanitize=float-cast-overflow with C FE (PR sanitizer/64344)

2014-12-18 Thread Joseph Myers
On Thu, 18 Dec 2014, Jakub Jelinek wrote: c/ * c-typeck.c (convert_for_assignment, c_finish_return): For -fsanitize=float-cast-overflow casts from REAL_TYPE to integer/enum types also set in_late_binary_op around convert call. * c-convert.c (convert): For

patch fixing a typo in LRA

2014-12-18 Thread Vladimir Makarov
The following patch fixes a typo in usage of register_move_cost. For most targets, the change does not matter as the cost function is symmetric relative to args. The patch was bootstrapped on x86-64. Committed as rev. 218875. 2014-12-18 Vladimir Makarov vmaka...@redhat.com

Re: [PATCH] PR 62173, re-shuffle insns for RTL loop invariant hoisting

2014-12-18 Thread Segher Boessenkool
On Thu, Dec 18, 2014 at 05:00:01PM +, Jiong Wang wrote: On 17/12/14 15:54, Richard Biener wrote: ick. I realize we don't have SSA form on RTL but doesn't DF provide at least some help in looking up definition statements for pseudos? In fact we want to restrict the transform to single-use

C++ PATCH for c++/64251 (unnecessary template instantiation)

2014-12-18 Thread Jason Merrill
Here we were seeing errors because of template instantiation triggered by mark_used within fold_non_dependent_expr. We don't want to instantiate templates based on uses in other uninstantiated templates, and we were already trying to avoid that; let's add another check to avoid it. Tested

C++ PATCH for c++/64352 (decltype and SFINAE)

2014-12-18 Thread Jason Merrill
We were just missing a case where we want to pass complain down to avoid a hard error in SFINAE context. Tested x86_64-pc-linux-gnu, applying to trunk. commit 83bea8460ce8e14e5b9d1c0dc33a0d22b63b566d Author: Jason Merrill ja...@redhat.com Date: Thu Dec 18 12:42:34 2014 -0500 PR

Re: [patch c++]: Fix PR/61198: Crash when selecting specializations through aliases.

2014-12-18 Thread Jason Merrill
On 12/18/2014 01:16 PM, Kai Tietz wrote: Well, in general I would have assumed to be able to get alias decl of tmpl. Wasn't able to find a simple way to get it. So, by looking into source I found that most cases handling args tmpl-args by using inner_most_template_args instead. Defaulting to

Re: flatten expr.h

2014-12-18 Thread Prathamesh Kulkarni
Hi, I had a discussion with Michael, and would like to put this patch on hold for now till we have tree.h and rtl.h flattening patches checked in to avoid conflicts. Thankyou, Prathamesh On 16 December 2014 at 17:13, Prathamesh Kulkarni prathamesh.kulka...@linaro.org wrote: Hi, The attached

Speedup and cleanup hash-table.h

2014-12-18 Thread Jan Hubicka
Hi, this patch started as experiment moving hash_table_mod1 inline because it shows high in streaming profiles and it represents a branch-less code that is good to schedule to surrounding instructions. While looking at hash-tab.h I however noticed that it was not updated very well while it was

C++ PATCH for c++/64105 (ICE on generic lambda with -std=c++11)

2014-12-18 Thread Jason Merrill
Since generic lambdas don't seem to work properly in C++11 mode, let's upgrade the pedwarn to an error. Tested x86_64-pc-linux-gnu, applying to trunk. commit 6c7d0ebb35b4918be3200625ef0cbb1816ddd7e9 Author: Jason Merrill ja...@redhat.com Date: Thu Dec 18 16:56:33 2014 -0500 PR c++/64105

Flatten tree.h and tree-core.h

2014-12-18 Thread Michael Collison
This patch flattens tree.h and tree-core.h. This work is part of the GCC Re-Architecture effort being led by Andrew MacLeod. I removed the includes in tree.h and tree-core.h except for the include of tree-core.h in tree.h. I modified genattrtab.c, genautomata.c, genemit.c, gengtype.c,

Re: Speedup and cleanup hash-table.h

2014-12-18 Thread Andi Kleen
Jan Hubicka hubi...@ucw.cz writes: Hi, this patch started as experiment moving hash_table_mod1 inline because it shows high in streaming profiles and it represents a branch-less code that is good to schedule to surrounding instructions. FWIW with a good modern hash function it shouldn't be

Re: [PATCH] -fsanitize-recover=list

2014-12-18 Thread Alexey Samsonov
Hi Jakub, On Tue, Nov 18, 2014 at 11:36 AM, Alexey Samsonov samso...@google.com wrote: On Mon, Nov 17, 2014 at 11:53 PM, Jakub Jelinek ja...@redhat.com wrote: On Mon, Nov 17, 2014 at 11:39:47PM -0800, Alexey Samsonov wrote: That is not what I think we've agreed on and what is implemented

  1   2   >