Hi,
On Fri, 8 May 2015 20:20:55, Joseph S. Myers wrote:
>
> On Fri, 8 May 2015, Bernd Edlinger wrote:
>
>> But it made the in-tree gmp configure script fail. That would not
>> have happened if we did not pass our version of missing to a sub-
>> module like gmp, that already has a working missing
On Mon, May 04, 2015 at 11:32:38PM +0300, Mikhail Maltsev wrote:
> > You'd probably be better off creating a unique rtx_insn * object and
> > using that as the marker.
> OK. Fixed the patch. Rebased and tested on x86_64-linux (fortunately, it
> did not conflict with Trevor's series of rtx_insn-rela
noexcept () is a syntax error; I think you mean plain "noexcept" here.
In C++11 complex.h does not #define complex, so the testcase should use
_Complex or __complex.
Tested x86_64-pc-linux-gnu, applying to trunk as obvious.
commit edb5a4d326b2e0ad7e4ed777d8f2917fd2f4eb2a
Author: Jason Merrill
The first patch adds -std=c++98 for building C++98 sources.
The second patch fixes all but one of the testsuite failures that turned
up with a compiler that defaults to C++11.
Tested x86_64-pc-linux-gnu, OK'd by Jonathan in Lenexa.
commit be5a2d163e2a25ab5a9978475e0e6e2954c9ddb3
Author: Jason
One C++11 compatibility issue that turns up a lot in the GCC sources is
that in C++98,
#define BAR "bar"
const char *p = "foo"BAR;
is well-formed, giving p the value "foobar". But in C++11 this is a
user-defined literal with the suffix BAR, which is ill-formed because
there is no BAR suffix
From: Trevor Saunders
gcc/ChangeLog:
2015-05-08 Trevor Saunders
* ira-color.c: Change argument types to rtx_insn *.
* lra-eliminations.c: Likewise.
* ira.h: Adjust.
---
gcc/ChangeLog | 6 ++
gcc/ira-color.c| 4 ++--
gcc/ira.h | 2 +-
From: Trevor Saunders
gcc/ChangeLog:
2015-05-08 Trevor Saunders
* emit-rtl.c (prev_cc0_setter): Change argument type to rtx_insn *.
* rtl.h: Adjust.
---
gcc/ChangeLog | 5 +
gcc/emit-rtl.c | 4 +---
gcc/rtl.h | 2 +-
3 files changed, 7 insertions(+), 4 deletions(-)
From: Trevor Saunders
gcc/ChangeLog:
2015-05-08 Trevor Saunders
* combine.c: Change argument type to rtx_insn *.
---
gcc/ChangeLog | 4
gcc/combine.c | 2 +-
2 files changed, 5 insertions(+), 1 deletion(-)
diff --git a/gcc/ChangeLog b/gcc/ChangeLog
index 6811ea1..b2767cf 10064
From: Trevor Saunders
gcc/ChangeLog:
2015-05-08 Trevor Saunders
* jump.c: Change argument types to rtx_insn *.
* rtl.h: Adjust.
---
gcc/ChangeLog | 5 +
gcc/jump.c| 6 +++---
gcc/rtl.h | 6 +++---
3 files changed, 11 insertions(+), 6 deletions(-)
diff --git a/gc
From: Trevor Saunders
gcc/ChangeLog:
2015-05-08 Trevor Saunders
* sched-deps.c: Change argument types to rtx_insn *.
* sched-int.h: Adjust.
---
gcc/ChangeLog| 5 +
gcc/sched-deps.c | 4 ++--
gcc/sched-int.h | 4 ++--
3 files changed, 9 insertions(+), 4 deletions(-)
From: Trevor Saunders
gcc/ChangeLog:
2015-05-08 Trevor Saunders
* ira.c (decrease_live_ranges_number): Changetype of local
variable to rtx_insn *.
* recog.c: Change argument types to rtx_insn *.
* recog.h: Adjust.
---
gcc/ChangeLog | 7 +++
gcc/ira.c
From: Trevor Saunders
gcc/ChangeLog:
2015-05-08 Trevor Saunders
* rtl.h: Adjust.
* rtlanal.c: Change argument type to rtx_insn *.
---
gcc/ChangeLog | 5 +
gcc/rtl.h | 2 +-
gcc/rtlanal.c | 2 +-
3 files changed, 7 insertions(+), 2 deletions(-)
diff --git a/gcc/Chang
From: Trevor Saunders
gcc/ChangeLog:
2015-05-08 Trevor Saunders
* lra-constraints.c: Change argument type to rtx_insn *.
---
gcc/ChangeLog | 4
gcc/lra-constraints.c | 2 +-
2 files changed, 5 insertions(+), 1 deletion(-)
diff --git a/gcc/ChangeLog b/gcc/ChangeLog
inde
From: Trevor Saunders
gcc/ChangeLog:
2015-05-08 Trevor Saunders
* df-problems.c: Change argument type to rtx_insn *.
---
gcc/ChangeLog | 4
gcc/df-problems.c | 2 +-
2 files changed, 5 insertions(+), 1 deletion(-)
diff --git a/gcc/ChangeLog b/gcc/ChangeLog
index b2767cf..9
From: Trevor Saunders
gcc/ChangeLog:
2015-05-08 Trevor Saunders
* dwarf2cfi.c: Change argument type to rtx_insn *.
---
gcc/ChangeLog | 4
gcc/dwarf2cfi.c | 2 +-
2 files changed, 5 insertions(+), 1 deletion(-)
diff --git a/gcc/ChangeLog b/gcc/ChangeLog
index e08209e..6804c36
From: Trevor Saunders
gcc/ChangeLog:
2015-05-08 Trevor Saunders
* reorg.c: Change argument types to rtx_insn *.
---
gcc/ChangeLog | 4
gcc/reorg.c | 10 +-
2 files changed, 9 insertions(+), 5 deletions(-)
diff --git a/gcc/ChangeLog b/gcc/ChangeLog
index efe4ca0..946
From: Trevor Saunders
gcc/ChangeLog:
2015-05-08 Trevor Saunders
* cse.c (cse_change_cc0_mode): Change argument type to rtx_insn *.
---
gcc/ChangeLog | 4
gcc/cse.c | 2 +-
2 files changed, 5 insertions(+), 1 deletion(-)
diff --git a/gcc/ChangeLog b/gcc/ChangeLog
index b591
From: Trevor Saunders
gcc/ChangeLog:
2015-05-08 Trevor Saunders
* gcse.c: Change argument types to rtx_insn *.
---
gcc/ChangeLog | 4
gcc/gcse.c| 8
2 files changed, 8 insertions(+), 4 deletions(-)
diff --git a/gcc/ChangeLog b/gcc/ChangeLog
index a595d23..598510c
From: Trevor Saunders
gcc/ChangeLog:
2015-05-08 Trevor Saunders
* emit-rtl.c (emit_note_after): Change argument type to rtx_insn *.
* rtl.h: Adjust.
---
gcc/ChangeLog | 5 +
gcc/emit-rtl.c | 3 +--
gcc/rtl.h | 2 +-
3 files changed, 7 insertions(+), 3 deletions(-)
From: Trevor Saunders
gcc/ChangeLog:
2015-05-08 Trevor Saunders
* emit-rtl.c (emit_note_before): Change argument type to rtx_insn *.
* rtl.h: Adjust.
---
gcc/ChangeLog | 5 +
gcc/emit-rtl.c | 3 +--
gcc/rtl.h | 2 +-
3 files changed, 7 insertions(+), 3 deletions(-)
From: Trevor Saunders
gcc/ChangeLog:
2015-05-08 Trevor Saunders
* emit-rtl.c (emit_debug_insn_before): Change argument type to
rtx_insn *.
* rtl.h: Adjust.
---
gcc/ChangeLog | 6 ++
gcc/emit-rtl.c | 2 +-
gcc/rtl.h | 2 +-
3 files changed, 8 insertions(+),
From: Trevor Saunders
gcc/ChangeLog:
2015-05-08 Trevor Saunders
* rtlanal.c (noop_move_p): Change argument type to rtx_insn *.
* rtl.h: Adjust.
---
gcc/ChangeLog | 5 +
gcc/rtl.h | 2 +-
gcc/rtlanal.c | 2 +-
3 files changed, 7 insertions(+), 2 deletions(-)
diff --g
From: Trevor Saunders
gcc/ChangeLog:
2015-05-08 Trevor Saunders
* rtlanal.c (add_shallow_copy_of_reg_note): Change argument type
to rtx_insn *.
* rtl.h: Adjust.
---
gcc/ChangeLog | 6 ++
gcc/rtl.h | 2 +-
gcc/rtlanal.c | 2 +-
3 files changed, 8 insertions(+)
From: Trevor Saunders
gcc/ChangeLog:
2015-05-08 Trevor Saunders
* reorg.c (stop_search_p): Change argument to rtx_insn *.
---
gcc/ChangeLog | 4
gcc/reorg.c | 4 ++--
2 files changed, 6 insertions(+), 2 deletions(-)
diff --git a/gcc/ChangeLog b/gcc/ChangeLog
index db7f740..f
From: Trevor Saunders
gcc/ChangeLog:
2015-05-08 Trevor Saunders
* rtlanal.c (in_insn_list_p): Renamed from in_expr_list_p.
* cfgrtl.c (can_delete_label_p): Adjust.
* rtl.h: likewise.
---
gcc/ChangeLog | 6 ++
gcc/cfgrtl.c | 2 +-
gcc/rtl.h | 2 +-
gcc/rtlana
From: Trevor Saunders
gcc/ChangeLog:
2015-05-08 Trevor Saunders
* except.c (can_nonlocal_goto): Change type of argument to
rtx_insn *.
* rtl.h: Adjust.
---
gcc/ChangeLog | 6 ++
gcc/except.c | 2 +-
gcc/rtl.h | 2 +-
3 files changed, 8 insertions(+), 2 delet
From: Trevor Saunders
gcc/ChangeLog:
2015-05-08 Trevor Saunders
* rtlanal.c (remove_reg_equal_equiv_notes): Change argument type
to rtx_insn *.
* rtl.h: Likewise.
---
gcc/ChangeLog | 6 ++
gcc/rtl.h | 2 +-
gcc/rtlanal.c | 2 +-
3 files changed, 8 insertions(
From: Trevor Saunders
gcc/ChangeLog:
2015-05-08 Trevor Saunders
* rtlanal.c (computed_jump_p): Cange argument type to rtx_insn *.
* rtl.h: Adjust.
---
gcc/ChangeLog | 5 +
gcc/rtl.h | 2 +-
gcc/rtlanal.c | 2 +-
3 files changed, 7 insertions(+), 2 deletions(-)
diff
From: Trevor Saunders
gcc/ChangeLog:
2015-05-08 Trevor Saunders
* except.c (make_reg_eh_region_note): Change argument to
rtx_insn *.
(make_reg_eh_region_note_nothrow_nononlocal): Likewise.
* except.h: Adjust.
---
gcc/ChangeLog | 7 +++
gcc/except.c | 4 +
From: Trevor Saunders
gcc/ChangeLog:
2015-05-08 Trevor Saunders
* mode-switching.c (commit_mode_sets): Change type of local
variable from rtx to rtx_insn *.
---
gcc/ChangeLog| 5 +
gcc/mode-switching.c | 4 ++--
2 files changed, 7 insertions(+), 2 deletions(-)
d
From: Trevor Saunders
Hi,
This series does a bunch more "trivial" changes from rtx to rtx_insn *.
each patch bootstrapped + regtested on x86_64-linux-gnu, and the series was run
through config-list.mk. I think this all falls within Jeff's preapproval
again, so committing to trunk.
Trev
Trev
On 05/08/2015 08:08 PM, Jason Merrill wrote:
On 05/08/2015 07:45 PM, Aldy Hernandez wrote:
Are you OK with your own patch? :-)
Hmm...
...
Well, alright.
Jsaon
Wait...Jsaon? This is not the same author :).
Committed to mainline. Thank you.
Aldy
OK.
Jason
On 05/08/2015 07:45 PM, Aldy Hernandez wrote:
Are you OK with your own patch? :-)
Hmm...
...
Well, alright.
Jsaon
On 05/08/2015 12:20 PM, Richard Biener wrote:
Just meant that it's making the changes smaller if those bits can be checked in
independently. If that's just a hassle...
Well, who am I to look a gift horse in the mouth? Especially when
you've volunteered to review most of this, and hopefully
Hi Jason.
Here is another independent patch factored out from the debug-early work.
I believe you approved this already, and was merely queued for stage1.
I have tested this independently of any other patch and would like
permission to commit it to mainline.
OK?
commit 899570880aca6c3d2e6e85
Hi Jason.
At Richi's request, I'm splitting up a couple of patches from the C++
debug-early branch that can live on its own.
This attached patch is a patch you wrote in our original attempt to
remove deferred_asm_names from dwarf2out.c.
I have tested it independently of everything else, and
On Fri, May 8, 2015 at 4:00 PM, Rich Felker wrote:
> On Fri, Apr 17, 2015 at 05:36:30AM -0700, H.J. Lu wrote:
>> On Fri, Apr 17, 2015@4:59 AM, Jakub Jelinek wrote:
>> > On Fri, Apr 17, 2015 at 04:48:48AM -0700, H.J. Lu wrote:
>> >> > I don't like it. Nonshared libgcc is libgcc.a, period. No sen
On Fri, Apr 17, 2015 at 05:36:30AM -0700, H.J. Lu wrote:
> On Fri, Apr 17, 2015@4:59 AM, Jakub Jelinek wrote:
> > On Fri, Apr 17, 2015 at 04:48:48AM -0700, H.J. Lu wrote:
> >> > I don't like it. Nonshared libgcc is libgcc.a, period. No sense in
> >> > creating yet another library for that.
> >>
On May 8, 2015 11:58:18 PM GMT+02:00, Jan Hubicka wrote:
>> On Fri, May 08, 2015 at 01:09:52PM -0600, Jeff Law wrote:
>> > On 05/08/2015 01:07 PM, David Edelsohn wrote:
>> > Steve Ellcey wrote:
>> > >>
>> > >>After your change GCC sees that the code for f1 and f2 are
>identical
>> > >>so it re
Hi Dodji,
Thanks for the review. I followed all your suggestions. For the
accessor functions, I was not sure what type you would prefer, so I
implemented them as C++ methods and made use of 'private' to be sure
they are the only way to access the locations array. If you want me to
change it, just
On 05/08/2015 02:14 PM, Segher Boessenkool wrote:
> "Cleaner"? In this code? Heh.
Heh.
> use_crosses_set_p often estimates pessimistically. Is that what is
> happening here? Using real dataflow in combine would fix that (and many
> other problems). Not that that helps you right now ;-)
Yes,
Hello world,
this patch fixes the regression in PR 66041, plus one more case
that came up when I looked at this.
OK for trunk?
Regards, Thomas
2015-05-08 Thomas Koenig
PR fortran/66041
* frontend-passes.c (scalarized_expr): Clear as->start, as->end
and as->st
My concern with accepting this patch is that many of libstdc++'s hash
functions are awful from a mixing point of view -- you would get
exactly the same problem from users who have integers which are always
a multiple of a power of 2 (which is in practice not uncommon). This
would give exactly the s
On Fri, May 08, 2015 at 03:32:58PM -0600, Jeff Law wrote:
> On 05/08/2015 02:15 PM, Richard Henderson wrote:
> >
> >But it *does* try to match an intermediate pattern,
> >
> >(set (reg:CCGC 17 flags)
> > (compare:CCGC (reg:CCGC 17 flags)
> > (const_int 0 [0])))
> >
> >which can be consi
> On Fri, May 08, 2015 at 01:09:52PM -0600, Jeff Law wrote:
> > On 05/08/2015 01:07 PM, David Edelsohn wrote:
> > Steve Ellcey wrote:
> > >>
> > >>After your change GCC sees that the code for f1 and f2 are identical
> > >>so it replaced the body of f2 with a call to f1. This optimization will
On 05/08/2015 02:32 PM, Jeff Law wrote:
> On 05/08/2015 02:15 PM, Richard Henderson wrote:
>>
>> But it *does* try to match an intermediate pattern,
>>
>> (set (reg:CCGC 17 flags)
>> (compare:CCGC (reg:CCGC 17 flags)
>> (const_int 0 [0])))
>>
>> which can be considered a no-op move.
On 05/04/2015 02:18 PM, Mikhail Maltsev wrote:
Yes, FWIW, it is only needed for assertions in peep2_regno_dead_p and
peep2_reg_dead_p which check it against NULL (they are intended to
verify that live_before field in peep2_insn_data struct is valid). At
least, when I removed the assertions and ch
On 05/08/2015 09:59 AM, Joseph Myers wrote:
Paul, although glibc's copy of parts of tzcode is a bit out of date, it
looks like the currenthttps://github.com/eggert/tz.git still has the
problematic code in private.h, relying on left-shifting -1 which has
undefined behavior in C99/C11 (implementat
On 05/05/2015 12:21 PM, David Malcolm wrote:
gcc/java/ChangeLog:
* jcf-parse.c (set_source_filename): Replace write through
ORDINARY_MAP_FILE_NAME with direct access to "to_file".
libcpp/ChangeLog:
* include/line-map.h (MAP_START_LOCATION): Eliminate the non-const
On 05/05/2015 12:21 PM, 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.
(linemap_check_ordinary): Convert from a macro to a pair of inline
On 05/05/2015 12:21 PM, David Malcolm wrote:
On Mon, 2015-05-04 at 13:15 -0600, Jeff Law wrote:
On 05/01/2015 06:56 PM, David Malcolm wrote:
[I didn't mark the inline functions as "static"; should they be?]
Just a follow-up on this. I got burned by the ODR issues around
implicit extern inline
On 05/08/2015 05:27 PM, Jeff Law wrote:
On 05/07/2015 08:42 AM, Andrew MacLeod wrote:
There has been some debate over the strength requirement of barriers for
Curse you for the BZ reference. I read maybe the first 75%, then
glossed over 15%, then read the last bit in detail. Hard to do near
On 05/05/2015 12:21 PM, David Malcolm wrote:
---
libcpp/include/line-map.h | 71 ---
1 file changed, 36 insertions(+), 35 deletions(-)
Cobble together a quick ChangeLog and test and this is fine.
jeff
On 05/08/2015 02:15 PM, Richard Henderson wrote:
But it *does* try to match an intermediate pattern,
(set (reg:CCGC 17 flags)
(compare:CCGC (reg:CCGC 17 flags)
(const_int 0 [0])))
which can be considered a no-op move. If I add the attached pattern, then the
combination happens i
I want to make the timevar infrastructure be more flexible,
to better serve some JIT use-cases.
The following is a "removal of global state" patch which gathers
together the state within timevar.c into a new "timer" class.
Within the classic cc1/cc1plus/etc use-case, this timer class
is a singlet
On 05/07/2015 08:42 AM, Andrew MacLeod wrote:
There has been some debate over the strength requirement of barriers for
__sync operations... This is documented within the PR.
Originally __sync was suppose to be synonymous with SEQ_CST, but there
has been a slight slackening of the barrier-ness of
Hi Richard,
On Fri, May 08, 2015 at 01:15:25PM -0700, Richard Henderson wrote:
> But it *does* try to match an intermediate pattern,
>
> (set (reg:CCGC 17 flags)
> (compare:CCGC (reg:CCGC 17 flags)
> (const_int 0 [0])))
>
> which can be considered a no-op move.
Maybe we should teach
Hi,
Could you please review my patch for predicated lim?
Let me note some details about it:
1) Phi statements are still moved only if they have 1 or 2
arguments. However, phi statements could be move under conditions (as
it’s done for the other statements). Probably, phi statement motion
On 05/08/2015 02:22 PM, Joseph Myers wrote:
On Fri, 8 May 2015, Jeff Law wrote:
* Sort out whether to update those top-level "missing" scripts.
Seems like we should if they came from automake.
I say we should *update* (from git). Not *downgrade to an older version*,
which is what the posted
On Fri, 8 May 2015, Bernd Edlinger wrote:
> But it made the in-tree gmp configure script fail. That would not
> have happened if we did not pass our version of missing to a sub-
> module like gmp, that already has a working missing script that behaves
> differently.
However, that's still not an
On 05/08/2015 09:43 AM, Uros Bizjak wrote:
> @@ -1509,8 +1509,7 @@
> (and:DI (ashift:DI (match_operand:DI 1 "register_operand" "r")
> (match_operand:DI 2 "mul8_operand" "I"))
> (match_operand:DI 3 "immediate_operand" "i")))]
> - "HOST_BITS_PER_WIDE_INT
On Fri, 8 May 2015, Jeff Law wrote:
> > * Sort out whether to update those top-level "missing" scripts.
> Seems like we should if they came from automake.
I say we should *update* (from git). Not *downgrade to an older version*,
which is what the posted patch did.
--
Joseph S. Myers
jos...@co
On 08/05/2015 10:02, Richard Biener wrote:
On Wed, May 6, 2015 at 10:10 PM, François Dumont wrote:
Hi
Following Marc Glisse comment #4
on:https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65641 I would like to
propose this enhancement to the hash functor for pointers. It simply gets
rid of the
On 05/08/2015 08:54 AM, Richard Henderson wrote:
> I have a feeling I know why these didn't get merged.
>
> The global optimizers aren't allowed to operate on hard registers lest they
> extend the lifetime of the hard register such that it creates an impossible
> situation for the register allocat
Today is the first day I've had to look at these comments.
>>if (TEMPLATE_PARM_CONSTRAINTS (current_template_parms))
>> -TYPE_CANONICAL (type) = type;
>> +SET_TYPE_STRUCTURAL_EQUALITY (type);
>
>
> This seems like papering over an underlying issue. What was the testcase
> that motiva
On 05/08/2015 12:41 PM, Jeff Law wrote:
> I'm going to assume the include header order juggling/additions were all
> necessary.
Correct -- target.h now uses vec<>, which means that vec.h must be included
first. In the olden days I'd have just put the vec.h include at the top of
target.h. ;-)
>
On 05/07/2015 03:38 PM, Richard Henderson wrote:
Using proper vectors instead of lists of trees.
---
gcc/cfgexpand.c | 614 ---
gcc/config/cris/cris.c | 88 +++
gcc/config/i386/i386.c | 24 +-
gcc/config/mn10300/mn10300.c
On Fri, May 08, 2015 at 01:09:52PM -0600, Jeff Law wrote:
> On 05/08/2015 01:07 PM, David Edelsohn wrote:
> Steve Ellcey wrote:
> >>
> >>After your change GCC sees that the code for f1 and f2 are identical
> >>so it replaced the body of f2 with a call to f1. This optimization will
> >>save spa
On May 8, 2015 8:48:26 PM GMT+02:00, Aldy Hernandez wrote:
>On 05/08/2015 04:57 AM, Richard Biener wrote:
>> On Fri, May 8, 2015 at 2:34 AM, Aldy Hernandez
>wrote:
>>>
>>
>> Maybe you can split out the Java aliases stuff (that annoyed me
>multiple times
>> when trying to refactor the FE - middle-
Hi,
this patch fixes PR66010.
I.
Consider this test-case, with a va_list passed from f2 to f2_1:
...
#include
inline int __attribute__((always_inline))
f2_1 (va_list ap)
{
return va_arg (ap, int);
}
int
f2 (int i, ...)
{
int res;
va_list ap;
va_start (ap, i);
res = f2_1 (ap);
v
On 08-05-15 17:31, Richard Biener wrote:
>OK for trunk?
As noted in one of the PRs I think that it is the proper time to
re-implement the stdarg optimization on the un-lowered form which
should also fix this.
AFAIU, the implementation of the stdarg optimization on the un-lowered form
should c
On May 8, 2015 8:43:15 PM GMT+02:00, Jeff Law wrote:
>On 05/08/2015 03:27 AM, Richard Biener wrote:
>> On Wed, May 6, 2015 at 4:04 PM, Yuri Rumyantsev
>wrote:
>>> Hi All,
>>>
>>> Here is a patch which gives us significant speed-up on HASWELL for
>>> test containing masked stores. The main goal of
On 05/08/2015 10:38 AM, David Malcolm wrote:
I tested your branch (at 09263eae7c260c305fa19ffa186afd1d89654fb8) with
the jit configuration from the docs:
../src/configure \
--enable-host-shared \
--enable-languages=jit,c++ \
--disable-bootstrap \
--enable-checking=release \
--
On 05/08/2015 01:07 PM, David Edelsohn wrote:
Steve Ellcey wrote:
After your change GCC sees that the code for f1 and f2 are identical
so it replaced the body of f2 with a call to f1. This optimization will
save space but it is not going to be faster because any call to f2 will
now include an
>>> Steve Ellcey wrote:
>
> After your change GCC sees that the code for f1 and f2 are identical
> so it replaced the body of f2 with a call to f1. This optimization will
> save space but it is not going to be faster because any call to f2 will
> now include an extra call/return. Do other platfor
On 05/06/2015 11:29 AM, Michael Meissner wrote:
On Wed, May 06, 2015 at 04:03:00PM +0100, Richard Sandiford wrote:
Jeff Law writes:
So my worry here is that folks writing these loops to iterate over modes
are going to easily miss the != VOIDmode terminator, or not know when to
use GET_MODE_WID
Jeff Law writes:
> On 05/08/2015 10:50 AM, Joseph Myers wrote:
> >
> > Note that however the dynamic linker does properly need to save and
> > restore call-clobbered registers used for argument passing (because of
> > IFUNCs, user-provided malloc, audit hooks etc. that might affect them
> > even i
On 05/08/2015 11:08 AM, Ian Lance Taylor wrote:
This is fine if it works. Thanks.
Ian
On Thu, May 7, 2015 at 5:36 PM, Aldy Hernandez wrote:
As mentioned, no regressions with --enable-languages=go.
I will mark the Go changes as approved pending on the rest of the
patches being approved.
Thanks for working on this. Have you tried building with
--enable-languages=go? On a GNU/Linux system it should build and pass
all tests with no extra effort.
Yes. No regressions.
Thanks.
Aldy
On 05/07/2015 03:38 PM, Richard Henderson wrote:
Prepatory to converting from tree chains to vectors.
---
gcc/cfgexpand.c | 167
1 file changed, 72 insertions(+), 95 deletions(-)
OK with a ChangeLog and the usual testing.
jeff
On 05/08/2015 08:38 AM, Joel Brobecker wrote:
Hello,
Attempting to build libiberty on LynxOS-178 fails trying to compile
mkstemps.c with the following error:
LynxOS? I haven't had to do anything with that since, umm, the early
90s. So sorry you've got that task...
libiberty/ChangeLog:
On 05/07/2015 09:51 AM, Michael Haubenwallner wrote:
Are there any cases where you had to do something other than just regenerate
the various files?
Nope - at least nothing that I would call "doing something other". What I did:
* touch each configure.ac to ensure automake is really used
* do
On 05/08/2015 04:57 AM, Richard Biener wrote:
On Fri, May 8, 2015 at 2:34 AM, Aldy Hernandez wrote:
Maybe you can split out the Java aliases stuff (that annoyed me multiple times
when trying to refactor the FE - middle-end interface). It looks
unrelated enough.
Thanks,
Richard.
Jason al
On 05/08/2015 12:40 PM, H. Peter Anvin wrote:
On 05/07/2015 02:39 PM, Richard Henderson wrote:
All j mnemonics implemented as =@cc
to make it easy for someone reading the manual
to figure out what condition is desired.
One request: would it be possible to get a cpp symbol for this (e.g.
__GCC_
On 05/08/2015 03:27 AM, Richard Biener wrote:
On Wed, May 6, 2015 at 4:04 PM, Yuri Rumyantsev wrote:
Hi All,
Here is a patch which gives us significant speed-up on HASWELL for
test containing masked stores. The main goal of that patch is attempt
to avoid HW hazard for maskmove instructions thr
Hi,
On Fri, 8 May 2015 16:41:02, Joseph S. Myers wrote:
>
> On Fri, 8 May 2015, Bernd Edlinger wrote:
>
>> One example where there is an incompatibility is "missing":
>>
>> Formerly it had code that emulated the missing "flex" by
>> creating a dummy lex.yy.c from the hopefully installed
>> pre-com
On 05/07/2015 02:39 PM, Richard Henderson wrote:
> All j mnemonics implemented as =@cc
> to make it easy for someone reading the manual
> to figure out what condition is desired.
One request: would it be possible to get a cpp symbol for this (e.g.
__GCC_X86_INLINE_ASM_CC__) so we don't have to do
On 05/08/2015 03:09 AM, Thomas Schwinge wrote:
Hi!
On Mon, 27 Apr 2015 11:20:30 +0100, Richard Sandiford
wrote:
This patch [...] by replacing most of genrecog [...]
OK to commit?
Is it a bug that I'm seeing these warnings only in the stage 1 build with
the bootstrap GCC 4.6 compiler, but n
On 05/08/2015 10:50 AM, Joseph Myers wrote:
Note that however the dynamic linker does properly need to save and
restore call-clobbered registers used for argument passing (because of
IFUNCs, user-provided malloc, audit hooks etc. that might affect them even
if the dynamic linker itself doesn't);
On 05/08/2015 08:25 AM, Matthew Fortune wrote:
There is also the perspective that we should be able to aim for
an ABI variant agnostic dynamic linker at some point over the next
year by working towards a build that truly uses no float and is
hence compatible with all the ABI variants.
Having see
On 05/07/2015 04:15 PM, Jim Wilson wrote:
ping
https://gcc.gnu.org/ml/gcc-patches/2015-04/msg01690.html
OK. Must have missed it when it flew by -- sorry.
jeff
On 05/07/2015 03:38 PM, Richard Henderson wrote:
For testing parity coming out of asm flags.
---
gcc/config/i386/i386-modes.def | 2 ++
gcc/config/i386/i386.c | 19 +++
2 files changed, 13 insertions(+), 8 deletions(-)
Seems like it ought to move forward now.
Oh yea
On 05/07/2015 03:38 PM, Richard Henderson wrote:
If gimple_asm_volatile_p is correct, no point re-checking.
This is also done by the C and C++ front ends, but not Ada.
So we can't yet trust ASM_VOLATILE_P from the front end.
---
gcc/cfgexpand.c | 11 +++
gcc/gimplify.c | 2 +-
2 fil
On 05/07/2015 03:38 PM, Richard Henderson wrote:
We do it in the front end already; no need to repeat.
---
gcc/cfgexpand.c | 2 --
gcc/stmt.c | 7 ---
2 files changed, 4 insertions(+), 5 deletions(-)
Any reason this shouldn't go into the tree immediately? Seems like it
stands on it
This is fine if it works. Thanks.
Ian
On Thu, May 7, 2015 at 5:36 PM, Aldy Hernandez wrote:
>
On Wed, Apr 29, 2015 at 5:56 PM, Aldy Hernandez wrote:
>
> Despite what Go thinks:
>
> /* The debug hooks are used to implement -fdump-go-spec because it
> gives a simple and stable API for all the information we need to
> dump. */
>
> ...the debug hooks are not stable... :).
Alas.
Jan and Martin,
I just noticed that your patch for PR bootstrap/65150 broke one of the
MIPS tests (gcc.target/mips/branch-1.c). I can fix the test with no
problem but I am wondering if the change I am seeing with your patch
is intended or not.
A cutdown version of the test is:
void bar (void);
On May 7, 2015, at 5:39 PM, Aldy Hernandez wrote:
So, I don’t feel there is anything in there for me to review, I’d like the
front-end maintainer to review.
dearly-testsuite.patch
Description: Binary data
On May 7, 2015, at 5:38 PM, Aldy Hernandez wrote:
Ok.
1 - 100 of 191 matches
Mail list logo
