Re: [PATCH, libiberty]: Avoid "enum conversion when passing argument 1 of ‘getrusage’ is invalid in C++" warning

On Wed, Jul 1, 2015 at 12:07 PM, Uros Bizjak wrote: > This patch avoids "enum conversion when passing argument 1 of > ‘getrusage’ is invalid in C++" warning when compiling getruntime.c. > The condition is copied from sys/resource.h. > > 2015-07-01 Uros Bizjak > > * getruntime.c (get_run_ti

[PATCH] Fix fold_widened_comparison typo

I once wondered why fold didn't optimize (long)int-val > INT_MAX when niter analysis tried to fold this. Now when moving fold_widened_comparison to match.pd I noticed that a bogus condition on the limit folding causes it to only apply in the case when it won't simplify anything... Thus, fixed.

Re: [PATCH][ARM] Add debug dumping of cost table fields

Ping. https://gcc.gnu.org/ml/gcc-patches/2015-06/msg01064.html Thanks, Kyrill On 16/06/15 09:36, Kyrill Tkachov wrote: On 27/05/15 09:39, Andrew Pinski wrote: On Wed, May 27, 2015 at 4:38 PM, Kyrill Tkachov wrote: Ping. https://gcc.gnu.org/ml/gcc-patches/2015-05/msg00054.html This and the o

Re: [PATCH][ARM][stage-1] Initialise cost to COSTS_N_INSNS (1) and increment in arm rtx costs

Ping. Thanks, Kyrill On 17/06/15 11:16, Kyrill Tkachov wrote: Ping^5. https://gcc.gnu.org/ml/gcc-patches/2015-04/msg01130.html Thanks, Kyrill On 02/06/15 09:16, Kyrill Tkachov wrote: Ping^4. Thanks, Kyrill On 21/05/15 18:00, Kyrill Tkachov wrote: Ping^3. Thanks, Kyrill On 12/05/15 10:0

[PATCH] Fix for PR bootstrap/66744

Hello. Following small patch does what is described in $subject. Patch can boostrap on ppc64-linux-unknown-pc and on i686-w64-mingw32. Ready for trunk? Thanks, Martin >From 0b3f863ff9806ed9f802b18aabe0276830a3c38e Mon Sep 17 00:00:00 2001 From: mliska Date: Fri, 3 Jul 2015 17:48:01 +0200 Subject

Re: [PATCH 3/16][ARM] Add float16x4_t intrinsics

I haven't seen the patch yet but here are my thoughts on where this should be going. On 07/07/15 18:17, Alan Lawrence wrote: > Kyrill Tkachov wrote: >> On 07/07/15 17:34, Alan Lawrence wrote: >>> Kyrill Tkachov wrote: On 07/07/15 14:09, Kyrill Tkachov wrote: > Hi Alan, > > On 07/

Re: [PATCH][ARM][stage-1] Initialise cost to COSTS_N_INSNS (1) and increment in arm rtx costs

On 21/04/15 10:11, Kyrill Tkachov wrote: > Hi all, > > This is the first of a series to clean up and simplify the arm rtx costs > function. > This patch initialises the cost to COSTS_N_INSNS (1) at the top and > increments it when appropriate > in the rest of the function. This makes it more s

RE: [PATCH] MIPS: Update stack-1.c testcase to match micromips jraddiusp instruction.

> Yes, this is OK. Committed as SVN 225536. Regards, Andrew

Drop -Wswitch-bool warning in function.c

Bootstrapped & regression-tested on x86_64-linux-gnu :) 2015-07-08 Kito Cheng * function.c (stack_protect_epilogue): Use if rather than switch for check targetm.have_stack_protect_test(). From 0306990aac578167872a80ab55085d335e2bea14 Mon Sep 17 00:00:00 2001 From: Kito Cheng D

Re: fix PR46029: reimplement if conversion of loads and stores

Abe wrote: I`m uncertain to what that is intended to refer, but I believe Sebastian would agree that the new if converter is safer than the old one in terms of correctness at the time of running the code being compiled. > even if they take us a step backwards from a performance standpoint.

Re: [PATCH, i386]: Generate BT with immedate operand

On Tue, Jul 7, 2015 at 8:35 PM, Uros Bizjak wrote: > BT has *slightly* higher latency than TEST (0.33 vs. 0.25 cycles on a > modern processor), so I have limited the conversion to -Os in case the > bit-test is in the low 32 bits. A small update, in case of -Os, unpatched compiler generates andb

Re: [PATCH 15/16][fold-const.c] Fix bigendian HFmode in native_interpret_real

On Wed, Jul 8, 2015 at 12:07 AM, Jeff Law wrote: > On 07/07/2015 06:37 AM, Alan Lawrence wrote: >> >> As per https://gcc.gnu.org/ml/gcc-patches/2015-04/msg01346.html. Fixes >> FAIL of advsimd-intrinsics vcreate.c on aarch64_be-none-elf from >> previous patch. >> >> 15_native_interpret_real.patch >

Re: [PATCH] Fix for PR bootstrap/66744

On Wed, Jul 8, 2015 at 10:23 AM, Martin Liška wrote: > Hello. > > Following small patch does what is described in $subject. > Patch can boostrap on ppc64-linux-unknown-pc and on i686-w64-mingw32. > > Ready for trunk? Ok. Richard. > Thanks, > Martin

Re: [PATCH] Fix for PR bootstrap/66744

On Wed, Jul 08, 2015 at 10:23:43AM +0200, Martin Liška wrote: > PR bootstrap/66744 > * tree-sra.c (create_access_1): Can ctor without brackets. > (create_artificial_child_access): Likewise. When committing this, please s/Can/Call/ or something similar. Marek

Re: fix PR46029: reimplement if conversion of loads and stores

On Tue, Jul 7, 2015 at 11:23 PM, Abe wrote: >> (if-conversion could directly generate masked load/stores >> of course and not use a scratch-pad at all in that case). > > > IMO that`s a great idea, but I don`t know how to do it. Hints would be > welcome. In particular, how does one "generate mas

Re: [PATCH] Make SSA propagator iteration order consistent

On 2015.07.06 at 14:40 +0200, Richard Biener wrote: > > The intent (as I read it) of the iteration order in ssa_propagate is > to process stmts in the following order: > > 1) complete simulation of BBs from making one of their entries executable > 2) simulation of stmts fed by stmts that change

Re: [PATCH, 2/2][PR66642] Add empty loop exit block in transform_to_exit_first_loop_alt

Tom de Vries writes: > * testsuite/libgomp.c/parloops-exit-first-loop-alt-3.c (main): Test low > iteration count case. ../../../../libgomp/testsuite/libgomp.c/parloops-exit-first-loop-alt-3.c: In function 'main': ../../../../libgomp/testsuite/libgomp.c/parloops-exit-first-loop-alt-3

[PATCH, MIPS] Support interrupt handlers with hard-float

Hi Matthew/Catherine, The attached patch removes the restriction to compile a TU with an ISR with -mhard-float. Instead of forcing -msoft-float, the coprocessor 1 is disabled in an ISR for -mhard-float. Ok to apply? Regards, Robert gcc/ * config/mips/mips.c (mips_compute_frame_info):

[PATCH, MIPS] Support new interrupt handler options

Hi, This patch adds support for optional arguments for interrupt and use_shadow_register_set attributes. The patch also fixes an ICE if both interrupt and use_shadow_register_set are enabled and compiled with -mips64r2 -mabi=64 discovered during testing of the attached test. The interrupt att

[PATCH, MIPS] Fix restoration of hi/lo in MIPS64R2 interrupt handlers

Hi, The attached patch fixes an ICE (unrecognizable insn) when accumulators are used in interrupt handlers for MIPS64R2. There was just a typo in the function name. Ok to apply? Regards, Robert gcc/ * config/mips/mips.c (mips_emit_save_slot_move): Fix typo. gcc/testsuite/ * g

Re: [PATCH 15/16][fold-const.c] Fix bigendian HFmode in native_interpret_real

Richard Biener wrote: On Wed, Jul 8, 2015 at 12:07 AM, Jeff Law wrote: On 07/07/2015 06:37 AM, Alan Lawrence wrote: [snip] Fix native_interpret_real for HFmode floats on Bigendian with UNITS_PER_WORD>=4 (with missing space) OK with ChangeLog in proper form. Err - but now offset

Re: [PATCH 4/7] Fix int overflow

On Mon, Jul 6, 2015 at 12:36 PM, Mikhail Maltsev wrote: > > diff --git a/libiberty/cp-demangle.c b/libiberty/cp-demangle.c > index 44a0a9b..befa6b6 100644 > --- a/libiberty/cp-demangle.c > +++ b/libiberty/cp-demangle.c > @@ -103,6 +103,7 @@ > #include "config.h" > #endif > > +#include All exis

Re: [PATCH 3/7] Fix trinary op

On Tue, Jul 7, 2015 at 3:40 PM, Jeff Law wrote: > > And a generic question on the testsuite -- presumably it turns on type > demangling?I wanted to verify the flow through d_expression_1 was what I > expected it to be and it took a while to realize that c++filt doesn't > demangle types by defa

[Committed] S/390: Remove assertion in s390_init_frame_layout

Hi, since r225260 the backend register elimination hooks also get invoked after reload is completed e.g. from sched2. During register elimination we call s390_init_frame_layout which is not supposed to be invoked after reload. We had an assertion here to make sure this does not happen. This got

Re: [PATCH, testcase, committed] Exit with zero status from g++.dg/ipa/pr61160-3.C

Hi, apparently I have forgot to backport this to the 4.9 branch which was now pointed out in a separate PR 61820. The patch is obvious and testcase-only, so I have done the backport now, after some basic testing, so that I can close the bug. Thanks, Martin On Tue, Jul 22, 2014 at 06:31:32PM +

[PATCH] PR target/66806: Don't pass/return vectors in registers for IAMCU

Vectors should be passed in memory for IAMCU. OK for trunk? H.J. --- gcc/ PR target/66806 * config/i386/i386.c (function_arg_advance_32): Don't pass vectors in registers for IAMCU. (function_arg_32): Likewise. (ix86_return_in_memory): Don't return vectors

Re: [PATCH] PR target/66806: Don't pass/return vectors in registers for IAMCU

On Wed, Jul 8, 2015 at 1:41 PM, H.J. Lu wrote: > Vectors should be passed in memory for IAMCU. > > OK for trunk? Bootstrapped and regression tested? Uros.

[PATCH] Fix PR66794

Passes do not expect post-dominators being around and thus forget to invalidate them properly. Thus passes computing them have to free them. The patch fixes path-isolation and adds an assert so this doesn't happen again. Bootstrapped and tested on x86_64-unknown-linux-gnu, applied to trunk. Ri

Re: [PATCH] PR target/66806: Don't pass/return vectors in registers for IAMCU

On Wed, Jul 8, 2015 at 1:41 PM, H.J. Lu wrote: > Vectors should be passed in memory for IAMCU. > > OK for trunk? > > H.J. > --- > gcc/ > > PR target/66806 > * config/i386/i386.c (function_arg_advance_32): Don't pass > vectors in registers for IAMCU. > (function_arg_

[PATCH] Fix PR66793

The following fixes path-isolation to properly split the block if it inserts a trap after a stmt ending a BB (in this case a noreturn stmt). Bootstrapped and tested on x86_64-unknown-linux-gnu, applied to trunk. Richard. 2015-07-08 Richard Biener PR tree-optimization/66793

[PATCH] Fix PR66805 - #pragma pack affecting gcov_info_type layout

The following fixes #pragma pack effect leaking to all types built from the middle-end (so possibly even vector types built by the vectorizer?). The PR in question is about gcov_info_type where layout is affected and inconsistency between that and the libgcov.a copy causes libgcov to crash. As t

[patch] Simplify signatures of std::list members: merge, splice, insert, erase

C++0x changed the signature of std::list::merge to take an rvalue reference, and then LWG DR 1133 [1] added the lvalue reference signature back as an overload. Our implementation follows that history rather literally, as we changed list::merge to take an rvalue in C++0x mode, then later [2] added

Re: [PATCH] Fix PR66805 - #pragma pack affecting gcov_info_type layout

On Wed, 8 Jul 2015, Richard Biener wrote: > > The following fixes #pragma pack effect leaking to all types built > from the middle-end (so possibly even vector types built by the > vectorizer?). The PR in question is about gcov_info_type where > layout is affected and inconsistency between that

Re: [PATCH] Fix PR66805 - #pragma pack affecting gcov_info_type layout

On Wed, Jul 08, 2015 at 01:58:38PM +0200, Richard Biener wrote: > > The following fixes #pragma pack effect leaking to all types built > from the middle-end (so possibly even vector types built by the > vectorizer?). The PR in question is about gcov_info_type where > layout is affected and incons

Re: [patch] Simplify signatures of std::list members: merge, splice, insert, erase

On 08/07/15 13:00 +0100, Jonathan Wakely wrote: patch2.txt then does the same thing for splice(), introducing a __const_iterator typedef [3] to make the signatures consistent for different language modes. I think this makes the code more maintainable, as there are fewer #if blocks with subtly di

Re: [PATCH] Fix PR66805 - #pragma pack affecting gcov_info_type layout

The same bug was earlier reported as PR gcov-profile/43341: https://gcc.gnu.org/bugzilla/show_bug.cgi?id=43341 Alexander

Re: [PATCH] Fix PR66805 - #pragma pack affecting gcov_info_type layout

On Wed, 8 Jul 2015, Jakub Jelinek wrote: > On Wed, Jul 08, 2015 at 01:58:38PM +0200, Richard Biener wrote: > > > > The following fixes #pragma pack effect leaking to all types built > > from the middle-end (so possibly even vector types built by the > > vectorizer?). The PR in question is about

Re: [PATCH] Fix PR66805 - #pragma pack affecting gcov_info_type layout

On Wed, Jul 08, 2015 at 02:40:33PM +0200, Richard Biener wrote: > > toplev.c already sets maximum_field_alignment directly, and to a different > > value: > > maximum_field_alignment = initial_max_fld_align * BITS_PER_UNIT; > > so I'm not sure you need a new function. And, shouldn't you reset to >

Re: [PATCH 15/16][fold-const.c] Fix bigendian HFmode in native_interpret_real

On Wed, Jul 8, 2015 at 12:51 PM, Alan Lawrence wrote: > Richard Biener wrote: >> >> On Wed, Jul 8, 2015 at 12:07 AM, Jeff Law wrote: >>> >>> On 07/07/2015 06:37 AM, Alan Lawrence wrote: > > [snip] Fix native_interpret_real for HFmode floats on Bigendian with UNITS_PER_WORD>=4

Re: [PATCH, 2/2][PR66642] Add empty loop exit block in transform_to_exit_first_loop_alt

On 08/07/15 12:40, Andreas Schwab wrote: Tom de Vries writes: * testsuite/libgomp.c/parloops-exit-first-loop-alt-3.c (main): Test low iteration count case. ../../../../libgomp/testsuite/libgomp.c/parloops-exit-first-loop-alt-3.c: In function 'main': ../../../../libgomp/tests

Re: Tests for libgomp based on OpenMP Examples 4.0.2.

Updated the attached files: renamed also libgomp.c++ tests, corrected ChangeLog. 2015-07-07 21:29 GMT+03:00 Ilya Verbin : > On Tue, Jul 07, 2015 at 20:17:48 +0200, Jakub Jelinek wrote: >> On Tue, Jul 07, 2015 at 08:08:16PM +0300, Maxim Blumental wrote: >> > > Added 16 tests for simd construct and

Re: Tests for libgomp based on OpenMP Examples 4.0.2.

>The Examples-4/ >directory is supposed to only contain the tests from the 4.0.* examples >document and no other tests. All right: everything I added in it was only tests from the examples document. Renamings were made in accordance with filenames from GitHub. > Any new tests that aren't in Example

[gomp4, committed] Add rewrite_virtuals_into_loop_closed_ssa

[ was: Re: [PING][PATCH, 1/2] Merge rewrite_virtuals_into_loop_closed_ssa from gomp4 branch ] On 07/07/15 17:58, Tom de Vries wrote: Bootstrapped reg-tested on x86_64. Committed to trunk as attached. Reverted related patches on gomp-4_0-branch, and committed this patch instead. Thanks, -

Re: Drop -Wswitch-bool warning in function.c

On Wed, Jul 08, 2015 at 04:49:19PM +0800, Kito Cheng wrote: > Bootstrapped & regression-tested on x86_64-linux-gnu :) > > 2015-07-08 Kito Cheng > > * function.c (stack_protect_epilogue): Use if rather than switch for > check targetm.have_stack_protect_test(). Do you really nee

Re: [PING][PATCH, 1/2] Merge rewrite_virtuals_into_loop_closed_ssa from gomp4 branch

On Tue, 7 Jul 2015, Tom de Vries wrote: > On 06/07/15 15:44, Richard Biener wrote: > > Please add this to tree-cfg.[ch] instead, there are multiple places > > in GCC that would benefit from it > > Done. > > A lot of calls to mark_virtual_phi_result_for_renaming look like they could be > rewritte

Re: [PATCH 3/7] Fix trinary op

> "Ian" == Ian Lance Taylor writes: Ian> I don't know of anybody who actually uses the DMGL_TYPES support. I Ian> don't know why anybody would. It's used in gdb's DWARF reader, though I no longer remember why. Tom

Re: [PATCH 3/7] Fix trinary op

On Wed, Jul 8, 2015 at 6:42 AM, Tom Tromey wrote: >> "Ian" == Ian Lance Taylor writes: > > Ian> I don't know of anybody who actually uses the DMGL_TYPES support. I > Ian> don't know why anybody would. > > It's used in gdb's DWARF reader, though I no longer remember why. Looking at the code

Re: [PATCH 3/4] Add libgomp plugin for Intel MIC

Hi! On Tue, 21 Oct 2014 21:24:13 +0400, Ilya Verbin wrote: > This patch contains a plugin for libgomp and appropriate changes for > makefiles. > > The plugin uses liboffloadmic_host.so to interact with the device (or with an > emulator). Also the patch contains offload_target_main executable,

[PATCH] Teach genmatch.c to generate single-use restrictions from flags

This introduces a :s flag to match expressions which enforces the expression to have a single-use if(!) the simplified expression is larger than one statement. Thus with that we for example allow tem = a + 1; x = tem - 3; foo (tem); to simplify to tem = a + 1; x = a - 2; foo (tem);

[PATCH, committed] PR jit/66783: improve error messages

Tested with "make check-jit"; jit.sum remains at 8494 passes. Committed to trunk as r225557. gcc/jit/ChangeLog: PR jit/66783 * libgccjit.c (gcc_jit_context_new_field): Show name of field in "unknown size" error message. (gcc_jit_struct_set_fields): Show name of str

Re: [gomp] Move openacc vector& worker single handling to RTL

On 07/07/15 10:22, Jakub Jelinek wrote: I agree that fork/join might be less confusing. this version is the great renaming. I've added fork & join internal fns. In the PTX backend I've added 4 new unspecs: fork -- the final single mode insn forked -- the first partitioned mode insn joinin

Re: [PATCH 2/4 v2: part 2] libcpp: Replace macro usage with C++ constructs

Hi! On Tue, 5 May 2015 14:21:13 -0400, David Malcolm wrote: > libcpp/ChangeLog: > * include/line-map.h (MAX_SOURCE_LOCATION): Convert from a macro > to a const source_location. > (RESERVED_LOCATION_COUNT): Likewise. > [...] > --- a/libcpp/include/line-map.h > +++ b/libcp

Re: [C++/66443] virtual base of abstract class

On 06/30/15 19:21, Nathan Sidwell wrote: On 06/30/15 00:19, Jason Merrill wrote: On 06/29/2015 06:57 PM, Nathan Sidwell wrote: * method.c (synthesized_method_walk): Skip virtual bases of abstract classes in C++14 mode. Let's not limit this to C++14 mode; most DRs apply to earlier stan

Patch to make dwarf2 tests pass when dwarf-2 is not default

Currently, most tests in g++.dg/debug/dwarf2 include a "dg-options" line which passes "-g" to the compiler. This overrides the "-gdwarf-2" that appears in DEFAULT_CFLAGS in dwarf2.exp. As a consequence, default debug information is generated, which in systems when dwarf-2 is not the default makes

[PATCH, committed] fix comments in a jit testcase

Fix a copy-and-paste error. Tested with "make check-jit"; jit.sum remains at 8494 passes. Committed to trunk as r225559. gcc/testsuite/ChangeLog: * jit.dg/test-error-gcc_jit_block_end_with_switch-NULL-case.c: Fix comments. --- ...t-error-gcc_jit_block_end_with_switch-NULL-case.c

Re: [PATCH] PR target/66806: Don't pass/return vectors in registers for IAMCU

On Wed, Jul 8, 2015 at 4:49 AM, Uros Bizjak wrote: > On Wed, Jul 8, 2015 at 1:41 PM, H.J. Lu wrote: >> Vectors should be passed in memory for IAMCU. >> >> OK for trunk? >> >> H.J. >> --- >> gcc/ >> >> PR target/66806 >> * config/i386/i386.c (function_arg_advance_32): Don't pass >>

Re: [gomp] Move openacc vector& worker single handling to RTL

On Wed, Jul 08, 2015 at 10:47:56AM -0400, Nathan Sidwell wrote: > +/* Generate loop head markers in outer->inner order. */ > + > +static void > +gen_oacc_fork (gimple_seq *seq, unsigned mask) > +{ > + { > +// TODDO: Determine this information from the parallel region itself TODO ? > +//

Re: [PATCH] Fix PR66794

On Wed, Jul 8, 2015 at 4:46 AM, Richard Biener wrote: > > Passes do not expect post-dominators being around and thus forget > to invalidate them properly. Thus passes computing them have to > free them. The patch fixes path-isolation and adds an assert so > this doesn't happen again. > > Bootstr

[nvptx offloading] Only 64-bit configurations are currently supported (was: nvptx-tools and nvptx-newlib)

Hi! On Wed, 18 Feb 2015 09:50:15 +0100, I wrote: > So far, we have concentrated only on the 64-bit x86_64 configuration; > 32-bit has several known issues to be resolved. > filed. I have committed the following patch in r225560. This gets us rid of the lots of "expe

[PATCH]Fix PR66556. Don't drop side-effect in simplify_const_relational_operation function.

Hi all, In simplify_const_relational_operation function, there are cases a const rtx will be returned. Three cases are considered in this function: 1, comparisons with upper and lower bounds. 2, Integer comparisons with zero. 3, comparison of ABS with zero. It's fine to to the optimization if t

Patch to fix PR66334

The following patch fixes https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66334 The patch was bootstrapped and tested on x86/x86-64. Committed as rev. 225561. 2015-07-08 Vladimir Makarov PR middle-end/66334 * ira-lives.c (process_bb_node_lives): Make conflicts with PIC

Re: [PATCH 3/4] Add libgomp plugin for Intel MIC

> On 8 июля 2015 г., at 17:16, Thomas Schwinge wrote: > > Hi! > >> On Tue, 21 Oct 2014 21:24:13 +0400, Ilya Verbin wrote: >> This patch contains a plugin for libgomp and appropriate changes for >> makefiles. >> >> The plugin uses liboffloadmic_host.so to interact with the device (or with an

Re: [PATCH 2/4 v2: part 2] libcpp: Replace macro usage with C++ constructs

On Wed, 2015-07-08 at 16:50 +0200, Thomas Schwinge wrote: > Hi! > > On Tue, 5 May 2015 14:21:13 -0400, David Malcolm wrote: > > libcpp/ChangeLog: > > * include/line-map.h (MAX_SOURCE_LOCATION): Convert from a macro > > to a const source_location. > > (RESERVED_LOCATION_COUNT): Likewi

RE: [PATCH] MIPS: fix failing branch range checks for micromips

> > testsuite/ > > * gcc.target/mips/branch-2.c: Change NOMIPS16 to > > NOCOMPRESSION. > > * gcc.target/mips/branch-3.c: Ditto > > * gcc.target/mips/branch-4.c: Ditto. > > * gcc.target/mips/branch-5.c: Ditto. > > * gcc.target/mips/branch-6.c: Ditto. > > * gcc.target/mips/bra

[C++ Patch] PR 66644

Hi, this is about the anonymous struct extension, but I think submitter is right that we should accept the testcase (indeed both clang and edg already do): in the below I propose to simply add a test of ANON_AGGR_TYPE_P (DECL_CONTEXT (field)). Note that, given the definition of ANON_AGGR_TYPE

Re: [PATCH] fix PR46029: reimplement if conversion of loads and stores [2nd submitted version of patch]

[Alan wrote:] Where can I find info on what the different flag values mean? (I had thought they were booleans [...] [Abe wrote:] Sorry; I don`t know if that is documented anywhere yet. In this case, (-1) simply means "defaulted": on if the vectorizer is on, and off if it is off. (0) mea

