Re: Fix PR57268

I'd guess you don't want to flush on DEBUG_INSN_Ps, because then you'd flush differently between -g and -g0. So perhaps something like: yes, If I skipped to flush all DEBUG_INSN_Ps, then dependence lists are the same under -g0 and -g. I bootstrapped Jakub's change on x86_64-linux with no

Re: PR57073 - Optimize __builtin_powif (-1.0, k) to k 1 ? -1.0 : 1.0

On Fri, May 31, 2013 at 5:02 PM, Tobias Burnus bur...@net-b.de wrote: Am 31.05.2013 10:24, schrieb Richard Biener: On Thu, May 30, 2013 at 10:54 PM, Jeff Law l...@redhat.com wrote: Don't worry about it. The patch is good as-is. Why sink the !host_integerp check? Please keep it where it is

Re: [PATCH] Reduce -fopt-info verbosity

On Fri, 31 May 2013, Teresa Johnson wrote: This patch changes the -fopt-info default to optimized instead of all, since the latter is extremely verbose. This reduced the -fopt-info output by over 75% in at least one case, since the vectorizer has many messages under MSG_NOTE (and that should

Re: [PATCH] Improve folding of bitwise ops on booleans

On Fri, May 31, 2013 at 10:18 PM, Jeff Law l...@redhat.com wrote: This is an implementation to fix a missed optimization pointed out to me by Kai. In all these examples, assume a b are single bit types. ~a b -- a b For a signed 1-bit type you'll have values -1, 0 and clearly 0 -1

Re: default_no_named_section bad default

On Sat, Jun 1, 2013 at 12:28 AM, Mike Stump mikest...@comcast.net wrote: On May 31, 2013, at 2:56 PM, Andrew Pinski pins...@gmail.com wrote: This will only fix the GCC source but not other sources which does: void f(void) { __builtin_unreachable(); } Yes. Speaking of which, so how should

Re: PR middle-end/57366

Jan Hubicka hubi...@ucw.cz writes: In any case this is quite self contained fix that is backportable to 4.8 if needed. It should be since it also fixes PR target/47333, a 4.8 regression. Rainer -- - Rainer

Re: [Patch, Fortran] PR57496 use finiteq etc. for __float in write_float.def

Janne has approved the patch on IRC, hence, I committed it as Rev. 199598. Thus, the REAL(16) issue is now solved. However, REAL(10) still fails - and has still to be debugged. See PR. Tobias

Re: Updated MAINTAINERS

2013/6/1 Dinar Temirbulatov di...@kugelworks.com: Hi, I updated MAINTAINERS list with following change(patch attached). thanks, Dinar. Hi, Dinar, You forgot to add an entry in ChangeLog like this: Index: ChangeLog

Minor formatting tweaks in output_constant

Tested on x86_64-suse-linux, applied on the mainline as obvious. 2013-06-03 Eric Botcazou ebotca...@adacore.com * varasm.c (output_constant) CONSTRUCTOR: Minor formatting tweak. STRING_CST: Likewise. VECTOR_CST: Likewise. -- Eric BotcazouIndex: varasm.c

[PATCH,ARM] Fix PR57329 - backport to gcc 4.8

Hello, This patch (trunk r197155) http://gcc.gnu.org/ml/gcc-cvs/2013-03/msg00784.html fixes an ICE in gcc 4.8: http://gcc.gnu.org/bugzilla/show_bug.cgi?id=57329 OK to backport to 4.8 branch? Tested with 4.8 regression test on QEMU, no new regression. BR, Terry

Re: [PATCH] Fix linking with -findirect-dispatch

Jakub Jelinek ja...@redhat.com writes: Both libraries are actually intended to be installed, not just one. Does your patch achieve that? The patch makes sure that the standalone libgcj_bc library is correctly built, instead of trying to install the dummy libgcj_bc library that should only be

RE: [PATCH][gensupport] Add optional attributes field to define_cond_exec

Any comments/suggestions on my implementation of the idea are very welcome. http://gcc.gnu.org/ml/gcc-patches/2013-05/msg01093.html Otherwise, OK for trunk? Ping? Thanks, Kyrill

RE: [PATCH RX] Added target specific macros for macros for RX100, RX200, and RX600

Hi Gerald, Below is the update to mention under changes link on GCC website. RX The port now allows to specify the RX100, RX200, and RX600 processors with the command line options -mcpu=rx100, -mcpu=rx200 and -mcpu=rx600 This update will go into: New Targets and Target Specific

Re: [x86, PATCH 2/2] Enabling of the new Intel microarchitecture Silvermont

Hi, Jakub! Thanks a lot for you remarks! Attached is the patch which fixes all of them. Ok to install if all testing passes? Changelog: gcc/ 2013-06-03 Yuri Rumyantsev yuri.s.rumyant...@intel.com * config/i386/i386.c (ix86_lea_outperforms): Fix formatting.

Re: [x86, PATCH 2/2] Enabling of the new Intel microarchitecture Silvermont

On Mon, Jun 03, 2013 at 05:00:38PM +0400, Yuri Rumyantsev wrote: Thanks a lot for you remarks! Attached is the patch which fixes all of them. Ok to install if all testing passes? Changelog: gcc/ 2013-06-03 Yuri Rumyantsev yuri.s.rumyant...@intel.com Two spaces before name, not

Re: [Patch libsanitizer] merge rev 182922 (helps running under qemu)

Hi, Just committed as r199606. Thanks Christophe. On 31 May 2013 16:53, Konstantin Serebryany konstantin.s.serebry...@gmail.com wrote: On Fri, May 31, 2013 at 6:49 PM, Jack Howarth howa...@bromo.med.uc.edu wrote: On Fri, May 31, 2013 at 04:42:21PM +0200, Christophe Lyon wrote: Hi, I'd

[Patch, Fortran] PR57508 - Fix ICE/Reject-valid issue with get_temp_from_expr (intrinsic assignment with defined assignment)

Dear all, Due to copying the attributes, the temporary variable could get marked as function (attr.function, attr.flavor == FL_PROCEDURE). This either lead to leaking those attributes into the assembler file - or to cause an error due to the call to gfc_add_flavor. With this patch, I now

symtab cleanups 3/17: fix alias visibility logic

Hi, GCC's alias support follows how C level alias attributes work: function bodies/variable initializers are associated with particular symbols and other aliases are separate symbol referring to the one they are aliasing. In ELF implementation and by my understanding of BFD also everywher else

[gomp4] Array section C++ parsing

Hi! This patch parses array sections in map/to/from/depend clauses and provides diagnostics (+ testcases for that) for it. Depend clause right now is just ignored during omp lowering, and for map/to/from clause we would ICE during omp expansion, which isn't done yet, but as the testcases test

Re: [PATCH] Improve folding of bitwise ops on booleans

Hmm, it isn't necessary to restrict this optimization AFAICS. We have just two cases. (~X Y) - X Y (X ~Y) - Y X (~X | Y) - X = Y (X | ~Y) - Y = X is true for unsigned 1-bit types. For signed 1-bit types we need to invert logic here as following: (~X Y) - X Y (X ~Y) - Y X (~X | Y) - X

Re: [PATCH, libcpp] Do not decrease highest_location if the included file has be included twice.

Hello Dehao, Dehao Chen de...@google.com a écrit: This patch fixes the bug that when include a header file, if the header file is already included (with #define _HEADER_H_), libcpp should not decrease its highest_location, otherwise it'll cause incorrect source location when source location

Re: [PATCH RX] Added target specific macros for macros for RX100, RX200, and RX600

2013/6/3 Sandeep Kumar Singh sandeep.sin...@kpitcummins.com: Hi Gerald, Below is the update to mention under changes link on GCC website. RX The port now allows to specify the RX100, RX200, and RX600 processors with the command line options -mcpu=rx100, -mcpu=rx200 and

Re: PING^2 [libitm,PATCH] Fix bootstrap due to __always_inline in libitm

On 06/01/2013 03:44 PM, Gerald Pfeifer wrote: 2013-05-31 Gerald Pfeifer ger...@pfeifer.com PR bootstrap/56714 * local_atomic (__always_inline): Always define our version. (__calculate_memory_order): Mark inline. (atomic_thread_fence): Ditto.

Re: [PATCH] Disable no-section-anchors-vect-68.C for aarch64 tiny memory model.

On 31/05/13 12:52, Marcus Shawcroft wrote: The vect/no-section-anchors-vect-68.C test case results in a binary that is tool large for the aarch64 tiny memory model. This patch disables the test case for that memory model. /Marcus 2013-05-31 Marcus Shawcroft marcus.shawcr...@arm.com

Re: [x86, PATCH 2/2] Enabling of the new Intel microarchitecture Silvermont

Thanks, Jakub! Updated Changelog: gcc/ 2013-06-03 Yuri Rumyantsev yuri.s.rumyant...@intel.com * config/i386/i386.c (ix86_lea_outperforms): Fix formatting. (ix86_avoid_lea_for_addr): Likewise. (exact_dependency_1): Likewise. (ix86_adjust_cost): Likewise.

[C++ Patch] PR 57419

Hi, in this SFINAE issue, finish_qualified_id_expr is called with complain == 0 and calls mark_used, which has naked error calls. The straightforward fix works well. I'm also taking the occasion to change an error to inform (then some testcases must be adjusted). Tested x86_64-linux.

RE: [AArch64] Fix printf format warning in aarch64_print_operand

--- gcc/ 2013-04-22 James Greenhalgh james.greenha...@arm.com * config/aarch64/aarch64.c (aarch64_print_operand): Fix asm_fprintf format specifier in 'X' case. OK. R. Hi, This warning also occurs on gcc-4_8-branch, can I also backport the fix to there now that

Re: [AArch64] Fix printf format warning in aarch64_print_operand

On 03/06/13 16:17, James Greenhalgh wrote: --- gcc/ 2013-04-22 James Greenhalgh james.greenha...@arm.com * config/aarch64/aarch64.c (aarch64_print_operand): Fix asm_fprintf format specifier in 'X' case. OK. R. Hi, This warning also occurs on gcc-4_8-branch, can I

Re: [x86, PATCH 2/2] Enabling of the new Intel microarchitecture Silvermont

On Mon, Jun 3, 2013 at 4:43 PM, Yuri Rumyantsev ysrum...@gmail.com wrote: Updated Changelog: gcc/ 2013-06-03 Yuri Rumyantsev yuri.s.rumyant...@intel.com * config/i386/i386.c (ix86_lea_outperforms): Fix formatting. (ix86_avoid_lea_for_addr): Likewise.

Re: [GOOGLE] Unrestrict early inline restrictions for AutoFDO

The performance testing is ok. But it does not solve the problem: for some recursive function calls, the size growth is calculated as 0. So I think we may want to just fall back to the 4.7 to limit the iterations to 10 for AutoFDO enabled build? Dehao On Sun, Jun 2, 2013 at 9:36 PM, Xinliang

[PATCH v2 1/6] rs6000: Introducing define_dot_insn

This adds a program (mdm.pl) that does some processing on machine description files. It is careful to keep formatting and comments intact as much as possible, so that its output is well readable. The only transform it does so far is convert a define_dot_insn construct to three define_insns and

[PATCH v2 2/6] rs6000: dot for lshrsi3

This moves lshrsi3 over and merges it with lshrdi3. The immediate version is split off since it needs a different condition (and is a separate instruction anyway). Tested as per usual; okay? 2013-06-03 Segher Boessenkool seg...@kernel.crashing.org gcc/ * config/rs6000/rs6000.md

[PATCH v2 0/6] define_dot_insn

David alerted me to the proper way to ensure generated files will not be regenerated for end-users. I also cleaned up some whitespace/comment churn; and there was an embarrassing bug in the dot condition for srwdi, for -m32 -mpowerpc64. So, v2. Bootstrapped and tested these six together, no

[PATCH v2 4/6] rs6000: dot for var_shift_rotate/var_delayed_compare

This changes all scheduling descriptions to treat insn type var_shift_rotate with dot=yes the same as var_delayed_compare is treated, and to require dot=no for var_shift_rotate that already was handled. 2013-06-03 Segher Boessenkool seg...@kernel.crashing.org gcc/ *

[PATCH v2 3/6] dot for ashlmode3

Similar to lshr, for ashl. Okay? 2013-06-03 Segher Boessenkool seg...@kernel.crashing.org gcc/ * config/rs6000/rs6000.md (ashlsi3, ashldi3_internal1, ashldi3_internal2, ashldi3_internal3): Delete. * config/rs6000/integer.mdm (ashlmode3, ashlmode3_imm): New. *

[PATCH v2 5/6] rs6000: dot for shift/delayed_compare

Similar, but now for shift and delayed_compare. 2013-06-03 Segher Boessenkool seg...@kernel.crashing.org gcc/ * config/rs6000/40x.md: Require dot=no for type=shift; where type=delayed_compare, also handle type=shift dot=yes. * config/rs6000/440.md: Ditto. *

[PATCH v2 6/6] rs6000: dot for ashiftrt

Last for now: move ashiftrt to integer.mdm. 2013-06-03 Segher Boessenkool seg...@kernel.crashing.org gcc/ * config/rs6000/rs6000.md (ashrsi3, ashrdi3_no_power, ashrdisi3_noppc64be, ashrdi3, ashrdi3_internal1, ashrdi3_internal2, ashrdi3_internal3): Delete.

[AArch64, PATCH 1/5] Improve MOVI handling (Change interface of aarch64_simd_valid_immediate)

(This patch is the first of five, where the first 4 do some clean-up and the last fixes a bug with scalar MOVI. The bug fix without the clean-up was particularly ugly!) This one is pretty simple - just altering an interface, so we can later remove an unnecessary wrapper function. OK for

[AArch64, PATCH 3/5] Improve MOVI handling (Don't update RTX operand in-place)

(This patch is the third of five, where the first 4 do some clean-up and the last fixes a bug with scalar MOVI. The bug fix without the clean-up was particularly ugly!) This one is focused on cleaning up aarch64_simd_valid_immediate, with better use of arguments and no in-place modification of

[AArch64, PATCH 2/5] Improve MOVI handling (Remove wrapper function)

(This patch is the second of five, where the first 4 do some clean-up and the last fixes a bug with scalar MOVI. The bug fix without the clean-up was particularly ugly!) This one is also very simple - removing a wrapper function that was an unnecessary level of indirection. OK for trunk?

[AArch64, PATCH 4/5] Improve MOVI handling (Other minor clean-up)

(This patch is the fourth of five, where the first 4 do some clean-up and the last fixes a bug with scalar MOVI. The bug fix without the clean-up was particularly ugly!) I think the changelog says it all here. Nothing major, just tidying up. OK for trunk? Cheers, Ian 2013-06-03 Ian

[AArch64, PATCH 5/5] Improve MOVI handling (Fix invalid assembler bug)

(This patch is the last of five, where the first 4 did some clean-up and this one fixes a bug with scalar MOVI. The bug fix without the clean-up was particularly ugly!) GCC will currently generator invalid assembler for MOVI, if the value in question needs to be shifted. For example:

Re: [PATCH] Improve folding of bitwise ops on booleans

2013/6/3 Jeff Law l...@redhat.com: The counter to that argument of leaving it to the backend to recover the and-not for is that the backend doesn't typically see these as single bit operations which makes turning the relational back into an and-not sequence considerably more difficult. Do we

Re: [PATCH] Improve folding of bitwise ops on booleans

On 06/03/2013 02:24 AM, Richard Biener wrote: On Fri, May 31, 2013 at 10:18 PM, Jeff Law l...@redhat.com wrote: This is an implementation to fix a missed optimization pointed out to me by Kai. In all these examples, assume a b are single bit types. ~a b -- a b For a signed 1-bit type

Re: [PATCH v2 0/6] define_dot_insn

On 06/03/2013 09:26 AM, Segher Boessenkool wrote: David alerted me to the proper way to ensure generated files will not be regenerated for end-users. I also cleaned up some whitespace/comment churn; and there was an embarrassing bug in the dot condition for srwdi, for -m32 -mpowerpc64. So,

Re: [PATCH, libcpp] Do not decrease highest_location if the included file has be included twice.

Hi, Dodji, Thanks for looking into this. The reason a test is missing is that it would need a super large source file to reproduce the problem. However, if you apply the attached patch, you can reproduce the problem with attached testcase: g++ a.cpp -g -S -c -o a.s in a.s, the linenos are

Re: [PATCH] Improve folding of bitwise ops on booleans

On 06/03/2013 11:00 AM, Richard Henderson wrote: On 06/03/2013 09:37 AM, Kai Tietz wrote: foo: .seh_endprologue cmpb%cl, %dl seta%al ret .seh_endproc .p2align 4,,15 .globl boo .defboo;.scl2; .type

Re: [PATCH] Improve folding of bitwise ops on booleans

On 06/03/2013 09:37 AM, Kai Tietz wrote: foo: .seh_endprologue cmpb%cl, %dl seta%al ret .seh_endproc .p2align 4,,15 .globl boo .defboo;.scl2; .type 32; .endef .seh_proc boo boo:

[Fortran-Dev] Merge trunk into the branch

I updated the branch to the trunk. Committed as 199612. Tobias

[PATCH] Handle MIPS EVA

Hi Richard, This is the gcc patch to handle the -meva option to gcc. Okay? Thanks, Catherine 2013-06-03 Catherine Moore c...@codesourcery.com gcc/ * config/mips/mips.opt (meva): New. * config/mips/mips.h (TARGET_CPU_CPP_BUILTINS): Define __mips_eva.

Re: Fix PR57268

On 06/01/2013 06:55 AM, Jakub Jelinek wrote: On Sat, Jun 01, 2013 at 10:11:24AM +0200, Jakub Jelinek wrote: On Sat, Jun 01, 2013 at 08:39:58AM +0400, Dinar Temirbulatov wrote: I am investigating the problem. I'd guess you don't want to flush on DEBUG_INSN_Ps, because then you'd flush

Re: [PATCH] Basic support for MIPS r5900

On 06/02/2013 01:45 PM, Jürgen Urban wrote: Hello, after some months I reworked the patch for r5900. It would be nice if this could be accepted. The patch contains only changes to get basic support for MIPS r5900. It can be used to compile a working Linux kernel for the Playstation 2. It is

[patch, testsuite, cilk] Fix cilk tests for simulators

A number of the new cilk tests in gcc/testsuite/c-c++-common/cilk-plus/AN fail for me when run via the gnu simulator on mips. The problem is that the gnu simulator does not set up argc and argv. After asking in the gdb mailing list I believe this is an issue for multiple simulators on multiple

Re: [C++ Patch] PR 57419

Now that we're using C++, I'm inclined to give the sfinae version the same name. Jason

Re: [PATCH][gensupport] Add optional attributes field to define_cond_exec

On 06/03/2013 04:06 AM, Kyrylo Tkachov wrote: Any comments/suggestions on my implementation of the idea are very welcome. http://gcc.gnu.org/ml/gcc-patches/2013-05/msg01093.html Otherwise, OK for trunk? Ping? Ok. If we ever get serious about re-implementing define_cond_exec with more

Re: [C++ Patch] PR 57419

On 06/03/2013 07:55 PM, Jason Merrill wrote: Now that we're using C++, I'm inclined to give the sfinae version the same name. Indeed. I'm finishing testing the below, then. Thanks, Paolo. /// /gcc/cp 2013-06-03 Paolo Carlini paolo.carl...@oracle.com PR c++/57419

Re: [PATCH] Handle MIPS EVA

Moore, Catherine catherine_mo...@mentor.com writes: Index: config/mips/mips.opt === --- config/mips/mips.opt (revision 199610) +++ config/mips/mips.opt (working copy) @@ -141,6 +141,10 @@ membedded-data Target Report

Re: [patch, testsuite, cilk] Fix cilk tests for simulators

On 06/03/2013 11:49 AM, Steve Ellcey wrote: A number of the new cilk tests in gcc/testsuite/c-c++-common/cilk-plus/AN fail for me when run via the gnu simulator on mips. The problem is that the gnu simulator does not set up argc and argv. After asking in the gdb mailing list I believe this is

Re: [patch, testsuite, cilk] Fix cilk tests for simulators

On Mon, Jun 03, 2013 at 12:27:15PM -0600, Jeff Law wrote: On 06/03/2013 11:49 AM, Steve Ellcey wrote: A number of the new cilk tests in gcc/testsuite/c-c++-common/cilk-plus/AN fail for me when run via the gnu simulator on mips. The problem is that the gnu simulator does not set up argc and

Re: [PATCH] Improve folding of bitwise ops on booleans

2013/6/3 Jeff Law l...@redhat.com: On 06/03/2013 11:00 AM, Richard Henderson wrote: On 06/03/2013 09:37 AM, Kai Tietz wrote: foo: .seh_endprologue cmpb%cl, %dl seta%al ret .seh_endproc .p2align 4,,15 .globl boo

Re: [PATCH] Reduce -fopt-info verbosity

On Mon, Jun 3, 2013 at 1:05 AM, Richard Biener rguent...@suse.de wrote: On Fri, 31 May 2013, Teresa Johnson wrote: This patch changes the -fopt-info default to optimized instead of all, since the latter is extremely verbose. This reduced the -fopt-info output by over 75% in at least one case,

Re: [PATCH] Improve folding of bitwise ops on booleans

On 06/03/2013 12:32 PM, Kai Tietz wrote: 2013/6/3 Jeff Law l...@redhat.com: On 06/03/2013 11:00 AM, Richard Henderson wrote: On 06/03/2013 09:37 AM, Kai Tietz wrote: foo: .seh_endprologue cmpb%cl, %dl seta%al ret .seh_endproc

Re: [PATCH, libcpp] Do not decrease highest_location if the included file has be included twice.

Dehao Chen de...@google.com a écrit: Thanks for looking into this. You are welcome. Libcpp is fun. Is it not? :-) The reason a test is missing is that it would need a super large source file to reproduce the problem. I see. It's kind of a pity that we cannot have tests for interesting

Re: [PATCH] pr57457

On 05/31/2013 12:01 PM, Iyer, Balaji V wrote: -Original Message- From: gcc-patches-ow...@gcc.gnu.org [mailto:gcc-patches- ow...@gcc.gnu.org] On Behalf Of Jeff Law Sent: Friday, May 31, 2013 11:50 AM To: Iyer, Balaji V Cc: gcc-patches@gcc.gnu.org; Steve Ellcey Subject: Re: [PATCH]

Remove redundant target_cpu_default settings

Some (but not all) 64-bit MIPS configurations were setting target_cpu_default to MASK_64BIT|MASK_FLOAT64. This has been redundant for a long time. The masks are now derived from the ABI and ISA settings, and also aren't tested directly in preprocessor conditions (such as those used when setting

RE: [patch, testsuite, cilk] Fix cilk tests for simulators

-Original Message- From: Jeff Law [mailto:l...@redhat.com] Sent: Monday, June 03, 2013 2:27 PM To: Steve Ellcey Cc: Iyer, Balaji V; gcc-patches@gcc.gnu.org Subject: Re: [patch, testsuite, cilk] Fix cilk tests for simulators On 06/03/2013 11:49 AM, Steve Ellcey wrote: A

Re: [PATCH] Improve folding of bitwise ops on booleans

On Mon, Jun 03, 2013 at 10:00:19AM -0700, Richard Henderson wrote: On 06/03/2013 09:37 AM, Kai Tietz wrote: foo: .seh_endprologue cmpb%cl, %dl seta%al ret .seh_endproc .p2align 4,,15 .globl boo .defboo;

Re: [patch, testsuite, cilk] Fix cilk tests for simulators

On 06/03/2013 01:27 PM, Iyer, Balaji V wrote: I am OK with Steve's changes in most cases. In a few cases, I am using it as a parameter to pass into tests. On a top-level, the main reason why I used argc, and argv is that, I want to make sure the compiler will never do things like constant

Re: [PATCH] fix for pr 57474

On 05/31/2013 06:28 PM, Iyer, Balaji V wrote: Hello Everyone, PR reports that sec_implicit2 and sec_implicit regression tests were failing in darwin. I looked into it and it is due to an uninitialized variable (rhs_length). This patch pasted below should fix that issue. Is this OK for

Re: default_no_named_section bad default

On Jun 3, 2013, at 1:27 AM, Richard Biener richard.guent...@gmail.com wrote: Yes. Speaking of which, so how should this be handled? Imagine we have asm(# no bytes) before the unreachable. The compiler can't know the size (though, the linker can), and yet, a good solution handles this as

Re: [PATCH] Basic support for MIPS r5900

Jürgen Urban juergenur...@gmx.de writes: after some months I reworked the patch for r5900. It would be nice if this could be accepted. The patch contains only changes to get basic support for MIPS r5900. It can be used to compile a working Linux kernel for the Playstation 2. It is also

Re: [C++ Patch] PR 57419

OK. Jason

Re: [PATCH v2 0/6] define_dot_insn

Well, my first comment is that this is more or less exactly what we invented define_subst to attempt to handle. Could we have more constructive feedback on that mechanism rather than inventing a new mechanism private to the rs6000 backend? Hi Richard, I don't think this can be done using

Re: [PATCH] Expanding array notations inside conditions

On 05/30/2013 10:00 PM, Iyer, Balaji V wrote: Hello Everyone, When I was looking at the erroneous test on PR 57452, I found out that array notations in conditions were not expanded correctly. The rank for the array notation condition must be same (or equal to zero) as the rank of the

RE: [PATCH] fix for pr 57474

-Original Message- From: gcc-patches-ow...@gcc.gnu.org [mailto:gcc-patches- ow...@gcc.gnu.org] On Behalf Of Jeff Law Sent: Monday, June 03, 2013 3:51 PM To: Iyer, Balaji V Cc: gcc-patches@gcc.gnu.org; domi...@lps.ens.fr Subject: Re: [PATCH] fix for pr 57474 On 05/31/2013 06:28

Re: [PATCH] fix for pr 57474

On 06/03/2013 02:34 PM, Iyer, Balaji V wrote: -Original Message- From: gcc-patches-ow...@gcc.gnu.org [mailto:gcc-patches- ow...@gcc.gnu.org] On Behalf Of Jeff Law Sent: Monday, June 03, 2013 3:51 PM To: Iyer, Balaji V Cc: gcc-patches@gcc.gnu.org; domi...@lps.ens.fr Subject: Re:

Re: [PATCH v2 0/6] define_dot_insn

On Jun 3, 2013, at 1:16 PM, Segher Boessenkool seg...@kernel.crashing.org wrote: define_subst is not flexible enough for our purpose by far I had Kenny review define_subst when it went in, though it is a step in the right direction, it misses out on a lot completeness that a port maintainer

Re: [ada, build] host/target configuration

On Jun 2, 2013, at 21:18 , Alexandre Oliva aol...@redhat.com wrote: - Use target_alias explicitly just at the points where we know that we need to depart from the canonical name I suggest another approach: if there are significant differences between the run-time systems, they ought to be

Re: [PATCH] Handle MIPS EVA

On Mon, 3 Jun 2013, Richard Sandiford wrote: Index: config/mips/mips.opt === --- config/mips/mips.opt(revision 199610) +++ config/mips/mips.opt(working copy) @@ -141,6 +141,10 @@ membedded-data Target Report

C++ PATCH to avoid false positives with -Wabi-tag

It occurred to me that -Wabi-tag would probably warn about a template instantiated with a class that has an ABI tag, and indeed it does. This is unnecessary; since the template argument is part of the signature of the instantiation, we don't need to worry about the tag being hidden when the

Re: [patch, testsuite, cilk] Fix cilk tests for simulators

On Mon, 2013-06-03 at 13:47 -0600, Jeff Law wrote: On 06/03/2013 01:27 PM, Iyer, Balaji V wrote: I am OK with Steve's changes in most cases. In a few cases, I am using it as a parameter to pass into tests. On a top-level, the main reason why I used argc, and argv is that, I want to make

RE: [patch, testsuite, cilk] Fix cilk tests for simulators

-Original Message- From: Steve Ellcey [mailto:sell...@mips.com] Sent: Monday, June 03, 2013 6:31 PM To: Jeff Law Cc: Iyer, Balaji V; gcc-patches@gcc.gnu.org Subject: Re: [patch, testsuite, cilk] Fix cilk tests for simulators On Mon, 2013-06-03 at 13:47 -0600, Jeff Law wrote:

Re: [c++-concepts]

On 06/01/2013 10:57 AM, Gabriel Dos Reis wrote: Well, we don't want to check cxx0x one place and cxx11 others. But we still do. I'd rather fix that all together in a separate patch than mess with it in this context. Jason

Re: [PATCH, libcpp] Do not decrease highest_location if the included file has be included twice.

On Mon, Jun 3, 2013 at 11:46 AM, Dodji Seketeli do...@seketeli.org wrote: Dehao Chen de...@google.com a écrit: Thanks for looking into this. You are welcome. Libcpp is fun. Is it not? :-) It truly is ;-) The reason a test is missing is that it would need a super large source file to

RE: [PATCH] Handle MIPS EVA

-Original Message- From: Richard Sandiford [mailto:rdsandif...@googlemail.com] Sent: Monday, June 03, 2013 2:28 PM To: Moore, Catherine Cc: gcc-patches@gcc.gnu.org Subject: Re: [PATCH] Handle MIPS EVA Moore, Catherine catherine_mo...@mentor.com writes: Index:

Re: [PATCH] Basic support for MIPS r5900

Hello Richard, Thanks, looks good. The comments I have are only minor and seemed easier to spell out as a revised patch, attached below. The changes are: * removing the config.sub bit, which looked redundant. We already have the up-to-date upstream config.sub. * removing the

[PATCH] libffi documentation fix

This patch from libffi revision 675c9839224 allows libffi.pdf to be generated without tex related errors. I tested it on gcc trunk. Please backport it to gcc. The original patch can be found here: http://sourceware.org/ml/libffi-discuss/2013/msg00086.html Cesar 2013-06-03 Andreas Schwab

Added build_c_cast to c-family?

Hello Everyone, Is it OK to move build_c_cast prototype into c-common.h? The reason for this is that, I would like to share some of the code between array notation for C and C++ and this function is sort of required for both places. Also, the exact same call is available for both C and

Re: [GOOGLE] More strict checking for call args

Hi, This patch was committed to google branch. But I think it is of general interest. So is it ok for trunk? Thanks, Dehao gcc/ChangeLog: 2013-06-03 Dehao Chen de...@google.com *gimple-low.c (gimple_check_call_args): Restrict the call_arg check to contain same number of args. Index: