"Moore, Catherine" writes:
> @@ -16376,6 +16377,12 @@ Use (do not use) MT Multithreading instructions.
> @opindex mno-mcu
> Use (do not use) the MIPS MCU ASE instructions.
>
> +@item -meva
> +@itemx -mno-eva
> +@opindex meva
> +@opindex mno-eva
> +Use (do not use) the MIPS EVA instructions.
P
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
*gimple-low.c (gimple_check_call_args): Restrict the call_arg check to
contain same number of args.
Index: gcc/gimple-low.c
==
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 C
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
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 tar
> -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" writes:
> > Index: config/mips/mips.opt
> >
> =
On Mon, Jun 3, 2013 at 11:46 AM, Dodji Seketeli wrote:
> Dehao Chen 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 reproduce the problem.
>
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
> -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 wro
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 t
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 ty
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
On Jun 2, 2013, at 21:18 , Alexandre Oliva 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 preserved
On Jun 3, 2013, at 1:16 PM, Segher Boessenkool
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
might wish it had. For me
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: [PATCH]
> -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/201
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
a
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 defin
OK.
Jason
"Jürgen Urban" 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 possible to get Linux
On Jun 3, 2013, at 1:27 AM, Richard Biener 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 well.
>> Hopefully
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 propag
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
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
> >
> -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:
> >
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
up
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] pr574
Dehao Chen 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
cases like this
On 06/03/2013 12:32 PM, Kai Tietz wrote:
2013/6/3 Jeff Law :
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,,1
On Mon, Jun 3, 2013 at 1:05 AM, Richard Biener 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, since the
2013/6/3 Jeff Law :
> 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
>>>
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 a
"Moore, Catherine" 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 Var(TARGET_EMBEDDED
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
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
PR c++/57419
* decl2.c (mark_used
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 wi
Now that we're using C++, I'm inclined to give the sfinae version the
same name.
Jason
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
p
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
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
differen
I updated the branch to the trunk. Committed as 199612.
Tobias
Hi Richard,
This is the gcc patch to handle the -meva option to gcc. Okay?
Thanks,
Catherine
2013-06-03 Catherine Moore
gcc/
* config/mips/mips.opt (meva): New.
* config/mips/mips.h (TARGET_CPU_CPP_BUILTINS):
Define __mips_eva.
(ASM_SPEC): Handle -meva.
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
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
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 off-b
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 sri, for -m32 -mpowerpc64. So,
2013/6/3 Jeff Law :
> 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 consider
(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:
prog.s:
(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 Bolto
(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 R
(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?
Chee
(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 trunk?
Last for now: move ashiftrt to integer.mdm.
2013-06-03 Segher Boessenkool
gcc/
* config/rs6000/rs6000.md (ashrsi3, ashrdi3_no_power,
ashrdisi3_noppc64be, ashrdi3, ashrdi3_internal1, ashrdi3_internal2,
ashrdi3_internal3): Delete.
(ashr3, ashr3_imm): New.
Similar, but now for shift and delayed_compare.
2013-06-03 Segher Boessenkool
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.
* config/rs6000/476.md: Ditto.
Similar to lshr, for ashl. Okay?
2013-06-03 Segher Boessenkool
gcc/
* config/rs6000/rs6000.md (ashlsi3, ashldi3_internal1,
ashldi3_internal2, ashldi3_internal3): Delete.
* config/rs6000/integer.mdm (ashl3, ashl3_imm): New.
* config/rs6000/integer.md: Regenerat
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
gcc/
* config/rs6000/40x.md: Require dot=no for type
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
gcc/
* config/rs6000/rs6000.md (lshrsi3): Delete.
* co
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_insn"s an
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 sri, for -m32 -mpowerpc64. So, v2.
Bootstrapped and tested these six together, no regres
On 06/03/2013 02:24 AM, Richard Biener wrote:
On Fri, May 31, 2013 at 10:18 PM, Jeff Law 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 v
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 Davi
On Mon, Jun 3, 2013 at 4:43 PM, Yuri Rumyantsev wrote:
> Updated Changelog:
>
> gcc/
>
> 2013-06-03 Yuri Rumyantsev
>
> * config/i386/i386.c (ix86_lea_outperforms): Fix formatting.
> (ix86_avoid_lea_for_addr): Likewise.
> (exact_dependency_1): Likewise.
> (ix86_
On 03/06/13 16:17, James Greenhalgh wrote:
---
gcc/
2013-04-22 James Greenhalgh
* 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 the
> > ---
> > gcc/
> >
> > 2013-04-22 James Greenhalgh
> >
> > * 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 4.8 is
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.
Thanks
Thanks, Jakub!
Updated Changelog:
gcc/
2013-06-03 Yuri Rumyantsev
* config/i386/i386.c (ix86_lea_outperforms): Fix formatting.
(ix86_avoid_lea_for_addr): Likewise.
(exact_dependency_1): Likewise.
(ix86_adjust_cost): Likewise.
(swap_top_of_ready_list):
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
* gcc.dg/vect/no-section
On 06/01/2013 03:44 PM, Gerald Pfeifer wrote:
> 2013-05-31 Gerald Pfeifer
>
> PR bootstrap/56714
> * local_atomic (__always_inline): Always define our version.
> (__calculate_memory_order): Mark inline.
> (atomic_thread_fence): Ditto.
> (atomic_signal_fence): Ditto
2013/6/3 Sandeep Kumar Singh :
> 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 updat
Hello Dehao,
Dehao Chen 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 numbers are l
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 >
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 err
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
the
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
expl
Hi,
Just committed as r199606.
Thanks
Christophe.
On 31 May 2013 16:53, Konstantin Serebryany
wrote:
> On Fri, May 31, 2013 at 6:49 PM, Jack Howarth
> wrote:
>> On Fri, May 31, 2013 at 04:42:21PM +0200, Christophe Lyon wrote:
>>> Hi,
>>>
>>> I'd like to backport libsanitizer commit #182922:
>
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
Two spaces before name, not just one.
>
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
* config/i386/i386.c (ix86_lea_outperforms): Fix formatting.
(ix86_avoid_lea_for_addr): Likewise.
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
> 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
Jakub Jelinek 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 used for testing
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
Tested on x86_64-suse-linux, applied on the mainline as obvious.
2013-06-03 Eric Botcazou
* varasm.c (output_constant) : Minor formatting tweak.
: Likewise.
: Likewise.
--
Eric BotcazouIndex: varasm.c
=
2013/6/1 Dinar Temirbulatov :
> 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
===
--- Change
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
Jan Hubicka 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 Orth, Center
On Sat, Jun 1, 2013 at 12:28 AM, Mike Stump wrote:
> On May 31, 2013, at 2:56 PM, Andrew Pinski 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 this be handled? Imagine
On Fri, May 31, 2013 at 10:18 PM, Jeff Law 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
is f
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 shou
On Fri, May 31, 2013 at 5:02 PM, Tobias Burnus wrote:
> Am 31.05.2013 10:24, schrieb Richard Biener:
>>
>> On Thu, May 30, 2013 at 10:54 PM, Jeff Law wrote:
>>>
>>> Don't worry about it. The patch is good as-is.
>>
>> Why sink the !host_integerp check? Please keep it where it is now.
>>
>
> Answ
> 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 erro
90 matches
Mail list logo