[PATCH] Limit alignment on error_mark_node variable

There is no need to try different alignment on variable of error_mark_node. OK for trunk if there is no regression? Thanks. H.J. -- gcc/ PR target/66810 * varasm.c (align_variable): Don't try different alignment on variable of error_mark_node. gcc/testsuite/ PR

Re: fix PR46029: reimplement if conversion of loads and stores

[Abe wrote:] I believe Sebastian would agree that the new if converter is safer than the old one >> in terms of correctness at the time of running the code being compiled. [...] For now, we have a few performance regressions [snip] [Alan wrote:] TBH my two cents would be that a performanc

Re: [PATCH 3/4] Add libgomp plugin for Intel MIC

Hi! On Wed, 8 Jul 2015 18:13:56 +0300, Ilya Verbin wrote: > > > On 8 июля 2015 г., at 17:16, Thomas Schwinge > > wrote: > > With recent GCC trunk sources, builds of the Intel MIC Offload Plugin > > fail as follows: [...] > > [...] -- OK to commit the following? > Ok to me, thanks. Committed

Re: [PATCH] PR target/66806: Don't pass/return vectors in registers for IAMCU

On Wed, Jul 8, 2015 at 4:57 PM, H.J. Lu wrote: >>> Vectors should be passed in memory for IAMCU. >>> >>> OK for trunk? >>> >>> H.J. >>> --- >>> gcc/ >>> >>> PR target/66806 >>> * config/i386/i386.c (function_arg_advance_32): Don't pass >>> vectors in registers for IAMCU. >

