Hi,
On 05/19/2014 05:08 PM, Tim Shen wrote:
+ // TODO Refactor this piece of junk.
I suggest rewording this or avoiding it completely.
Paolo.
Sorry about it. Here is the patch. There is one remaining case where
cgraph_dump_file and dump_enable_p are checked separately --
cgraph_dump_file is set up differently from 'dump_file'.
David
On Mon, May 19, 2014 at 2:21 AM, Richard Biener
richard.guent...@gmail.com wrote:
On Fri, May 16,
On Mon, May 19, 2014 at 6:48 AM, Jan Hubicka hubi...@ucw.cz wrote:
Thanks for the pointer, there is indeed the recommendation in
optimization manual [1], section 3.6.4, where it is said:
--quote--
Misaligned data access can incur significant performance penalties.
This is particularly
On 05/05/2014 09:54 AM, Evgeny Stupachenko wrote:
@@ -42943,6 +42944,10 @@ expand_vec_perm_1 (struct expand_vec_perm_d *d)
if (expand_vec_perm_vpermil (d))
return true;
+ /* Try palignr on one operand. */
+ if (d-one_operand_p expand_vec_perm_palignr (d))
+return true;
No,
A fault in thumb1_reorg means we can try to get the insn_code of
something that isn't an insn. This appears to be a latent problem
that's suddenly started to bite on trunk. The code in question appears
to go back to gcc-4.8.
RTL checking would probably have found this quickly, but that's very
On May 19, 2014, at 12:27 AM, Richard Sandiford rdsandif...@googlemail.com
wrote:
Does anyone have any suggestions for a better name than wide_int though?
Please, no.
The main property of wide_int is that it has a variable precision, whereas
widest_int and offset_int have constant
On Mon, May 19, 2014 at 9:14 AM, Uros Bizjak ubiz...@gmail.com wrote:
On Mon, May 19, 2014 at 6:48 AM, Jan Hubicka hubi...@ucw.cz wrote:
Thanks for the pointer, there is indeed the recommendation in
optimization manual [1], section 3.6.4, where it is said:
--quote--
Misaligned data
On Mon, May 19, 2014 at 6:42 PM, H.J. Lu hjl.to...@gmail.com wrote:
Uros,
I am looking into libreoffice size and the data alignment seems to make huge
difference. Data section has grown from 5.8MB to 6.3MB in between GCC 4.8
and 4.9,
while clang produces 5.2MB.
The two patches I posted to
On 05/19/14 06:54, Richard Biener wrote:
In this PR we run into the issue that releasing SSA names from
FRE/PRE elimination corrupts the VN lattice and thus the VN lookup
we perform for removing redudnant stores ICEs. The patch works
around the particular case by making unreachable code
On Mon, May 19, 2014 at 9:45 AM, Uros Bizjak ubiz...@gmail.com wrote:
On Mon, May 19, 2014 at 6:42 PM, H.J. Lu hjl.to...@gmail.com wrote:
Uros,
I am looking into libreoffice size and the data alignment seems to make
huge
difference. Data section has grown from 5.8MB to 6.3MB in between GCC
On May 19, 2014 6:57:52 PM CEST, Jeff Law l...@redhat.com wrote:
On 05/19/14 06:54, Richard Biener wrote:
In this PR we run into the issue that releasing SSA names from
FRE/PRE elimination corrupts the VN lattice and thus the VN lookup
we perform for removing redudnant stores ICEs. The patch
On 05/19/14 02:58, Eric Botcazou wrote:
Hi,
this fixes an over-optimization of the GIMPLE optimizer, whereby two otherwise
identical calls to a pure function present in different EH regions are CSEd,
which changes the semantics of the program because the second EH handler is
not invoked:
On Wed, 2014-05-14 at 12:09 -0600, Jeff Law wrote:
On 05/12/14 15:36, David Malcolm wrote:
[ ... Big Snip ... ]
This series of 3 patches is approved.
FWIW, I've been working my way through the remainder of the patches,
updating them to take account of the largely mechanical changes [1] for
the
On 05/19/14 07:58, Richard Biener wrote:
On Wed, 7 May 2014, Richard Biener wrote:
This removes the need_64bit_hwi logic, nothing else (well, brings
libcpp in line with gcc).
Bootstrap / regtest pending on x86_64-unknown-linux-gnu.
Bootstrapped and tested on x86_64-unknown-linux-gnu, ok
On 05/19/14 00:38, Bin.Cheng wrote:
1) Should we do it in a separated pass, or just along with scheduler?
ISTM that when we're able to combine insns that can impact the schedule
we'd like to generate, possibly in significant ways. That argues for a
separate pass that runs before the
On 05/19/14 07:10, Bill Schmidt wrote:
Hi,
I'd like to once again ping this patch from 2014-04-22:
http://gcc.gnu.org/ml/gcc-patches/2014-04/msg01319.html
OK for the trunk. Thanks for your patience.
jeff
So, before we forget to do this…
https://gcc.gnu.org/ml/gcc/2014-04/msg00284.html
Index: MAINTAINERS
===
--- MAINTAINERS (revision 210619)
+++ MAINTAINERS (working copy)
@@ -301,6 +301,9 @@ register allocation Kenneth Zadeck
On 05/19/14 02:19, Ilya Enkovich wrote:
On 16 May 13:39, Jeff Law wrote:
On 04/16/14 05:35, Ilya Enkovich wrote:
Hi,
This patch introduces Intel MPX bound registers and instructions. It was
approved earlier for 4.9 and had no significant changes since then. I'll
assume patch is OK if no
Richard Sandiford rdsandif...@googlemail.com writes:
Gerald Pfeifer ger...@pfeifer.com writes:
On Sat, 17 May 2014, Richard Sandiford wrote:
To rule out one possibility: which GCC are you using for stage1?
I think that may the smoking gun. When I use GCC 4.7 to bootstrap,
FreeBSD 8, 9 and
OK.
Jason
On 05/18/14 22:36, David Wohlferd wrote:
2014-05-18 David Wohlferd d...@limegreensocks.com
* doc/extend.texi: Create Label Attributes section,
move all label attributes into it and reference it.
THanks. Installed.
If you're going to continue to submit changes with any
On Sun, 18 May 2014, Sandra Loosemore wrote:
H, I'm guessing this was some concern about invalid code motion around a
setjmp. Our original analysis document lists F does not call setjmp as a
requirement for the optimization, so this was probably a case where we were
being excessively
On 05/19/14 00:38, Bin.Cheng wrote:
On Sat, May 17, 2014 at 12:32 AM, Jeff Law l...@redhat.com wrote:
On 05/16/14 04:07, Bin.Cheng wrote:
Yes, I think this one does have a good reason. The target independent
pass just makes sure that two consecutive memory access instructions
are free of
On 05/18/14 09:33, John David Anglin wrote:
The attached change appears to fix PR middle-end/61141. On PA, we can get
deleted insn notes in call sequences. The attached change checks to
make sure we have
a valid insn before calling reset_insn_used_flags and verify_insn_sharing.
Tested on
Charles Baylis charles.bay...@linaro.org writes:
On 13 February 2014 09:18, Richard Sandiford rdsandif...@googlemail.com
wrote:
This patch tries to reduce that by providing an alternative single-script
version. I was torn between Python and Tcl, but given how most people
tend to react to
Ping.
On Thu, May 15, 2014 at 11:34 AM, Sriraman Tallam tmsri...@google.com wrote:
Optimize access to globals with -fpie, x86_64 only:
Currently, with -fPIE/-fpie, GCC accesses globals that are extern to the
module
using the GOT. This is two instructions, one to get the address of the
On 05/17/14 01:33, Richard Sandiford wrote:
I suppose we could put the onus on the users of the iterator to invoke
a handle subrtxes of this code routine once they know what the code is.
That could make things a bit ugly though. E.g.:
FOR_EACH_SUBRTX (iter, array, expr, NONCONST)
if
Ping.
On Thu, Apr 17, 2014 at 10:41 AM, Sriraman Tallam tmsri...@google.com wrote:
Ping.
On Wed, Feb 5, 2014 at 4:31 PM, Sriraman Tallam tmsri...@google.com wrote:
Hi,
I would like this patch reviewed and considered for commit when
Stage 1 is active again.
Patch Description:
A C++
How about doing digest_init in get_nsdmi, so that the conversion is also
exposed to walk_field_subobs?
Jason
The problem in this testcase was that when we go to instantiate the
lambda in the NSDMI, since it wasn't defined in a function we
push_to_top_level and thereby clobber the current_class_ptr we set up.
Tested x86_64-pc-linux-gnu, applying to trunk.
commit
Jeff Law l...@redhat.com writes:
On 05/17/14 01:33, Richard Sandiford wrote:
I suppose we could put the onus on the users of the iterator to invoke
a handle subrtxes of this code routine once they know what the code is.
That could make things a bit ugly though. E.g.:
FOR_EACH_SUBRTX
On 19 May 2014 19:07, Richard Sandiford rdsandif...@googlemail.com wrote:
Sorry for the breakage. I wanted to make the script as picky as I could
get away with though, so that results aren't lost accidentally.
Could you try the attached?
That works for me.
Thanks for looking into it.
Richard Biener rguent...@suse.de writes:
The following is my current idea on progressing on the HOST_WIDE_INT
removal
1) https://gcc.gnu.org/ml/gcc-patches/2014-05/msg00381.html (ping)
2) make sure [u]int64_t is available and use that to define HOST_WIDE_INT
3) s/HOST_WIDE_INT/int64_t/
On 04/29/14 09:28, Prathamesh Kulkarni wrote:
Append evaluates to 0, in Wundef diagnostic.
clang prints the following diagnostic for -Wundef:
undef.c:1:5: warning: 'FOO' is not defined, evaluates to 0 [-Wundef]
#if FOO
^
OK to commit ?
[libcpp]
* expr.c (eval_token): Modify Wundef
Rebased patch to current master attached. DWARF parts approved by
Cary Coutant, GDB already contains Tom Tromey's code to take advantage
of the new information. Earlier discussions:
https://gcc.gnu.org/ml/gcc-patches/2014-04/msg00713.html
https://gcc.gnu.org/ml/gcc-patches/2014-04/msg01859.html
Catherine included an earlier version of this patch with the microMIPS
submission a couple years ago:
https://gcc.gnu.org/ml/gcc-patches/2012-07/msg00972.html
Richard's response was:
Looks like the wrong place to do this. Please treat this as a separate
patch and get a tree expert to
Hi,
On 19/mag/2014, at 20:28, Jason Merrill ja...@redhat.com wrote:
How about doing digest_init in get_nsdmi, so that the conversion is also
exposed to walk_field_subobs?
Thanks, I'll look into that.
Paolo
On 15/05/2014 22:52, Jonathan Wakely wrote:
On 15/05/14 22:20 +0200, François Dumont wrote:
Hi
Here is a proposal to fix PR 61143. As explained in the PR I
finally prefer to leave the container in a valid state that is to say
with a non zero number of bucket, that is to say 1, after a
I was playing around with this a few months ago but set it aside because
we weren't in stage 1 at the time. The way we currently set up comdat
groups early means that we need to mangle decls early, significantly
negating the earlier work to lazily set DECL_ASSEMBLER_NAME. This patch
improves
I've updated the patch. Shall I move the check inside cgraph_clone_node?
Thanks,
Dehao
Index: gcc/ipa-inline-transform.c
===
--- gcc/ipa-inline-transform.c (revision 210535)
+++ gcc/ipa-inline-transform.c (working copy)
@@ -183,8
Hello,
some systems such as GNU Hurd, don't define PATH_MAX at all, and on
some other systems many syscalls apparently work for paths longer than
PATH_MAX. Thus GFortran shouldn't truncate paths to PATH_MAX
characters, but rather use heap allocated buffers limited only by the
available memory.
On 19/05/14 11:08 -0400, Tim Shen wrote:
* testsuite/28_regex/algorithms/regex_match/ecma/char/quoted_char.cc:
New testcase.
This sounds like it's adding a new file, not extending it with new
tests.
@@ -435,6 +439,7 @@ _GLIBCXX_BEGIN_NAMESPACE_VERSION
std::vector_CharT
I've updated the patch. Shall I move the check inside cgraph_clone_node?
Thanks,
I think it is OK as it is. I belive individual users should know what do to
in such cases themselves.
You may want to also check what ipa-cp is doing.
Patch is OK (with Changelog)
Honza
Thanks,
Dehao
Index:
On 04/02/14 11:40, Joern Rennecke wrote:
On 2 April 2014 17:34, Joern Rennecke joern.renne...@embecosm.com wrote:
Hmm, the sanity check in new_seginfo caused a boostrap failure
building libjava on x86.
There was a block with CODE_LABEL as basic block head, otherwise empty.
I've added the
On 05/13/2014 03:21 AM, Mark Wielaard wrote:
So the debugger doesn't have to guess the properties of the enum's
underlying base type, like size, encoding and signedness.
Well, the enum already has DW_AT_byte_size. It seems to me that it
should also have DW_AT_encoding to provide the other
On 05/17/2014 04:07 AM, Jonathan Wakely wrote:
On 17 May 2014 10:50, Jonathan Wakely wrote:
On 17 May 2014 01:16, Sandra Loosemore wrote:
It appears that this patch from last fall never got reviewed.
https://gcc.gnu.org/ml/gcc-patches/2013-10/msg02340.html
Can someone take a look? I'll
On May 19, 2014, at 10:30 AM, Jeff Law l...@redhat.com wrote:
Yes, I think it's more than upsizing the mode. There is another
example from one of x86's candidate peephole patch at
https://gcc.gnu.org/ml/gcc-patches/2014-04/msg00467.html
The patch wants to do below transformation, which I
On Mon, May 19, 2014 at 1:40 PM, Jan Hubicka hubi...@ucw.cz wrote:
I've updated the patch. Shall I move the check inside cgraph_clone_node?
Thanks,
I think it is OK as it is. I belive individual users should know what do to
in such cases themselves.
You may want to also check what ipa-cp is
Hi Jeff,
On 19-May-14, at 1:51 PM, Jeff Law wrote:
On 05/18/14 09:33, John David Anglin wrote:
The attached change appears to fix PR middle-end/61141. On PA, we
can get
deleted insn notes in call sequences. The attached change checks to
make sure we have
a valid insn before calling
On Fri, May 16, 2014 at 06:49:45PM +0100, Vladimir Makarov wrote:
The following patch fixes
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=60969
The patch was bootstrapped and tested on x86/x86-64.
Committed as rev. 210519 to gcc 4.9 branch and as rev. 210520 to trunk.
2014-05-16
On Thu, May 15, 2014 at 1:00 AM, Janne Blomqvist
blomqvist.ja...@gmail.com wrote:
Hi,
a common malloc() pattern is malloc(num_foo * sizeof(foo_t), that
is, create space for an array of type foo_t with num_foo elements.
There is a slight danger here in that the multiplication can overflow
and
This is a partial patch to make the printers work with Python2 and
Python3, there are more changes needed but this bit is simple.
Tested x86_64-linux, committed to trunk.
commit 0f4c27366c14976900d0420c7c4318d6cc1cbf67
Author: Jonathan Wakely jwak...@redhat.com
Date: Mon May 19 22:18:13 2014
I was playing around with this a few months ago but set it aside
because we weren't in stage 1 at the time. The way we currently set
up comdat groups early means that we need to mangle decls early,
significantly negating the earlier work to lazily set
DECL_ASSEMBLER_NAME. This patch
On Mon, May 19, 2014 at 1:40 PM, Jan Hubicka hubi...@ucw.cz wrote:
I've updated the patch. Shall I move the check inside cgraph_clone_node?
Thanks,
I think it is OK as it is. I belive individual users should know what do to
in such cases themselves.
You may want to also check what
Hi,
first blush, this is just an ICE on invalid for a quite special case.
However, comparing the union case to the struct case (on which we don't
ICE):
struct S
{
S s[1] = { 0 };
};
xxx.C:3:16: error: could not convert ‘0’ from ‘int’ to ‘S’
S s[1] = { 0 };
shows that in this area there is
... in fact, I need to be more careful when comparing the types, also
because of cv-qualifiers, eg, for:
struct S
{
const S s[1] = { 0 };
};
the trees for S and const S are definitely different. Done in the attached.
Thanks,
Paolo.
///
/cp
2014-05-20 Paolo Carlini
On Mon, May 19, 2014 at 2:37 PM, James Greenhalgh
james.greenha...@arm.com wrote:
On Fri, May 16, 2014 at 06:49:45PM +0100, Vladimir Makarov wrote:
The following patch fixes
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=60969
The patch was bootstrapped and tested on x86/x86-64.
Committed
Ian,
On Sat, 17 May 2014, Richard Biener wrote:
On May 17, 2014 12:22:23 AM CEST, Maciej W. Rozycki
ma...@codesourcery.com wrote:
On Fri, 16 May 2014, Joseph S. Myers wrote:
2014-05-16 Maciej W. Rozycki ma...@codesourcery.com
PR libgcc/60166
*
On Mon, May 19, 2014 at 11:28 AM, Paolo Carlini
paolo.carl...@oracle.com wrote:
I suggest rewording this or avoiding it completely.
Sorry, I just can't tolerate my former quick dirty code.
On Mon, May 19, 2014 at 2:49 PM, Jonathan Wakely jwak...@redhat.com wrote:
I'm going to check in my
Hi,
This patch fixes sched_insn debug counter that was broken by
modulo/backtracking changes.
Tested on arm-linux-gnueabihf and x86_64-linux-gnu, though testsuite does not
exercise this execution path. Real testing was done by me debugging an
unrelated problem in the scheduler using
From: Richard Biener [mailto:richard.guent...@gmail.com]
Agreed, but I am happy with doing that as a followup. Btw,
a very simple one would be to reject unaligned
SLOW_UNALIGNED_ACCESS (TYPE_MODE (load_type), align).
[of course that may be true on MIPS even for the cases where
a
On Fri, May 16, 2014 at 9:58 AM, pins...@gmail.com wrote:
On May 16, 2014, at 4:13 AM, Richard Biener richard.guent...@gmail.com
wrote:
On Fri, May 16, 2014 at 1:03 PM, Richard Biener
richard.guent...@gmail.com wrote:
On Fri, May 16, 2014 at 12:56 PM, pins...@gmail.com wrote:
On May
On 05/19/2014 06:43 PM, Paolo Carlini wrote:
if (unqualified_id)
- error (field %qD has incomplete type %qT,
-unqualified_id, type);
+ cxx_incomplete_type_error (unqualified_id, type);
else
error (name %qT has
This patch improves debug printouts for scheduling passes.
Tested on arm-linux-gnueabihf and x86_64-linux-gnu.
OK to apply?
Thank you,
--
Maxim Kuvyrkov
www.linaro.org
0002-Improve-scheduling-debug-output.patch
Description: Binary data
Hi,
This patch removes msched-prefer-non-data-spec-insns and
msched-prefer-non-control-spec-insns IA64 options and handling thereof. I have
implemented these option during my youthful indiscrimination and, as far as I
can tell, no one uses them. Yet, handling of these options complicate
On 05/18/2014 08:45 PM, Sandra Loosemore wrote:
On 05/18/2014 02:59 PM, Jan Hubicka wrote:
For cases like local-statics-7 your approach can be saved by adding
simple IPA analysis
to look for static vars that are used only by one function and keeping
your DSE code active
for them, so we can
Hi,
This patch cleans up haifa-sched.c:choose_ready() function while allow more
powerful customization by backends at the same time.
The primary change is that
targetm.sched.first_cycle_multipass_dfa_lookahead_guard hook is converted from
returning a boolean OK/Nada value to returning an
On 05/18/2014 08:45 PM, Sandra Loosemore wrote:
On 05/18/2014 02:59 PM, Jan Hubicka wrote:
For cases like local-statics-7 your approach can be saved by adding
simple IPA analysis
to look for static vars that are used only by one function and keeping
your DSE code active
for them, so we can
On Mon, May 19, 2014 at 11:49 AM, Jonathan Wakely jwak...@redhat.com wrote:
The patch is OK with the ChangeLog tweak and omitting the new file.
Booted, tested and committed.
Thanks!
--
Regards,
Tim Shen
Ping.
Teresa
On Wed, May 14, 2014 at 4:39 PM, Teresa Johnson tejohn...@google.com wrote:
This patch applies profile fixups to COMDATs on the dyn ipa callgraph
at the end of LIPO module grouping (either in the profile gen run or
in gcov-tool). This is to address issues with missing profiles in
201 - 270 of 270 matches
Mail list logo