Re: [GSoC] Elimination of CLooG library installation dependency

Richard, could you please review these patches? We would be very glad for your comments. P.S: I’ve attached an improved ChangeLog_entry. 2014-08-15 17:44 GMT+06:00 Richard Biener : > On Fri, Aug 15, 2014 at 1:13 PM, Roman Gareev wrote: >>> I've attached the patch, which should eliminate CLooG

[PATCH] Move caret printing to diagnostics_finalizer

Hi Dodji, This patch is in preparation for further patches moving the Fortran FE to use the common diagnostics machinery. Fortran has its own way of printing the caret information, so we need a way to override the default in the diagnostics machinery. A simple way is to move the printing of the c

Re: [PATCH 204/236] final.c: Use rtx_sequence

On Fri, 2014-08-15 at 18:38 -0400, Trevor Saunders wrote: > On Fri, Aug 15, 2014 at 04:24:49PM -0600, Jeff Law wrote: > > On 08/06/14 11:23, David Malcolm wrote: > > >gcc/ > > > * final.c (get_attr_length_1): Replace GET_CODE check with a > > > dyn_cast, introducing local "seq" and the use of m

[PATCH 4/4] rs6000: Merge andsi3 and anddi3

"AND" is more complex. It was a huge tangled mess, where very often the order of patterns mattered. So I made most patterns mutually exclusive (via their condition), and also made the "S" constraint (for mask64_operand) require TARGET_POWERPC64. There is no reason the various immediate-operand i

[PATCH 3/4] rs6000: Merge boolccsi3 and boolccdi3