RE: [PATCH] MIPS: fix failing branch range checks for micromips

> -Original Message- > From: Andrew Bennett [mailto:andrew.benn...@imgtec.com] > Sent: Wednesday, July 08, 2015 11:17 AM > To: Moore, Catherine; gcc-patches@gcc.gnu.org > Cc: Matthew Fortune > Subject: RE: [PATCH] MIPS: fix failing branch range checks for micromips > > > > testsuite/ > >

Re: Drop -Wswitch-bool warning in function.c

Hi Marek: Yes, I know it's non-fatal warning, but I think gcc should build with --enable-werror-always by it's self and it's the *ONLY* warning in trunk now. Of cause, cast to int can suppress the warning, but it's not good solution so gcc complain that switch condition has bool type :) On Wed,

Re: fix PR46029: reimplement if conversion of loads and stores

(if-conversion could directly generate masked load/stores of course and not use a scratch-pad at all in that case). [Abe wrote:] IMO that`s a great idea, but I don`t know how to do it. Hints would be welcome. In particular, how does one >> "generate masked load/stores" at the GIMPLE level?

darwin fix for gcc-5 (RM please)

I’d like to merge in the fix from https://gcc.gnu.org/PR66523 into the gcc-5-branch. RM Ok? https://gcc.gnu.org/bugzilla/attachment.cgi?id=35773: diff --git a/gcc/config/darwin.c b/gcc/config/darwin.c index 40804b8..0080299 100644 --- a/gcc/config/darwin.c +++ b/gcc/config/darwin.c @@ -1259,6 +1

