Hi Mike,
I plan on applying a similar patch to the mainline sources once I have
finished regression testing them.
Really, trunk should always go in first... Could you hold 4.7 until trunk goes
in?
Sorry, I had already checked the patch in. I have now checked in the
trunk patch, with
Hi Dodji Seketeli:
Is it possible to gcc to accept libcpp.patch and plugin.patch?
I recently rewrite my doc.txt which mainly add a new section Macro
Expansion Overview, it's focused on pfile.context usage linking to
macro. I think it's important to use cb_macro_start/end callbacks
because most
Hi,
In general, invariant motion itself can not reduce code size. But it will
change the liverange of the invariant, which might lead to more spilling.
The patch disables loop2_invariant when optimizing for size.
I measured the code size benefit for four targets based on CSiBE benchmark:
ARM:
On Tue, Jun 26, 2012 at 11:28 PM, H.J. Lu hjl.to...@gmail.com wrote:
On Tue, Jun 26, 2012 at 2:04 PM, Alexandre Oliva aol...@redhat.com wrote:
I test i686-linux-gnu in a presumably unusual setting: it's an
x86_64-linux-gnu system, and I've configured the GCC build to use as
builddev tools
On Wed, Jun 27, 2012 at 5:02 AM, Richard Henderson r...@redhat.com wrote:
The problem I'd like to solve is stuff like
pxor %xmm4, %xmm4
...
movdqa %xmm4, %xmm2
pcmpgtd %xmm0, %xmm2
In that there's no point performing the copy from xmm4
rather than just emitting a
On Wed, Jun 27, 2012 at 2:35 AM, Magnus Fromreide ma...@lysator.liu.se wrote:
On Mon, 2012-06-25 at 15:17 -0700, Lawrence Crowl wrote:
On 6/25/12, Joseph S. Myers jos...@codesourcery.com wrote:
On Mon, 25 Jun 2012, Diego Novillo wrote:
[ Added doc maintainers in CC ]
I have added a bit
[...]
| Two spaces for members is common practice with GNU, and it seems to be
| used for libstdc++.
|
| One space for protection labels is not something I have heard of before
| and libstdc++ uses no indentation for them.
|
| A freshly started emacs also doesn't indent access labels.
|
On Wed, Jun 27, 2012 at 10:40 AM, Zhenqiang Chen zhenqiang.c...@arm.com wrote:
Hi,
In general, invariant motion itself can not reduce code size.
It can expose CSE opportunities across loops though.
But it will
change the liverange of the invariant, which might lead to more spilling.
might
On Wed, Jun 27, 2012 at 10:40 AM, Zhenqiang Chen zhenqiang.c...@arm.com wrote:
Hi,
In general, invariant motion itself can not reduce code size. But it will
change the liverange of the invariant, which might lead to more spilling.
This may be true for ARM but it's not true in general.
[Adding gcc@]
On Jun 26, 2012, H.J. Lu hjl.to...@gmail.com wrote:
On Tue, Jun 26, 2012 at 3:39 PM, Mike Stump mikest...@comcast.net wrote:
On Jun 26, 2012, at 2:04 PM, Alexandre Oliva wrote:
I test i686-linux-gnu in a presumably unusual setting
I like the setup and testing...
This
On 26.06.2012 22:17, Alexandre Oliva wrote:
On Jun 26, 2012, Christophe Lyon christophe.l...@st.com wrote:
On 25.06.2012 17:24, Joseph S. Myers wrote:
On Mon, 25 Jun 2012, Christophe Lyon wrote:
Ping?
I advise CCing appropriate maintainers (in this case, build system
maintainers) on pings.
On Jun 21, 2012, Uros Bizjak ubiz...@gmail.com wrote:
Hello!
During htab_delete (dropped_values), loc_exp_dep_pool
allocated objects might be accessed, so it is better to free the
pool afterwards.
Bootstrapped/regtested on i686-linux, ok for trunk?
Looks obvious.
The patch doesn't
On Jun 25, 2012, Alexandre Oliva aol...@redhat.com wrote:
On Jun 24, 2012, Richard Sandiford rdsandif...@googlemail.com wrote:
gcc.c-torture/compile/vector-2.c fails on mips64-elf with RTL checking
enabled because dead_debug_insert_temp tries to read the REGNO of something
that it has already
All,
This patch enables the dump-noaddr test to work in out-of-build-tree testing.
It does this by making sure that the dump files generated during the
test are created under $tmpdir.
gcc/testsuite/ChangeLog:
2012-06-27 Matthew Gretton-Dann matthew.gretton-d...@arm.com
*
This fixes PR53774, a case where reassoc produced non-canonical
statements like a = 4 + b. The reason it did so was that it
assigned a rank of zero to things that are not constant. Fixed
by pre-computing a rank for all SSA default defs.
Bootstrapped and tested on x86_64-unknown-linux-gnu,
This fixes PR53695 where we recognize a loop which has only an
abnormal goto edge from its header to its tail. We already reject
loops during recognition that have at least a single abnormal
predecessor of its head so it seems reasonable to at least require
one regular CFG path from its header
On Wed, Jun 27, 2012 at 2:19 AM, Lawrence Crowl cr...@google.com wrote:
On 6/26/12, Jason Merrill ja...@redhat.com wrote:
On 06/25/2012 06:26 PM, Lawrence Crowl wrote:
+orcodegcc_unreachable/code. If the checks are expensive or the
+compiler can reasonably carry on after the error, they may
Hi all,
here is a patch related to type-bound operators, which fixes both PR
49591 and parts of PR 41951 (namely comment #12). The central piece of
the patch is the resolve.c part. This adds all type-bound operator
routines also to the non-typebound operator list (i.e. ns-op). In
this way,
On Wed, Jun 27, 2012 at 2:35 PM, Chiheng Xu chiheng...@gmail.com wrote:
On Wed, Jun 27, 2012 at 2:19 AM, Lawrence Crowl cr...@google.com wrote:
On 6/26/12, Jason Merrill ja...@redhat.com wrote:
On 06/25/2012 06:26 PM, Lawrence Crowl wrote:
+orcodegcc_unreachable/code. If the checks are
Hi Guys,
I am checking in the patch below to the mainline and 4.7 branch
sources to fix a typo in the comparesi3_extend patterns in the rx.md
file. Operand 0 is an input operand but it had an = modifier applied
to it. This confused gcc's internals and resulted in several ICEs in
the
On Tue, Jun 19, 2012 at 6:28 AM, Lawrence Crowl cr...@google.com wrote:
pFunction prototypes for extern functions should only occur in
header files. Functions should be ordered within source files to
minimize the number of function prototypes, by defining them before
@@ -121,13 +208,13 @@
Hi,
On Tue, Jun 26, 2012 at 11:06:15AM -0700, Lawrence Crowl wrote:
On 6/26/12, Martin Jambor mjam...@suse.cz wrote:
On Mon, Jun 25, 2012 at 03:26:01PM -0700, Lawrence Crowl wrote:
I have no idea. I don't use emacs. The two-space rule for
members comes from the wiki. The one-space
Hello,
this patch fixes an ICE on valid code for preprocessor as described in PR 37215
ChangeLog
2012-06-27 Kai Tietz
PR preprocessor/37215
* c-ppoutput.c (preprocess_file): Check for none-empty buffer.
Tested for x86_64-unknown-linux-gnu, and i688-pc-cygwin. Ok for apply?
On Wed, 27 Jun 2012, Kai Tietz wrote:
2012-06-27 Kai Tietz
PR preprocessor/37215
* c-ppoutput.c (preprocess_file): Check for none-empty buffer.
nonempty
Tested for x86_64-unknown-linux-gnu, and i688-pc-cygwin. Ok for apply?
OK with the ChangeLog typo fixed.
--
Joseph
On 06/27/2012 08:35 AM, Chiheng Xu wrote:
dynamic_cast use RTTI, while TREE_CODE are poor man's type info. RTTI
is better than TREE_CODE. But, If you decide to use RTTI, TREE_CODE
become redundant, that means all use of TREE_CODE should be removed,
sooner or later. Are you prepared for that ?
On Fri, Jun 22, 2012 at 9:16 AM, Richard Guenther rguent...@suse.de wrote:
This bumps the requirement to enable Graphite to using cloog 0.17.0
which is the last release from upstream. The patch removes the
support for the legacy cloog versions, too.
I am bootstrapping and testing this now
On 25 June 2012 04:32, Jason Merrill ja...@redhat.com wrote:
On 06/18/2012 09:04 AM, Ramana Radhakrishnan wrote:
+ location_t loc = EXPR_LOC_OR_HERE (t);
We should only use EXPR_LOC_OR_HERE for diagnostics. For a location to use
in building other expressions, use EXPR_LOCATION.
Thanks
Hi Janus,
On 06/27/2012 02:42 PM, Janus Weil wrote:
here is a patch related to type-bound operators, which fixes both PR
49591 and parts of PR 41951 (namely comment #12). The central piece of
the patch is the resolve.c part. This adds all type-bound operator
routines also to the non-typebound
Hi!
This patch makes veclower2 attempt to emit integer division/modulus of
vectors by constants using vector multiplication, shifts or masking.
It is somewhat similar to the vect_recog_divmod_pattern, but it needs
to analyze everything first, see if all divisions or modulos are doable
using the
On 19 June 2012 17:20, Joseph S. Myers jos...@codesourcery.com wrote:
On Mon, 18 Jun 2012, Iain Buclaw wrote:
[PATCH 1/4]:
The D compiler frontend
- gcc/d
Only selectively reviewed, but here are some comments:
diff -Naur gcc-4.8-20120617/gcc/d/asmstmt.cc gcc-4.8/gcc/d/asmstmt.cc
---
Hi!
When testing SVN valgrind AVX support, I've discovered a bug in
this testcase, which has been initializing only half of the array and
performed testing on the second half of the array with uninitialized random
stack data.
Tested on x86_64-linux and i686-linux, committed to trunk.
2012-06-27
Hi,
We'd like to note about CodeSourcery's patch for ARM backend, from which
GCC mainline can gain 4% on SPEC2K INT:
http://cgit.openembedded.org/openembedded/plain/recipes/gcc/gcc-4.5/linaro/gcc-4.5-linaro-r99369.patch
(also the patch is attached).
Originally, we noticed that GNU Go works
On 19 June 2012 17:08, Steven Bosscher stevenb@gmail.com wrote:
Many functions have no leading comment, and other GNU coding standard
requirements are not followed either. Those should IMHO be fixed also,
before this front end can be accepted.
To separate this from the other listed
On Wed, Jun 27, 2012 at 4:58 PM, Dmitry Melnik d...@ispras.ru wrote:
This patch can be applied to current trunk and passes regtest successfully
on qemu-arm.
Maybe it will be good to have it in trunk?
If everybody agrees, we can take care of committing it.
If the patch is approved, can you
On 06/27/2012 01:45 AM, Richard Guenther wrote:
Of course, this fires for normal integer code as well.
Some cases it's a clear win:
-: 41 be 1f 00 00 00 mov$0x1f,%r14d
...
-: 4c 89 f1mov%r14,%rcx
+: b9 1f 00 00 00 mov
On 06/27/2012 10:45 AM, Richard Guenther wrote:
On Wed, Jun 27, 2012 at 5:02 AM, Richard Henderson r...@redhat.com wrote:
sometimes not (increased code size):
-: 41 bd 01 00 00 00 mov$0x1,%r13d
-: 4d 89 ecmov%r13,%r12
+: 41 bc 01 00 00 00
On Wed, 27 Jun 2012, Iain Buclaw wrote:
It's copied as including c-common.c / .h causes problems with a fair
number of references pulled in that need to be stubbed out - also,
some GCC function attributes that we use do not make any sense to have
in D code (eg: gnu_inline, artificial,
On Wed, 27 Jun 2012 18:58:36 +0400
Dmitry Melnik d...@ispras.ru wrote:
This patch can be applied to current trunk and passes regtest
successfully on qemu-arm.
Maybe it will be good to have it in trunk?
If everybody agrees, we can take care of committing it.
No objection from me (as the
On 27/06/12 15:58, Dmitry Melnik wrote:
Hi,
We'd like to note about CodeSourcery's patch for ARM backend, from which
GCC mainline can gain 4% on SPEC2K INT:
http://cgit.openembedded.org/openembedded/plain/recipes/gcc/gcc-4.5/linaro/gcc-4.5-linaro-r99369.patch
(also the patch is
On 27 June 2012 15:58, Dmitry Melnik d...@ispras.ru wrote:
Hi,
We'd like to note about CodeSourcery's patch for ARM backend, from which GCC
mainline can gain 4% on SPEC2K INT:
http://cgit.openembedded.org/openembedded/plain/recipes/gcc/gcc-4.5/linaro/gcc-4.5-linaro-r99369.patch
(also the
On 8 June 2012 10:12, Carrot Wei car...@google.com wrote:
Hi
In rtl expression, substract a constant c is expressed as add a value -c, so
it
is alse processed by adddi3, and I extend it more to handle a subtraction of
64bit constant. I created an insn pattern arm_subdi3_immediate to
May I ask about the purpose of the following piece of change? Doesn't
it affect non-sse cases either?
@@ -32038,7 +32042,15 @@ ix86_rtx_costs (rtx x, int code, int
outer_code_i, int opno, int *total,
case ASHIFTRT:
case LSHIFTRT:
case ROTATERT:
- if (!TARGET_64BIT GET_MODE (XEXP
On 28 May 2012 11:08, Carrot Wei car...@google.com wrote:
Hi
This is the second part of the patches that deals with 64bit and. It directly
extends the patterns anddi3, anddi3_insn and anddi3_neon to handle 64bit
constant operands.
Comments about const_di_ok_for_op still apply from earlier
On 28 May 2012 11:08, Carrot Wei car...@google.com wrote:
Hi
This is the second part of the patches that deals with 64bit and. It directly
extends the patterns anddi3, anddi3_insn and anddi3_neon to handle 64bit
constant operands.
Comments about const_di_ok_for_op still apply from my review
On 27 June 2012 16:47, Joseph S. Myers jos...@codesourcery.com wrote:
On Wed, 27 Jun 2012, Iain Buclaw wrote:
It's copied as including c-common.c / .h causes problems with a fair
number of references pulled in that need to be stubbed out - also,
some GCC function attributes that we use do not
On Wed, 27 Jun 2012, Iain Buclaw wrote:
OK, thanks. As the D frontend goes through a sometimes experimental
development process between each release, I'd rather have it so that I
merge the frontend into GDC as each release happens, instead of
keeping in constant sync. This is how I handle
On 06/27/2012 09:07 AM, Igor Zamyatin wrote:
May I ask about the purpose of the following piece of change? Doesn't
it affect non-sse cases either?
Err, no, it doesn't affect non-sse cases. All MODE_VECTOR_INT
cases will be implemented in the xmm registers (ignoring the
deprecated and largely
On Wed, Jun 27, 2012 at 9:00 PM, Richard Henderson r...@redhat.com wrote:
On 06/27/2012 09:07 AM, Igor Zamyatin wrote:
May I ask about the purpose of the following piece of change? Doesn't
it affect non-sse cases either?
Err, no, it doesn't affect non-sse cases. All MODE_VECTOR_INT
cases
On 06/27/2012 10:08 AM, Igor Zamyatin wrote:
On Wed, Jun 27, 2012 at 9:00 PM, Richard Henderson r...@redhat.com wrote:
On 06/27/2012 09:07 AM, Igor Zamyatin wrote:
May I ask about the purpose of the following piece of change? Doesn't
it affect non-sse cases either?
Err, no, it doesn't
On Wed, Jun 27, 2012 at 9:14 PM, Richard Henderson r...@redhat.com wrote:
On 06/27/2012 10:08 AM, Igor Zamyatin wrote:
On Wed, Jun 27, 2012 at 9:00 PM, Richard Henderson r...@redhat.com wrote:
On 06/27/2012 09:07 AM, Igor Zamyatin wrote:
May I ask about the purpose of the following piece of
2012/6/7 Fabien Chêne fabien.ch...@gmail.com:
[...]
... committed as rev 188294.
I will backport it to 4.7 when it unfreezes.
... Eventually backported as rev 189021.
--
Fabien
On Jun 27, 2012, at 7:45 AM, Iain Buclaw wrote:
I do have a question though, what is available for the transition of
development from git to svn? Other than a lot of ready and getting
used to the various switches and commands on my part.
Why transition? Quite a few people around here use git
On Jun 27, 2012, at 3:33 AM, Matthew Gretton-Dann wrote:
This patch enables the dump-noaddr test to work in out-of-build-tree testing.
* gcc.c-torture/unsorted/dump-noaddr.x: Generate dump files in
tmpdir.
OK?
Ok.
On Tue, Jun 26, 2012 at 10:56 AM, nick clifton ni...@redhat.com wrote:
Hi Matt,
There's also a trivial documentation fix:
[PATCH 1/2] doc: Correct __builtin_arm_tinsr prototype documentation
and a test to exercise the intrinsics:
[PATCH 2/2] arm: add iwMMXt mmx-2.c test
These have
On 06/26/2012 01:54 PM, Alexandre Oliva wrote:
+ track_stack_pointer (dst, src1, src2);
Why does this function return a value then?
r~
Here, we were parsing svoid::svoid wrong. When we've seen a
class-key, an injected-class-name must be considered to name the class,
not the constructor, because the class-key limits lookup to finding type
names. Fixing that corrects the error message for this testcase, and
avoids the ICE.
On Jun 27, 2012, at 2:07 AM, Alexandre Oliva wrote:
Why? We don't demand a working plugin. Indeed, we disable the use of
the plugin if we find a linker that doesn't support it. We just don't
account for the possibility of finding a linker that supports plugins,
but that doesn't support the
Hi,
this patch fixes a testsuite-failure for LLP64 targets.
ChangeLog
2012-06-27 Kai Tietz
* g++.dg/cpp0x/constexpr-52672.C (ul_ptr): Use SIZE_TYPE instead of
hard-coded 'unsigned long'.
Tested for x86_64-w64-mingw32, and x86_64-unknown-linux-gnu. Ok for apply?
Regards,
Kai
Hello,
this patch makes sure that for pe(+)-coff targets always relocations
are allowed in readonly memory.
This fixes for x86_64-w64-mingw32 target some testcases.
ChangeLog
2012-06-27 Kai Tietz
* config/i386/winnt.c (i386_pe_reloc_rw_mask): New function.
*
On Wed, Jun 27, 2012 at 3:33 AM, Matthew Gretton-Dann
matthew.gretton-d...@arm.com wrote:
All,
This patch enables the dump-noaddr test to work in out-of-build-tree
testing.
It does this by making sure that the dump files generated during the
test are created under $tmpdir.
I created a much
Hello,
This small patch moves a couple of #defines into the only files that
use them. En passant xcoff.h (gcc/xcoff.h, *not* the rs6000/xcoff.h)
becomes unused and can be removed.
Tested by building a cross cc1 from x86_64-unknown-linux-gnu to
rs6000-ibm-aix4.3, just to be sure.
OK for trunk?
On 27 June 2012 21:27, Richard Henderson r...@redhat.com wrote:
On 06/20/2012 05:44 AM, Ramana Radhakrishnan wrote:
+ case NEON_DUP:
+ if (TREE_CODE (argp[0]) == INTEGER_CST
+ || TREE_CODE (argp[0]) == REAL_CST)
+ return build_vector_from_val
I was sitting on this patch until I got around to fixing up Jakub's
existing vector divmod code to use it. But seeing as how he's adding
more uses, I think it's better to get it in earlier.
Tested via a patch sent under separate cover that changes
__builtin_alpha_umulh to immediately fold to
* config/alpha/alpha.c (alpha_dimode_u): New.
(alpha_init_builtins): Initialize it, and use it.
(alpha_fold_builtin_cmpbge): Use alpha_dimode_u.
(alpha_fold_builtin_zapnot, alpha_fold_builtin_insxx): Likewise.
(alpha_fold_vector_minmax,
On Wed, Jun 27, 2012 at 11:37 PM, Richard Henderson r...@redhat.com wrote:
* tree.def (MULT_HIGHPART_EXPR): New.
* cfgexpand.c (expand_debug_expr): Ignore it.
* expr.c (expand_expr_real_2): Handle it.
* fold-const.c (int_const_binop_1): Likewise.
* optabs.c
On 06/27/2012 02:42 PM, Steven Bosscher wrote:
Maybe also a bit in doc/generic.texi? Or is this not supposed to be
exposed to the front ends?
I can't imagine it being terribly useful to front ends, but there's
certainly nothing that ought to prevent it. How's this?
r~
diff --git
These scans from gcc.dg/vect/vect-50.c, and others similar to them in
other vect tests, hurt my brain:
/* { dg-final { scan-tree-dump-times Vectorizing an unaligned access 2 vect
{ xfail { vect_no_align } } } } */
/* { dg-final { scan-tree-dump-times Vectorizing an unaligned access 2 vect
{
Hello,
The only user of IFCVT_EXTRA_FIELDS is FR-V -- and it's not even using
the macro to define extra fields...
This patch removes IFCVT_EXTRA_FIELDS and replaces the related
IFCVT_INIT_EXTRA_FIELDS with IFCVT_MACHDEP_INIT.
Bootstrappedtested on x86_64-unknown-linux-gnu, and buildtested with
On 06/27/2012 03:53 PM, Steven Bosscher wrote:
* system.h (IFCVT_EXTRA_FIELDS): Poison.
(IFCVT_INIT_EXTRA_FIELDS): Poison.
* basic-block.h (struct ce_if_block): Remove IFCVT_EXTRA_FIELDS.
* ifcvt.c (find_if_header): Use IFCVT_MACHDEP_INIT instead of
On Jun 27, 2012, at 1:13 PM, Richard Henderson wrote:
Would it be of any use to introduce an .rdata$N section (equivalent
to .data.ro) so that most of the runtime relocations are adjacent,
and more of the executable image is sharable?
I can't help but think that grouping them together would be
On Jun 27, 2012, at 12:21 PM, Kai Tietz wrote:
this patch fixes a testsuite-failure for LLP64 targets.
ChangeLog
2012-06-27 Kai Tietz
* g++.dg/cpp0x/constexpr-52672.C (ul_ptr): Use SIZE_TYPE instead of
hard-coded 'unsigned long'.
Ok for apply?
Ok.
On Jun 27, 2012, at 3:36 PM, Janis Johnson wrote:
These scans from gcc.dg/vect/vect-50.c, and others similar to them in
other vect tests, hurt my brain:
/* { dg-final { scan-tree-dump-times Vectorizing an unaligned access 2
vect { xfail { vect_no_align } } } } */
/* { dg-final {
On Jun 27, 2012, at 1:35 PM, Andrew Pinski wrote:
On Wed, Jun 27, 2012 at 3:33 AM, Matthew Gretton-Dann
matthew.gretton-d...@arm.com wrote:
All,
This patch enables the dump-noaddr test to work in out-of-build-tree
testing.
It does this by making sure that the dump files generated during
On 6/27/12, Martin Jambor mjam...@suse.cz wrote:
On Tue, Jun 26, 2012 at 11:06:15AM -0700, Lawrence Crowl wrote:
On 6/26/12, Martin Jambor mjam...@suse.cz wrote:
On Mon, Jun 25, 2012 at 03:26:01PM -0700, Lawrence Crowl wrote:
+but think twice before using it in code
+intended to last
2012/6/27 Richard Henderson r...@redhat.com:
On 06/27/2012 12:47 PM, Kai Tietz wrote:
2012-06-27 Kai Tietz
* config/i386/winnt.c (i386_pe_reloc_rw_mask): New function.
* config/i386/i386-protos.h (i386_pe_reloc_rw_mask): Add
prototype.
* config/i386/cygming.h
75 matches
Mail list logo