2014-08-15 Segher Boessenkool gcc/ * config/rs6000/rs6000.md (*boolccsi3_internal1, *boolccsi3_internal2 and split, *boolccsi3_internal3 and split): Delete. (*boolccdi3_internal1, *boolccdi3_internal2 and split, *boolccdi3_internal3 and split): Delete. (*

[PATCH 2/4] rs6000: Merge boolcsi3 and boolcdi3

2014-08-15 Segher Boessenkool gcc/ * config/rs6000/rs6000.md (*boolcsi3_internal1, *boolcsi3_internal2 and split, *boolcsi3_internal3 and split): Delete. (*boolcdi3_internal1, *boolcdi3_internal2 and split, *boolcdi3_internal3 and split): Delete. (*boolc3

[PATCH 0/4] rs6000: Merge most logical SI and DI patterns

All patches were bootstrapped and regression checked separately, on powerpc64-linux -m64,-m32,-m32/-mpowerpc64; no regressions. Is this okay to apply? Segher gcc/config/rs6000/constraints.md |3 +- gcc/config/rs6000/htm.md |6 +- gcc/config/rs6000/predicates.md | 23 +-

[PATCH 1/4] rs6000: Merge boolsi3 and booldi3

This adds a new output modifier "e" that prints an 's' for things like xoris, and changes "u" to work for both xoris and xori. With that, both SI and DI can simply use an "n" constraint, where previously they needed "K,L" resp. "K,J" (and it used "JF" in fact, but the F doesn't do anything there).

Go patch committed: Initialize frame variable

In the change https://codereview.appspot.com/83750044, committed to the GCC repo in https://gcc.gnu.org/ml/gcc-patches/2014-04/msg00720.html , the initialization of the compiler-generated frame variable was accidentally lost. When the variable happened to be initialized to a non-zero value on the

Re: [PATCH 169/236] Strengthen haifa_sched_info callbacks and 3 scheduler hooks

On Fri, 2014-08-15 at 16:03 -0600, Jeff Law wrote: > On 08/06/14 11:22, David Malcolm wrote: > > gcc/ > > * target.def (reorder): Strengthen param "ready" of this DEFHOOK > > from rtx * to rtx_insn **. > > (reorder2): Likewise. > > (dependencies_evaluation_hook): Strengthen params "

Re: [PATCH 156/236] PHASE 4: Removal of scaffolding

On Thu, 2014-08-14 at 23:30 -0600, Jeff Law wrote: > On 08/06/14 11:22, David Malcolm wrote: > > / > > * rtx-classes-status.txt: Update. > > --- > > rtx-classes-status.txt | 4 ++-- > > 1 file changed, 2 insertions(+), 2 deletions(-) > > > > diff --git a/rtx-classes-status.txt b/rtx-classes-

[SH][committed] Update SH options documentation

Hi, The attached patch updates the SH options documentation. Tested with 'make info dvi pdf'. Committed to trunk, 4.9 and 4.8 branches. Cheers, Oleg gcc/ChangeLog: * doc/invoke.texi (SH options): Document missing processor variant options. Remove references to Hitachi. Undocum

Re: [PATCH 204/236] final.c: Use rtx_sequence

On Fri, Aug 15, 2014 at 04:24:49PM -0600, Jeff Law wrote: > On 08/06/14 11:23, David Malcolm wrote: > >gcc/ > > * final.c (get_attr_length_1): Replace GET_CODE check with a > > dyn_cast, introducing local "seq" and the use of methods of > > rtx_sequence. > > (shorten_branches): Like

libgo patch committed: Don't lose track of m value in GC

The runtime_gc function in libgo invokes the garbage collector proper on the g0 thread (a thread with a large stack that is not involved in scheduling). This is done via runtime_mcall. Upon return from runtime_mcall, the caller may be running on a different thread. Unfortunately, the runtime_gc f

Re: C++ PATCH for c++/61566 (ICE with lambda in template default arg)

On 06/30/2014 02:49 PM, Jason Merrill wrote: decl_mangling_context was failing to recognize a lambda in template context as a lambda. It turns out that was far from the only issue with a lambda in a member template Tested x86_64-pc-linux-gnu, applying to trunk. commit 55ef57277be5885a78

Re: [PATCH 207/236] reorg.c: Use rtx_sequence

On 08/06/14 11:23, David Malcolm wrote: gcc/ * reorg.c (redundant_insn): In two places in the function, replace a check of GET_CODE with a dyn_cast, introducing local "seq", and usings methods of rtx_sequence to clarify the code. Some concerns here with the dynamic cast.

Re: [PATCH 205/236] function.c: Use rtx_sequence

On 08/06/14 11:23, David Malcolm wrote: gcc/ * function.c (contains): Introduce local "seq" for PATTERN (insn), with a checked cast, in the region for where we know it's a SEQUENCE. Use methods of rtx_sequence. OK. As is #206. Jeff

Re: [PATCH 204/236] final.c: Use rtx_sequence

On 08/06/14 11:23, David Malcolm wrote: gcc/ * final.c (get_attr_length_1): Replace GET_CODE check with a dyn_cast, introducing local "seq" and the use of methods of rtx_sequence. (shorten_branches): Likewise, introducing local "body_seq". Strengthen local

Re: [PATCH 194/236] Use rtx_insn for various target.def hooks

On 08/06/14 11:22, David Malcolm wrote: This patch updates params of 22 of the target hooks to pass an rtx_insn * rather than an rtx, as appropriate. Known to compile on: alpha arc arm bfin c6x epiphany ia64 m32c m32r m68k mep microblaze mips pa pdp11 picochip rs6000 s390 sh sparc spu tilegx til

[Patch, Fortran] Fix DECL of namelist I/O function; fix FINALIZATION

This patch fixes two minor issues a) The argument issue mentioned in https://gcc.gnu.org/ml/fortran/2014-08/msg7.html The main issue is that the decl uses "void" as argument; the FE passes IARG() alias gfc_array_index_type while the library expects a GFC_INTEGER_4. As n_dim and ts->kind ar

Re: [PATCH 169/236] Strengthen haifa_sched_info callbacks and 3 scheduler hooks

On 08/06/14 11:22, David Malcolm wrote: gcc/ * target.def (reorder): Strengthen param "ready" of this DEFHOOK from rtx * to rtx_insn **. (reorder2): Likewise. (dependencies_evaluation_hook): Strengthen params "head", "tail" from rtx to rtx_insn *.

[patch, fortran] Fix PR 62142

Hello world, I committed the attached patch as obvious to trunk after regression testing. It fixes the regression in the test case by adding a NULL check to a pointer. Will commit to 4.9 soon. Regards Thomas 2014-08-15 Thomas Koenig PR fortran/62142 * trans-expr.c

Re: [PATCH] Avoid redundant indirect_info computation during inderct edge cloning

> Hi, > > I get a segafult in decl_maybe_in_construction_p during function versioning. > We have following steps in clone creation (e.g. as in > create_version_clone_with_body): > 1. Create function decl > 2. Create clone of cgraph node > 3. Copy function body > After the first step there is

Re: [PATCH] Remove current_function_decl usage from get_polymorphic_call_info

> Hi, > > Testing 'mpx' branch after merge with trunk I got a segfault in ipa-devirt.c. > It appears that cgraph_node cloning with indirect edge causes call to > get_polymorphic_call_info which uses current_function_decl. It happens in > IPA pass and therefore current_function_decl is NULL wh

Re: [wwwdocs] Re: gcc.gnu.org/simtest-howto.html (was: Question for ARM person re asm_fprintf)(

On Fri, 2014-08-15 at 22:58 +0200, Oleg Endo wrote: > On Mon, 2014-08-04 at 08:19 +0200, Oleg Endo wrote: > > > > On Aug 4, 2014, at 6:00 AM, Gerald Pfeifer wrote: > > > > > On Wed, 23 Jul 2014, Hans-Peter Nilsson wrote: > > >> The page is > > >> unfortun

[patch] fix guality/nrv-1.c LTO failure

This test is failing with LTO because in the LTRANS phase (DCE) we realize that the call to f() is useless, so we don't generate it. This leads to an uncalled f() which also gets deleted. We end up with an empty main(), and rightly so, gdb has nothing good to print. Marking `a1' as used keep

Re: [patch, testsuite] Applying non_bionic effective target to particular tests

On Fri, Aug 15, 2014 at 1:05 PM, Alexander Ivchenko wrote: > 2014-08-15 21:32 GMT+04:00 enh : >> can you file bugs against bionic for stuff like this? use >> b.android.com (and feel free to mail me to ensure that they get >> noticed). > > Sure, I will do that. > >> one thing we'd like to do is get

Re: [PATCH] Fix PR62077

On August 15, 2014 7:25:55 PM CEST, Jason Merrill wrote: >On 08/14/2014 12:28 PM, Jason Merrill wrote: >> On 08/14/2014 05:07 AM, Richard Biener wrote: >>> So - can you take over this C++ frontend issue? >> >> OK. > >Here's what I'm applying to trunk: Thanks Jason. Richard.

Re: [patch, testsuite] Applying non_bionic effective target to particular tests

2014-08-15 21:32 GMT+04:00 enh : > can you file bugs against bionic for stuff like this? use > b.android.com (and feel free to mail me to ensure that they get > noticed). Sure, I will do that. > one thing we'd like to do is get to a point where we're building > gcc/gdb et cetera without any local

RE: [PATCH, AArch64] Fix typo

Thanks for the review. -- Evandro Menezes Austin, USA e.mene...@samsung.com +1-512-425-3365 -Original Message- From: gcc-patches-ow...@gcc.gnu.org [mailto:gcc-patches-ow...@gcc.gnu.org] On Behalf Of James Greenhalgh Sent: Friday, August 15, 2

Re: [C++ Patch] PR 57466 (DR 1584)

Hi, On 08/15/2014 09:22 PM, Jason Merrill wrote: On 08/15/2014 03:16 PM, Paolo Carlini wrote: + bool in_function = (TREE_TYPE (tparms) + && DECL_FUNCTION_TEMPLATE_P (TREE_TYPE (tparms))); Huh? There's no such thing as a template parameter of function type. Works fine, in fact,

Re: [C++ Patch] PR 57466 (DR 1584)

On 08/15/2014 03:16 PM, Paolo Carlini wrote: + bool in_function = (TREE_TYPE (tparms) + && DECL_FUNCTION_TEMPLATE_P (TREE_TYPE (tparms))); Huh? There's no such thing as a template parameter of function type. Jason

Re: [C++ Patch] PR 57466 (DR 1584)

... in practice, something like the below. Paolo. /// Index: cp/pt.c === --- cp/pt.c (revision 214027) +++ cp/pt.c (working copy) @@ -162,7 +162,7 @@ static tree tsubst_friend_class (tree, tree); static int

Re: [PATCH][LTO] Streamer re-org (what's left)

> > Yeah, the 2MB was just a "guess", I'll change it to 64k blocks. Note > the original code exponentially increased block size to not have > too many blocks (for whatever reason). A 800MB compressed decl section > would need 12800 64k blocks. But in the end it matters only that > the block all

Re: [PATCH i386 AVX512] [20/n] AVX-512 integer shift pattern.

On Fri, Aug 15, 2014 at 1:56 PM, Kirill Yukhin wrote: > Hello, > This patch extends shift pattern to support AVX-512 > new insn. > > Bootstrapped. > New tests on top of patch-set all pass > under simulator. > > Is it ok for trunk? > > gcc/ > * config/i386/sse.md > (define_mode_iter

Re: [PATCH] Asan static optimization (draft)

On Thu, Aug 14, 2014 at 11:55 PM, Yuri Gribov wrote: > On Thu, Aug 14, 2014 at 8:53 PM, Konstantin Serebryany > wrote: >> In order for your work to be generally useful, I'd ask several things: >> - Update >> https://code.google.com/p/address-sanitizer/wiki/CompileTimeOptimizations >> with exampl

Re: [PATCH i386 AVX512] [19/n] Extends AVX-512 broadcasts.

On Fri, Aug 15, 2014 at 1:52 PM, Kirill Yukhin wrote: > Hello, > This patch introduces new patterns to support > AVX-512Vl,DQ broadcast insns. > > Bootstrapped. > New tests on top of patch-set all pass > under simulator. > > Is it ok for trunk? > > gcc/ > * config/i386/sse.md > (de

Re: [PATCH i386 AVX512] [18/n] Extend vpbroadcastmb2q.

On Fri, Aug 15, 2014 at 1:47 PM, Kirill Yukhin wrote: > Hello, > This patch extends pattern for vpbroadcastmb2q insn > pattern. > > Bootstrapped. > New tests on top of patch-set all pass > under simulator. > > Is it ok for trunk? > > gcc/ > * config/i386/sse.md > (define_mode_itera

Re: [PATCH i386 AVX512] [17/n] Split VI48_AVX512F into VI4_AVX512VL and VI248_AVX512, extend vcvtps2udq,vpbroadcastmb2d.

On Fri, Aug 15, 2014 at 1:42 PM, Kirill Yukhin wrote: > Hello, > This patch splits VI48_AVX512F iterator into two. > It extends vcvtps2udq,vpbroadcastmb2d patterns as well. > > Bootstrapped. > New tests on top of patch-set all pass > under simulator. > > Is it ok for trunk? > > gcc/ > * co

[PATCH,rs6000] Add __VEC_ELEMENT_REG_ORDER__ builtin define for PowerPC

Hi, This adds a macro to indicate the order in which vector elements appear in a register on PowerPC. Elements may appear in right-to-left order for little endian, or in left-to-right order for big endian and when -maltivec=be is selected for little endian. The same macro is being implemented in

Re: [C++ Patch] PR 57466 (DR 1584)

Hi, On 08/15/2014 07:30 PM, Jason Merrill wrote: On 08/15/2014 12:17 PM, Paolo Carlini wrote: By the way, more generally, I don't understand at the moment how we can safely use complete_type in the middle of tsubst & co: it can emit hard errors about, eg, incompleteness (see c++/62072) irrespec

Re: [RFC, PATCH][LRA, MIPS] ICE: in decompose_normal_address, at rtlanal.c:5817

On Fri, Aug 15, 2014 at 5:45 PM, Robert Suchanek wrote: > gcc/ > * rtlanal.c (get_base_term): Accept HIGH as the base term. > > > diff --git gcc/rtlanal.c gcc/rtlanal.c > index 82cfc1bf..2bea2ca 100644 > --- gcc/rtlanal.c > +++ gcc/rtlanal.c > @@ -5624,6 +5624,7 @@ get_base_term (rtx *inner

Re: [PATCH i386 AVX512] [16/n] Add AVX-512BW's psadbw insn.

On Fri, Aug 15, 2014 at 1:37 PM, Kirill Yukhin wrote: > Hello, > This patch introduces AVX-512BW's psadbw insn pattern. > > Bootstrapped. > New tests on top of patch-set all pass > under simulator. > > Is it ok for trunk? > > gcc/ > * config/i386/sse.md > (define_mode_iterator VI8_

Re: [patch, testsuite] Applying non_bionic effective target to particular tests

On Fri, Aug 15, 2014 at 10:56 AM, Mike Stump wrote: > On Aug 15, 2014, at 10:32 AM, enh wrote: >> can you file bugs against bionic for stuff like this? > > I suspect you meant to ask Alexander this question… but just in case you did > intend to ask me, no. I don’t have any visibility into andr

Re: [patch, testsuite] Applying non_bionic effective target to particular tests

On Aug 15, 2014, at 10:32 AM, enh wrote: > can you file bugs against bionic for stuff like this? I suspect you meant to ask Alexander this question… but just in case you did intend to ask me, no. I don’t have any visibility into android local patches or procedures. In the end, people with lo

Re: [patch, testsuite] Applying non_bionic effective target to particular tests

can you file bugs against bionic for stuff like this? use b.android.com (and feel free to mail me to ensure that they get noticed). one thing we'd like to do is get to a point where we're building gcc/gdb et cetera without any local hacks, and when we've got to that point, we're going to have to g

Re: [C++ Patch] PR 57466 (DR 1584)

On 08/15/2014 12:17 PM, Paolo Carlini wrote: By the way, more generally, I don't understand at the moment how we can safely use complete_type in the middle of tsubst & co: it can emit hard errors about, eg, incompleteness (see c++/62072) irrespective of the tsubst_flags_t argument... Yes, it ca

Re: [patch, testsuite] Applying non_bionic effective target to particular tests

On Aug 15, 2014, at 6:49 AM, Alexander Ivchenko wrote: > * lib/target-supports.exp (error_h): New check. > (libc_has_complex_functions): Ditto. > (tgmath_h): Ditto. > * gcc.dg/builtins-59.c: Add libc_has_complex_functions check. > * gcc.dg/builtins-61.c: Likewise. >

Re: [PATCH] Fix PR62077

On 08/14/2014 12:28 PM, Jason Merrill wrote: On 08/14/2014 05:07 AM, Richard Biener wrote: So - can you take over this C++ frontend issue? OK. Here's what I'm applying to trunk: commit 76cce1d62861b6c99e3ecd97bcd607cd242d1efa Author: Jason Merrill Date: Wed Aug 13 13:11:09 2014 -0400

Re: [PATCH, AArch64] Fix typo

On Fri, Aug 15, 2014 at 05:24:58PM +0100, Evandro Menezes wrote: > I tripped at a typo that goes undetected because the macro NAMED_PARAM > doesn't apply in the absence of designated initializers. > > Since struct scale_addr_mode_cost has the cost for DI, but not for QI, the > instances of struct

[Patch, Fortran] Fix CRITICAL handling with -fcoarray=lib

It turned out that the CRITICAL patch had two issues: a) The lock variable was named "__lock_var@0". That's unambiguous but the linker didn't like the file name. As the variable is unused (only the associated token gets used), the assembler error only occurred with -O0 and hence not in the tes

Re: [Patch] PR55189 enable -Wreturn-type by default

On 14/08/2014 20:48, Manuel López-Ibáñez wrote: > --- a/gcc/fortran/options.c > +++ b/gcc/fortran/options.c > @@ -693,6 +693,10 @@ gfc_handle_option (size_t scode, const char *arg, > int value, >gfc_option.warn_line_truncation = value; >break; > > +case OPT_Wmissing_return: > +

Re: [PATCH i386 AVX512] [10/n] Add vector move/load/store.

On Thu, Aug 14, 2014 at 2:51 PM, Kirill Yukhin wrote: > On 14 Aug 13:45, Uros Bizjak wrote: >> Please update the above entry. > Whoops. Updated ChangeLog: > gcc/ > * config/i386/i386.c > (ix86_expand_special_args_builtin): Handle avx512vl_storev8sf_mask, > avx512vl_storev8s

[PATCH, AArch64] Fix typo

I tripped at a typo that goes undetected because the macro NAMED_PARAM doesn't apply in the absence of designated initializers. Since struct scale_addr_mode_cost has the cost for DI, but not for QI, the instances of struct cpu_addrcost_table are not initialized as intended due to the different ord

Re: [C++ Patch] PR 57466 (DR 1584)

.. unfortunately something is wrong with this commit, thus c++/62072. For the time being I'm simply reverting it and adding the new testcase. By the way, more generally, I don't understand at the moment how we can safely use complete_type in the middle of tsubst & co: it can emit hard errors a

[PATCH] Add OPT_Wextra to warning call

I committed the below as obvious in r214026. Since the condition is already controlled by extra_warnings, the only different is that we now print [-Wextra] and that -Werror=extra now works. Cheers, Manuel. Index: gcc/cp/ChangeLog =

Re: RFC: Patch for switch elimination (PR 54742)

On 14/08/14 19:25, Steve Ellcey wrote: On Thu, 2014-08-14 at 10:21 -0600, Jeff Law wrote: On 08/14/14 10:12, David Malcolm wrote: On Thu, 2014-08-14 at 09:56 -0600, Jeff Law wrote: On 08/14/14 04:32, Richard Biener wrote: You'll note in a separate thread Steve and I discussed this during Ca

[patch] libstdc++/62159 add missing headers for freestanding implementation

This seems to fix PR 62159, in that I can now include all the required headers in a C++11 program using a compiler built with --disable-libstdcxx-hosted, but 'make check' doesn't do anything for freestanding builds - should I be doing anything else to test it? commit bfaeb3f86ca436a0e32b1f0b

[RFC, PATCH][LRA, MIPS] ICE: in decompose_normal_address, at rtlanal.c:5817

Hi Vladimir, The following testcase fails when compiled with -O2 -mips32r2: long long a[]; long long b, c, d, k, m, n, o, p, q, r, s, t, u, v, w; int e, f, g, h, i, j, l, x; fn1() { for (; x; x++) if (x & 1) s = h | g; else s = f | e; l = ~0; m = 1 | k; n = i; o = j

[patch] libstdc++/62154 fix throw_with_nested and rethrow_if_nested

Rewritten to meet the C++11 spec, not the 2009 draft I used when adding nested_exception. I've used the builtin traits directly, to avoid making libsupc++ depend on , which shouldn't be necessary because should be present for freestanding implementations, but it isn't (PR 62159). Tested x86_64-

Re: [PATCH, Pointer Bounds Checker 3/x] Target hooks for Pointer Bounds Checker

On 17 Jul 03:36, Jeff Law wrote: > On 04/16/14 05:52, Ilya Enkovich wrote: > >diff --git a/gcc/doc/tm.texi b/gcc/doc/tm.texi > >index b8ca17e..d868129 100644 > >--- a/gcc/doc/tm.texi > >+++ b/gcc/doc/tm.texi > >@@ -4333,6 +4333,13 @@ This hook returns the va_list type of the calling > >convention

Re: [PR c/52952] More precise locations for Wformat

I should have pointed out that this is based on a preliminary patch by Steven attached to PR 52952. I will update the Changelog to reflect this when committing. Cheers, Manuel. On 15 August 2014 15:51, Manuel López-Ibáñez wrote: > Hi, > > This patch moves the location of various Wformat warnin

Re: [PATCH] Fix for PR/62089 (enable missing Asan checks)

On Fri, Aug 15, 2014 at 06:53:25PM +0400, Yury Gribov wrote: > 2014-08-15 Yury Gribov > > gcc/ > PR sanitizer/62089 > * asan.c (instrument_derefs): Fix bitfield check. > > gcc/testsuite/ > PR sanitizer/62089 > * c-c++-common/asan/pr62089.c: New test.

Re: [PATCH] Fix for PR/62089 (enable missing Asan checks)

On 08/14/2014 12:04 PM, Jakub Jelinek wrote: No, this should be if, not else if, and be after the } below. We really can't handle it otherwise. Generally, the bitfield COMPONENT_REFs should have DECL_BIT_FIELD_REPRESENTATIVE which is not a bitfield, therefore the common case will be handled. Ma

Re: [PATCH PR62011]

On Fri, Aug 15, 2014 at 2:26 PM, Yuri Rumyantsev wrote: > Is it important to have correct value for length attribute for Big Cores? > As I new this attribute is used for code layout alignment. > > 2014-08-15 15:54 GMT+04:00 Jakub Jelinek : >> On Fri, Aug 15, 2014 at 03:45:33PM +0400, Yuri Rumyant

[PR c/52952] More precise locations for Wformat

Hi, This patch moves the location of various Wformat warnings from pointing to the first character of the function name (such as printf) to the actual format string. This is specially useful when you have something like printf (cond ? "format 1" : "format 2"); It also moves (some) of the warning

Re: [patch, testsuite] Applying non_bionic effective target to particular tests

Hi Joseph, I seem to address what you've said. Except for I still left non_bionic check for three tests (Namely: builtins-67.c, strlenopt-14g.c, strlenopt-14gf.c) because checks for the presence of mempcpy, stpcpy and rintl (sorry, I didn't mention it last time) seem to be very narrow, I don't t

Re: [PATCH Fortran/Diagnostics] Move Fortran to common diagnostics machinery

Am 06.08.2014 18:50, schrieb Manuel López-Ibáñez: This is the first step for moving Fortran to use the common diagnostics machinery. This patch makes Fortran use the common machinery for those warnings that don't have a location or a controlling option. Bootstrapped and regression tested on x86_6

Re: RFC: Patch for switch elimination (PR 54742)

On 08/15/14 04:07, Richard Biener wrote: On Thu, Aug 14, 2014 at 8:45 PM, Sebastian Pop wrote: Steve Ellcey wrote: I understand the desire not to add optimizations just for benchmarks but we do know other compilers have added this optimization for coremark (See http://community.arm.com/groups/

Re: [PATCH] Put all constants last in tree_swap_operands_p, remove odd -Os check

On 15 August 2014 14:43, Franz Sirl wrote: > Am 15.08.2014 um 11:32 schrieb Manuel López-Ibáñez: >> On 15 August 2014 11:07, Richard Biener wrote: >>> - if (TREE_CODE (arg1) == INTEGER_CST) >>> + if (CONSTANT_CLASS_P (arg1) == INTEGER_CST) >> >> Huh? >> >> /* Nonzero if NODE represents a consta

[PATCH][RFC][match-and-simplify] "Manually" written patterns

The following introduces "manually" written patterns. That is, part of the matching and the transform are fully manual. An example where this is necessary is when the result isn't really an "expression" but a series of statements. For example take simplifications of the memset builtin. With th

Re: [PATCH] Put all constants last in tree_swap_operands_p, remove odd -Os check

Am 15.08.2014 um 11:32 schrieb Manuel López-Ibáñez: > On 15 August 2014 11:07, Richard Biener wrote: >> - if (TREE_CODE (arg1) == INTEGER_CST) >> + if (CONSTANT_CLASS_P (arg1) == INTEGER_CST) > > Huh? > > /* Nonzero if NODE represents a constant. */ > > #define CONSTANT_CLASS_P(NODE)\ >

[COMMITTED] Add myself to MAINTAINERS file (Write After Approval)

Hi, This patch adds myself to the MAINTAINERS file. Commmitted as 214017. Index: MAINTAINERS === --- MAINTAINERS (revision 214014) +++ MAINTAINERS (working copy) @@ -561,6 +561,7 @@ David Ung d

[PATCH][match-and-simplify] Refactor the deprecated APIs

This removes the deprecated APIs and inlines them into their single caller. This exposes publically the 'code_helper' class which will also be needed for manually written simplifiers (that's next). Bootstrapped on x86_64-unknown-linux-gnu, applied. Richard. 2014-08-15 Richard Biener

Re: [PATCH] Warn about unclosed pragma omp declare target.

Ping. On 29 Jul 18:45, Ilya Tocar wrote: > Hi, > > As discussed here in https://gcc.gnu.org/ml/gcc/2014-01/msg00189.html > Gcc should complain about pragma omp declare target without > corresponding pragma omp end declare target. This patch adds a warning > for those cases. > Bootstraps/passes ma

Re: [PATCH PR62011]

Jakub, Is it important to have correct value for length attribute for Big Cores? As I new this attribute is used for code layout alignment. 2014-08-15 15:54 GMT+04:00 Jakub Jelinek : > On Fri, Aug 15, 2014 at 03:45:33PM +0400, Yuri Rumyantsev wrote: >> gcc/ChangeLog >> 2014-08-15 Yuri Rumyantsev

Re: [c++-concepts] explicit instantiation and specialization

Just committed this patch, fixing the bootstrap. 2014-08-13 Andrew Sutton Fix regression in bootstrap. * gcc/cp/call.c (get_temploid): Removed. No longer called. (joust): Remove unused variable declarations. Andrew On Wed, Aug 13, 2014 at 9:50 PM, Andrew Sutton wrote: > Ah... sorry. Le

Re: [PATCH] Extended if-conversion for loops marked with pragma omp simd.

Richard! Here is updated patch with the following changes: 1. Any restrictions on phi-function were eliminated for extended conversion. 2. Put predicate for critical edges to 'aux' field of edge, i.e. negate_predicate was deleted. 3. Deleted splitting of critical edges, i.e. both outgoing edges c

[PATCH i386 AVX512] [20/n] AVX-512 integer shift pattern.

Hello, This patch extends shift pattern to support AVX-512 new insn. Bootstrapped. New tests on top of patch-set all pass under simulator. Is it ok for trunk? gcc/ * config/i386/sse.md (define_mode_iterator VI248_AVX2): Add V32HI mode. (define_insn "3"): Add masking. --

Re: [PATCH PR62011]

On Fri, Aug 15, 2014 at 03:45:33PM +0400, Yuri Rumyantsev wrote: > gcc/ChangeLog > 2014-08-15 Yuri Rumyantsev > > PR target/62011 > * config/i386/i386-protos.h (ix86_avoid_false_dep_for_bm): New function > prototype. > * config/i386/i386.c (ix86_avoid_false_dep_for_bm): New function. > * confi

[PATCH i386 AVX512] [19/n] Extends AVX-512 broadcasts.

Hello, This patch introduces new patterns to support AVX-512Vl,DQ broadcast insns. Bootstrapped. New tests on top of patch-set all pass under simulator. Is it ok for trunk? gcc/ * config/i386/sse.md (define_mode_iterator VI4F_BRCST32x2): New. (define_mode_attr 64x2_mode):

Re: [COMMITTED] Add myself to MAINTAINERS file (Write After Approval)

> > This patch adds myself to the MAINTAINERS file. Commmitted as 214012. > > Please keep this list sorted alphabetically. > Sorry attached wrong version of the patch. Actually commited vesrion (rev 214012), has alphabetical order Index: ChangeLog

[PATCH i386 AVX512] [18/n] Extend vpbroadcastmb2q.

Hello, This patch extends pattern for vpbroadcastmb2q insn pattern. Bootstrapped. New tests on top of patch-set all pass under simulator. Is it ok for trunk? gcc/ * config/i386/sse.md (define_mode_iterator VI8_AVX512VL): New. (define_insn "avx512cd_maskb_vec_dup"): Macroi

Re: [COMMITTED] Add myself to MAINTAINERS file (Write After Approval)

On Fri, Aug 15, 2014 at 01:43:55PM +0200, Richard Biener wrote: > On Fri, Aug 15, 2014 at 12:42 PM, Ilya Tocar wrote: > > Hi, > > > > This patch adds myself to the MAINTAINERS file. Commmitted as 214012. > > Please keep this list sorted alphabetically. Well, Ilya did, just swapped surname and g

Re: [PATCH PR62011]

I checked that zeroing destination operand for unary bit-manipulation instruction is helpful for 64- and 32-bit mode only. So the patch was changed. Is it OK for trunk? gcc/ChangeLog 2014-08-15 Yuri Rumyantsev PR target/62011 * config/i386/i386-protos.h (ix86_avoid_false_dep_for_bm): New func

Re: [GSoC] Elimination of CLooG library installation dependency

On Fri, Aug 15, 2014 at 1:13 PM, Roman Gareev wrote: >> I've attached the patch, which should eliminate CLooG library >> installation dependency from GCC. The CLooG AST generator is still the >> main code generator, but the isl ast generator will be chosen in case >> of nonavailability of CLooG li

Re: [COMMITTED] Add myself to MAINTAINERS file (Write After Approval)

On Fri, Aug 15, 2014 at 12:42 PM, Ilya Tocar wrote: > Hi, > > This patch adds myself to the MAINTAINERS file. Commmitted as 214012. Please keep this list sorted alphabetically. > --- > MAINTAINERS | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/MAINTAINERS b/MAINTAINERS > index 87fb9d

[PATCH i386 AVX512] [17/n] Split VI48_AVX512F into VI4_AVX512VL and VI248_AVX512, extend vcvtps2udq,vpbroadcastmb2d.

Hello, This patch splits VI48_AVX512F iterator into two. It extends vcvtps2udq,vpbroadcastmb2d patterns as well. Bootstrapped. New tests on top of patch-set all pass under simulator. Is it ok for trunk? gcc/ * config/i386/sse.md (define_mode_iterator VI48_AVX512F): Delete.

[PATCH i386 AVX512] [16/n] Add AVX-512BW's psadbw insn.

Hello, This patch introduces AVX-512BW's psadbw insn pattern. Bootstrapped. New tests on top of patch-set all pass under simulator. Is it ok for trunk? gcc/ * config/i386/sse.md (define_mode_iterator VI8_AVX2_AVX512BW): New. (define_insn "_psadbw"): Add evex version. --

Re: [GSoC] Elimination of CLooG library installation dependency

> I've attached the patch, which should eliminate CLooG library > installation dependency from GCC. The CLooG AST generator is still the > main code generator, but the isl ast generator will be chosen in case > of nonavailability of CLooG library. > > However, I've found out a problem. Almost all t

Re: RFC: Patch for switch elimination (PR 54742)

On Fri, Aug 15, 2014 at 12:13 PM, Richard Biener wrote: > On Thu, Aug 14, 2014 at 8:25 PM, Steve Ellcey wrote: >> On Thu, 2014-08-14 at 10:21 -0600, Jeff Law wrote: >>> On 08/14/14 10:12, David Malcolm wrote: >>> > On Thu, 2014-08-14 at 09:56 -0600, Jeff Law wrote: >>> >> On 08/14/14 04:32, Richa

[COMMITTED] Add myself to MAINTAINERS file (Write After Approval)

Hi, This patch adds myself to the MAINTAINERS file. Commmitted as 214012. --- MAINTAINERS | 1 + 1 file changed, 1 insertion(+) diff --git a/MAINTAINERS b/MAINTAINERS index 87fb9dd..a40a537 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -555,6 +555,7 @@ Dinar Temirbulatov

[PATCH] Allow components of allocatables in !$omp atomic (PR fortran/62131)

Hi! As discussed in the PR, nerrs(i,io) isn't allocatable var even when nerrs is allocatable, thus a reasonable reading of the new OpenMP 4.0 requirement is that the testcase is still valid. I went ahead and committed the change suggested by Tobias. For the rest mentioned in the PR, more thought

Re: RFC: Patch for switch elimination (PR 54742)

On Thu, Aug 14, 2014 at 8:25 PM, Steve Ellcey wrote: > On Thu, 2014-08-14 at 10:21 -0600, Jeff Law wrote: >> On 08/14/14 10:12, David Malcolm wrote: >> > On Thu, 2014-08-14 at 09:56 -0600, Jeff Law wrote: >> >> On 08/14/14 04:32, Richard Biener wrote: >> You'll note in a separate thread Steve

Re: RFC: Patch for switch elimination (PR 54742)

On Thu, Aug 14, 2014 at 8:45 PM, Sebastian Pop wrote: > Steve Ellcey wrote: >> I understand the desire not to add optimizations just for benchmarks but >> we do know other compilers have added this optimization for coremark >> (See >> http://community.arm.com/groups/embedded/blog/2013/02/21/corema

Re: [PATCH] Put all constants last in tree_swap_operands_p, remove odd -Os check

On Fri, 15 Aug 2014, Manuel López-Ibáñez wrote: > On 15 August 2014 11:07, Richard Biener wrote: > > - if (TREE_CODE (arg1) == INTEGER_CST) > > + if (CONSTANT_CLASS_P (arg1) == INTEGER_CST) > > Huh? Eh ;) > /* Nonzero if NODE represents a constant. */ > > #define CONSTANT_CLASS_P(NODE)\ >

Re: [PATCH] Put all constants last in tree_swap_operands_p, remove odd -Os check

On 15 August 2014 11:07, Richard Biener wrote: > - if (TREE_CODE (arg1) == INTEGER_CST) > + if (CONSTANT_CLASS_P (arg1) == INTEGER_CST) Huh? /* Nonzero if NODE represents a constant. */ #define CONSTANT_CLASS_P(NODE)\ (TREE_CODE_CLASS (TREE_CODE (NODE)) == tcc_constant) Sadly, we don't

[PATCH] Cleanup STRING_CST handing in PTA

points-to is one place that needs special handling of us allowing &"string" as operands (rather than having a CONST_DECL for each string literal). The following patch realizes that readonly_id is only used for string literals and adjusts code accordingly. Bootstrap and regtest running on x86_64-

[PATCH] Put all constants last in tree_swap_operands_p, remove odd -Os check

The following makes tree_swap_operands_p put all constants 2nd place, also looks through sign-changes when considering further canonicalzations and removes the odd -Os guard for those. That was put in with https://gcc.gnu.org/ml/gcc-patches/2003-10/msg01208.html just motivated by CSiBE numbers -

Re: [PATCH][LTO] Streamer re-org (what's left)

On Fri, 15 Aug 2014, Jan Hubicka wrote: > > > patched: > > > real6m12.437s > > > user51m18.829s > > > sys 4m30.809s > > > > > > WPA is 129s, stream in 29.23s, stream out 12.16s. > > > > > > Patched + fast compression > > > real6m4.383s > > > user49m15.123s > > > sys 4m31.

  1   2   >