Re: [AArch64][2/2] Define TARGET_UNSPEC_MAY_TRAP_P for AArch64

On Tue, Jul 07, 2015 at 01:52:29PM +0100, Jiong Wang wrote: > > A second patch to improve rtl loop iv on AArch64. > > We should define this to tell gcc the pattern hidden by these GOT unspec > is safe from trap, so gcc could make more positive decision when > handling them, for example in RTL loo

Re: [PATCH]Fix PR66556. Don't drop side-effect in simplify_const_relational_operation function.

On Wed, Jul 08, 2015 at 04:03:47PM +0100, Renlin Li wrote: > PR rtl/66556 > * simplify-rtx.c (simplify_const_relational_operation): Add > side_effects_p check. "checks"? The patch looks good to me, but someone else will need to approve. Segher

Re: [PATCH] PR target/66746: Failure to compile #include with -miamcu

On Thu, Jul 2, 2015 at 11:16 PM, Uros Bizjak wrote: > On Fri, Jul 3, 2015 at 5:53 AM, H.J. Lu wrote: >> x86intrin.h has useful intrinsics for instructions for IA MCU. This >> patch adds __iamcu__ check to x86intrin.h and ia32intrin.h. >> >> OK for trunk? >> >> H.J. >> --- >> gcc/ >> >> P

Re: [RS6000 7/7] Address cost

On Wed, Jun 24, 2015 at 10:25:46AM +0930, Alan Modra wrote: > @@ -30720,14 +30721,20 @@ rs6000_rtx_costs (rtx x, machine_mode mode, int > outer_code, > - (outer_code == SET ? 1 : 0)); >return true; > > -case CONST: > -case HIGH: > -case SYMBOL_REF:

Re: [PATCH] fix PR46029: reimplement if conversion of loads and stores [2nd submitted version of patch]

Abe wrote: [Alan wrote:] Where can I find info on what the different flag values mean? (I had thought they were booleans [...] [Abe wrote:] Sorry; I don`t know if that is documented anywhere yet. In this case, (-1) simply means "defaulted": on if the vectorizer is on, and off if it is

[committed, PATCH] Add abort prototype to readeflags-1.c/writeeflags-1.c

Index: ChangeLog === --- ChangeLog (revision 225565) +++ ChangeLog (working copy) @@ -1,3 +1,8 @@ +2015-07-08 H.J. Lu + + * gcc.target/i386/readeflags-1.c (abort): New prototype. + * gcc.target/i386/writeeflags-1.c (

Re: [committed] Flags outputs for asms

On Mon, Jun 29, 2015 at 7:44 AM, Richard Henderson wrote: > Sorry for the delay, but here's the third and final version. > This includes the requested cpp symbol and updated documentation. gcc.target/i386/asm-flag-5.c failed with -march=pentium: https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66813

Re: RFC: Add ADDR_EXPR lowering (PR tree-optimization/66718)

On Sun, Jul 05, 2015 at 11:10:14AM +0800, Bin.Cheng wrote: > > Just FYI, while bootstrapping/regtesting your patch together with the one > > I've posted and another one to vectorize pr59984.c better, I've noticed > > there > > is another regression with your patch (reverting my patches doesn't hel

Re: RFC: Add ADDR_EXPR lowering (PR tree-optimization/66718)

On Fri, Jul 03, 2015 at 03:41:29PM +0200, Richard Biener wrote: > On Fri, 3 Jul 2015, Marek Polacek wrote: > > > This patch implements a new pass, called laddress, which deals with > > lowering ADDR_EXPR assignments. Such lowering ought to help the > > vectorizer, but it also could expose more CS

Re: [PATCH] PR target/66746: Failure to compile #include with -miamcu

On Wed, Jul 8, 2015 at 6:43 PM, H.J. Lu wrote: >>> x86intrin.h has useful intrinsics for instructions for IA MCU. This >>> patch adds __iamcu__ check to x86intrin.h and ia32intrin.h. >>> >>> OK for trunk? >>> >>> H.J. >>> --- >>> gcc/ >>> >>> PR target/66746 >>> * config/i386/ia3

[gomp4] Handle Fortran deviceptr clause.

Hi, This patch adds handling of the deviceptr clause when used within a Fortran program. Committed to gomp-4_0-branch Jim diff --git a/libgomp/ChangeLog.gomp b/libgomp/ChangeLog.gomp index 418474d..1949d78 100644 --- a/libgomp/ChangeLog.gomp +++ b/libgomp/ChangeLog.gomp @@ -1,3 +1,8 @@ +2015-0

Re: [PATCH] fix PR46029: reimplement if conversion of loads and stores [2nd submitted version of patch]

[Abe wrote:] Is the following sufficient? From "doc/invoke.texi", I propose to replace: This is enabled by default if vectorization is enabled. ... with: This is enabled by default when vectorization is enabled anddisabled by default when vectorization is disabled. [Al

Re: [GOOGLE] Only save command-line options for LIPO

Resending in plain text mode. On Wed, Jul 8, 2015 at 11:34 AM, Teresa Johnson wrote: > This patch avoids saving optimization pragma or function level attributes in > the list of command line options saved in the LIPO module info structure. > Passes regression and internal testing. Ok for google/4

[committed, PATCH] Compile pr37870.c with -mlong-double-80

On x86, the "long double" type is platforma specific, which may be the same as double or __float128. Since pr37870.c requires 80-bit floating point type, it should be compiled with -mlong-double-80. * gcc.target/i386/pr37870.c (dg-options): Add -mlong-double-80. --- gcc/testsuite/gcc.tar

Re: [PATCH] S390: Support -mtune=native and -march=native.

> Version 2 of the patch to enable the configure options > --with-arch=native and --with-tune=native. This patch broke cross-compiling with --target=s390-* s390_host_detect_local_cpu is only defined if the --host is s390-* but EXTRA_SPEC_FUNCTIONS refers to it when --target is s390-*

[committed] Fix -fopenmp-simd C++ handling and one declare simd error recovery fix

Hi! While hacking on gomp-4.1 branch, I've noticed that -fopenmp-simd is broken for C++ for #pragma omp declare simd, and there is also an error recovery issue in c_omp_declare_simd_clauses_to_numbers. Additionally, tree-vect.h had some issues when included from C++ testcases, fixed those too so t

Re: [PATCH] PR target/35514: Gcc shoud generate symbol type for undefined symbol

On Sun, Jul 5, 2015 at 2:54 PM, H.J. Lu wrote: > On Sun, Jul 5, 2015 at 11:14 AM, H.J. Lu wrote: >> Update default_elf_asm_output_external to also output symbol type to >> help ELF linker to properly issue diagnostic message. We don't output >> symbol type for reference to external TLS symbol si

Re: [PATCH]Fix PR66556. Don't drop side-effect in simplify_const_relational_operation function.

On 07/08/2015 09:03 AM, Renlin Li wrote: Hi all, In simplify_const_relational_operation function, there are cases a const rtx will be returned. Three cases are considered in this function: 1, comparisons with upper and lower bounds. 2, Integer comparisons with zero. 3, comparison of ABS with ze

Re: Drop -Wswitch-bool warning in function.c

On 07/08/2015 02:49 AM, Kito Cheng wrote: Bootstrapped & regression-tested on x86_64-linux-gnu :) 2015-07-08 Kito Cheng * function.c (stack_protect_epilogue): Use if rather than switch for check targetm.have_stack_protect_test(). OK. Not necessarily because avoid the warn

[PATCH] PR target/66817: Check int_size_in_bytes in ix86_return_in_memory

ix86_return_in_memory should check negative return from int_size_in_bytes, similar to other ports. Tested on Linux/x86-64. OK for trunk? Thanks. H.J. --- gcc/ PR target/66817 * config/i386/i386.c (ix86_return_in_memory): Return true if int_size_in_bytes returns negative

Re: [gomp] Move openacc vector& worker single handling to RTL

On 07/08/15 10:58, Jakub Jelinek wrote: On Wed, Jul 08, 2015 at 10:47:56AM -0400, Nathan Sidwell wrote: +/* Generate loop head markers in outer->inner order. */ + +static void +gen_oacc_fork (gimple_seq *seq, unsigned mask) +{ + { +// TODDO: Determine this information from the parallel reg

Adjust -fdump-ada-spec to C++14 switch (2)

We need to skip the constexpr default constructors. Tested on x86_64-suse-linux, applied on the mainline as obvious. 2015-07-08 Eric Botcazou c-family/ * c-ada-spec.h (cpp_operation): Add IS_CONSTEXPR. * c-ada-spec.c (print_ada_declaration): Skip constexpr constructors. cp/

Re: RFC: Add ADDR_EXPR lowering (PR tree-optimization/66718)

On Wed, Jul 08, 2015 at 07:37:41PM +0200, Marek Polacek wrote: > DEFTIMEVAR (TV_INITIALIZE_RTL, "initialize rtl") > +DEFTIMEVAR (TV_GIMPLE_LADDRESS , "address lowering") Consider this whitespace issue fixed. Marek

[PATCH] Fix pr 66796, testsuite failure on hppa-linux

As detailed in the PR notes, this test behaves differently on linux vs hpux targets due to differences in how they handle the space/segment registers. By changing the test slightly we can remove that difference in behaviour across the platforms without compromising the test. Installed on

Re: [patch 0/9] Flattening and initial module rebuilding

On 07/07/2015 05:53 PM, Andrew MacLeod wrote: yes, with a bit of tweaking and enhancement they can be generally useful. They are all in python. And no one is allowed to make comments like "OMG thats so inefficient" or "what a horrible way to do that" :-) My goal was getting things done and so

[PATCH] PR target/66818: Define ATTRIBUTE_ALIGNED_VALUE to 32 for IA MCU

attribute ((aligned)) should align to the minimum of BIGGEST_ALIGNMENT, which is 4 bytes for -miamcu. Tested on Linux/x86-64. OK for trunk? Thanks. H.J. --- gcc/ PR target/66818 * config/i386/i386.h (ATTRIBUTE_ALIGNED_VALUE): Defined to 32 for IA MCU. gcc/testsuite/

  1   2   >