On Mon, Jun 18, 2012 at 10:06 PM, Richard Henderson r...@redhat.com wrote:
Please note that you will probably hit PR33329, this is the reason
that we expand multiplications after reload. Please see [1] for
further explanation. There is gcc.target/i386/pr33329.c test to cover
this issue, but
On Wed, Jun 20, 2012 at 2:21 AM, Hans-Peter Nilsson h...@bitrange.com wrote:
On Tue, 19 Jun 2012, Steven Bosscher wrote:
I've now committed this, see r188791.
Breaking cris-elf. Just try rebuilding cc1:
./gcc/gcc/../libdecnumber/dpd -I../libdecnumber \
I've applied the attached patch which is a tiny SH specific
change of gcc.dg/stack-usage-1.c test. Tested on sh-linux
and i686-pc-linux-gnu.
This is wrong, please remove the dg-options line and do like the other targets.
--
Eric Botcazou
The recent regression in libgomp leads me to want to add libgomp tests
to the check-c++ target. OK for trunk?
commit 3eaa6c5b268115cbf4ab762b5d7b50022389ef25
Author: Jason Merrill ja...@redhat.com
Date: Tue Jun 19 18:16:34 2012 -0700
* Makefile.tpl (check-target-libgomp-c++): New.
*
Eric Botcazou ebotca...@adacore.com wrote:
This is wrong, please remove the dg-options line and do like the other
targets.
I'll revert that line and use my patch in the trail #11 of
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53621.
Regards,
kaz
Applied, thanks. Note that your dg-error regexp doesn't make much sense:
// { dg-error expected type-specifier before 'final'||expected
';'||declaration doesn't declare anything }
Regular expression or uses a single |, so this ends up being a long
way of writing
// { dg-error }
I
On 06/15/2012 02:59 PM, Dominique Dhumieres wrote:
Back when we added C++11 auto deduction, I thought we could shortcut the
normal deduction in some templates, when the type is adequately
describable (thus the late, unlamented function describable_type). Over
time various problems with this
This is wrong, please remove the dg-options line and do like the other
targets.
I'll revert that line and use my patch in the trail #11 of
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53621.
I've applied the patch below. I'll backport it release branches.
Regards,
kaz
--
I've applied the patch below. I'll backport it release branches.
Thanks!
--
Eric Botcazou
On 20 June 2012 10:35, Jason Merrill ja...@redhat.com wrote:
Applied, thanks. Note that your dg-error regexp doesn't make much sense:
// { dg-error expected type-specifier before 'final'||expected
';'||declaration doesn't declare anything }
Regular expression or uses a single |, so this
Hi Michael
It seems the wiki page describes 64bit operations on NEON only. My
patches improves 64bit operations on core registers only. I touched
the neon patterns simply because those DI mode operations are enabled
separately according to the TARGET_NEON value, so in the neon patterns
I
On Tue, 19 Jun 2012, Iain Sandoe wrote:
On 19 Jun 2012, at 22:41, Mike Stump wrote:
On Jun 19, 2012, at 12:22 PM, Iain Sandoe i...@codesourcery.com wrote:
On 19 Jun 2012, at 13:53, Dominique Dhumieres wrote:
On Tue, 19 Jun 2012, Richard Guenther wrote:
Richard Guenther
Did you mean to put a bugzilla link here?
Yes;-( http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53565
copy and paste from the wrong window).
Dominique
On Tue, Jun 19, 2012 at 11:36 PM, Mikael Pettersson mi...@it.uu.se wrote:
Richard Guenther writes:
On Fri, Jun 15, 2012 at 5:00 PM, Ulrich Weigand uweig...@de.ibm.com
wrote:
Richard Guenther wrote:
On Fri, Jun 15, 2012 at 3:13 PM, Ulrich Weigand uweig...@de.ibm.com
wrote:
On Jun 19, 2012, at 6:47 PM, Richard Henderson wrote:
On 2012-06-18 05:22, Tristan Gingold wrote:
+ /* Win64 SEH, very large frames need a frame-pointer as maximum stack
+ allocation is 4GB (add a safety guard for saved registers). */
+ if (TARGET_64BIT_MS_ABI get_frame_size () +
Hello,
Attached is a concept patch to move the C front end to its own
sub-directory of the main gcc directory. Things like updates of
sourcebuild.texi are not yet included.
I'm posting this as an RFC: Does this look like the right approach?
Have I overlooked other things than just documentation
On Wed, Jun 20, 2012 at 9:26 AM, Jason Merrill ja...@redhat.com wrote:
The recent regression in libgomp leads me to want to add libgomp tests to
the check-c++ target. OK for trunk?
Ok. (what about libitm?)
Thanks,
Richard.
Tailcalls have no argument setup cost and no return value cost.
This patch adjusts estminate_num_insns to reflect that.
Honza, does this look correct?
Bootstrapped and tested on x86_64-unknown-linux-gnu.
Thanks,
Richard.
2012-06-20 Richard Guenther rguent...@suse.de
* tree-inline.c
On Tue, 19 Jun 2012, Dominique Dhumieres wrote:
On Tue, 19 Jun 2012, Richard Guenther wrote:
Richard Guenther rguent...@suse.de writes:
We are too eager to bump alignment of some decls when vectorizing.
The fix is to not bump alignment of decls the user explicitely
aligned or
Hello,
m32c-pragma.c doesn't need the includes that the patch below removes.
Tested with a cross from powerpc64-unknown-linux-gnu to m32c-elf.
Will commit as obvious unless someone objects.
Ciao!
Steven
* config/m32c/m32c-pragma.c: Remove unnecessary includes.
Index:
On Sun, Jun 10, 2012 at 4:16 PM, Marc Glisse marc.gli...@inria.fr wrote:
Hello,
currently, tree-ssa-ifcombine handles pairs of imbricated ifs that share
the same then branch, or the same else branch. There is no particular reason
why it couldn't also handle the case where the then branch of
On 19/06/12 23:44, Steven Bosscher wrote:
Hello,
Only a few front-end files to go that need output.h, and some of them
are in the c_target_objs: arm, mep, m32c, and rl78.
This patch tackles the ARM case. arm-c.c needs output.h because
EMIT_EABI_ATTRIBUTE wants to print to asm_out_file.
Hi,
this is a repost of
http://gcc.gnu.org/ml/gcc-patches/2010-10/msg02468.html
earlier in the development cycle, so with hopefully more time for discussion.
The command line option -fcallgraph-info is added and makes the compiler
generate another output file (xxx.ci) for each compilation
On Wed, Jun 20, 2012 at 12:52:12AM -0300, Alexandre Oliva wrote:
On Jun 16, 2012, H.J. Lu hjl.to...@gmail.com wrote:
from Alexandre Oliva aol...@redhat.com
PR debug/53671
PR debug/49888
* alias.c (memrefs_conflict_p): Improve handling of AND for
alignment.
from
On Wed, Jun 20, 2012 at 12:39:29AM -0300, Alexandre Oliva wrote:
When promote_debug_loc was first introduced, it would never be called
with a NULL loc list. However, because of the strategy of temporarily
resetting loc lists before recursion introduced a few months ago in
alias.c, the earlier
On Wed, Jun 20, 2012 at 12:40 PM, Steven Bosscher stevenb@gmail.com wrote:
On Wed, Jun 20, 2012 at 12:30 PM, Eric Botcazou ebotca...@adacore.com wrote:
* cgraph.c: Include expr.h and output.h.
What for?
Never mind, I see why you need this.
Hello,
These macros are now gone and should be poisoned. I'll commit this
later today unless someone objects.
Ciao!
Steven
* system.h: Poison ASM_OUTPUT_IDENT and IDENT_ASM_OP.
Index: system.h
===
--- system.h(revision
Hello!
2012-06-20 Uros Bizjak ubiz...@gmail.com
* config/i386/i386.md (rounding_insn): New int attribute.
(rounding_insnxf2): Macroize insn from
{floor,ceil,btrunc}xf2 using FRNDINT_ROUNDING int iterator.
(lrounding_insnxfmode2): Rename from lroundingxfmode2.
On Thu, Jun 14, 2012 at 3:21 PM, William J. Schmidt
wschm...@linux.vnet.ibm.com wrote:
Pro forma ping. :)
;)
I notice (with all of these functions)
+unsigned
+negate_cost (enum machine_mode mode, bool speed)
+{
+ static unsigned costs[NUM_MACHINE_MODES];
+ rtx seq;
+ unsigned cost;
+
+ if
On Wed, Jun 20, 2012 at 12:30 PM, Eric Botcazou ebotca...@adacore.com wrote:
Hi,
this is a repost of
http://gcc.gnu.org/ml/gcc-patches/2010-10/msg02468.html
earlier in the development cycle, so with hopefully more time for discussion.
The command line option -fcallgraph-info is added and
On Wed, 20 Jun 2012 11:56:39 +0100
Ramana Radhakrishnan ramana.radhakrish...@linaro.org wrote:
Hi,
This patch helps use the __builtin_shuffle intrinsics to implement the
Neon permute intrinsics following on from Julian's and my patch last
week. It needed support for __builtin_shuffle in the
Richard Guenther writes:
On Tue, Jun 19, 2012 at 11:36 PM, Mikael Pettersson mi...@it.uu.se wrote:
Richard Guenther writes:
On Fri, Jun 15, 2012 at 5:00 PM, Ulrich Weigand uweig...@de.ibm.com
wrote:
Richard Guenther wrote:
On Fri, Jun 15, 2012 at 3:13 PM, Ulrich Weigand
This concludes the VRP and anti-ranges series for now (well, it was
the motivation for this patch which was pending for quite some time).
This re-implements PLUS_EXPR support on integer ranges to cover
all cases, even those that generate an anti-range as result.
Bootstrapped and tested on
Richard Guenther wrote:
In this testcase the alignment of arr[i] should be irrelevant - it is
not part of the stmts that are going to be vectorized.
Agreed.
But of course this may be
simply an odering issue in how we analyze data-references / statements
in basic-block vectorization (thus
On Wed, Jun 20, 2012 at 4:44 AM, Steven Bosscher stevenb@gmail.com wrote:
I'm posting this as an RFC: Does this look like the right approach?
Have I overlooked other things than just documentation updates? I hope
this would not cause too much trouble for branches like the
cxx-conversion
Hi,
ChangeLog
2012-06-20 Kai Tietz
* gcc.target/i386/pr23943.c (size_t): Use compatible type-definition
for LLP64 targets.
* gcc.target/i386/pr38988.c: Likewise.
Regression-tested for x86_64-w64-mingw32, and x86_64-unknown-linux-gnu.
Ok for apply?
Regards,
Kai
On Mon, Jun 18, 2012 at 10:09 AM, Kai Tietz ktiet...@googlemail.com wrote:
2012/6/18 JonY jo...@users.sourceforge.net:
Hi,
I am told that this ABI test does not apply to mingw targets. OK to apply?
Hi JonY,
The test doesn't apply to x64 windows targets, as for it sse is part of its
ABI.
As both tests are checking already for !ia32, there is no additiona
check beside the targets necessary.
Cheers,
Kai
Hi ,
This improves the vdup_n intrinsics where one tries to form constant
vectors. This uses targetm.fold_builtin to fold these vector
initializations to actual vector constants. The vdup_n cases are fine
with both endian-ness as the vector constant is just duplicated. In
addition I've made the
Alan and I both re-implemented the locks and settled on the following
patch. This uses the __atomic intrinsics, not the __sync instrinsics,
to avoid generating expensive instructions for a memory model that is
stricter than necessary.
If these intrinsics correctly represent the semantics of the
On Wed, Jun 20, 2012 at 09:10:44AM -0400, David Edelsohn wrote:
inline static void
release_set (volatile obj_addr_t *addr, obj_addr_t new_val)
{
- __asm__ __volatile__ (sync : : : memory);
- *addr = new_val;
+ __atomic_store_n(addr, val, __ATOMIC_RELEASE);
A typo seems to have crept
On 06.06.2012 11:00, Ramana Radhakrishnan wrote:
Ok with those changes. Ramana .
Hi Ramana,
How about this version?
Christophe.
commit f57ce4b63ca1c30ee88e8c1a431d6e90ffbecb82
Author: Christophe Lyon christophe.l...@st.com
Date: Wed Jun 20 15:30:50 2012 +0200
2012-06-20 Christophe
Hi,
On 20 Jun 2012, at 09:23, Richard Guenther wrote:
On Tue, 19 Jun 2012, Iain Sandoe wrote:
On 19 Jun 2012, at 22:41, Mike Stump wrote:
On Jun 19, 2012, at 12:22 PM, Iain Sandoe i...@codesourcery.com wrote:
On 19 Jun 2012, at 13:53, Dominique Dhumieres wrote:
On Tue, 19 Jun 2012,
On Wed, Jun 20, 2012 at 9:35 AM, Alan Modra amo...@gmail.com wrote:
On Wed, Jun 20, 2012 at 09:10:44AM -0400, David Edelsohn wrote:
inline static void
release_set (volatile obj_addr_t *addr, obj_addr_t new_val)
{
- __asm__ __volatile__ (sync : : : memory);
- *addr = new_val;
+
dnovi...@google.com (Diego Novillo) writes:
Ian, could you please take a look to double check I have not missed
anything? There was more code dealing with it than I was expecting.
It all looks plausible to me.
Ian
On Wed, 20 Jun 2012, Richard Guenther wrote:
On Wed, 20 Jun 2012, Iain Sandoe wrote:
Hi,
On 20 Jun 2012, at 09:23, Richard Guenther wrote:
On Tue, 19 Jun 2012, Iain Sandoe wrote:
On 19 Jun 2012, at 22:41, Mike Stump wrote:
On Jun 19, 2012, at 12:22 PM, Iain
On Wed, Jun 20, 2012 at 1:08 AM, Diego Novillo dnovi...@google.com wrote:
diff --git a/configure.ac b/configure.ac
index 071b5e2..2a2a0c6 100644
--- a/configure.ac
+++ b/configure.ac
@@ -1667,7 +1653,7 @@ ACX_ELF_TARGET_IFELSE([# ELF platforms build the
lto-plugin always.
])
-# By
On Wed, Jun 20, 2012 at 4:10 PM, Steven Bosscher stevenb@gmail.com wrote:
On Wed, Jun 20, 2012 at 1:08 AM, Diego Novillo dnovi...@google.com wrote:
diff --git a/configure.ac b/configure.ac
index 071b5e2..2a2a0c6 100644
--- a/configure.ac
+++ b/configure.ac
@@ -1667,7 +1653,7 @@
On 06/13/2012 02:51 PM, Richard Earnshaw wrote:
This patch is the first of a series to remove support for the now
obsolete FPA and Maverick co-processors.
This patch removes those targets and configuration options that were
marked as deprecated in GCC-4.7 and removes the config fragments that
Hi,
On Tue, 19 Jun 2012, Richard Guenther wrote:
The MEM_REF is acceptable to the tree oracle and it can extract
points-to information from it.
Thus for simplicity unconditionally building the above is the best.
But it doesn't work, as refs_may_alias_p_1 only accepts certain operands
in
On Wed, Jun 20, 2012 at 4:57 PM, Michael Matz m...@suse.de wrote:
Hi,
On Tue, 19 Jun 2012, Richard Guenther wrote:
The MEM_REF is acceptable to the tree oracle and it can extract
points-to information from it.
Thus for simplicity unconditionally building the above is the best.
But it
On 12-06-20 10:10 , Steven Bosscher wrote:
-# By default, C is the only stage 1 language.
+# By default, C and C++ are the only stage 1 languages.
stage1_languages=,c,
So shouldn't you add c++ here?
That was a bad change on the comment. We only need C for stage1.
Thanks for spotting it.
Hi,
On Wed, 20 Jun 2012, Richard Guenther wrote:
+ exp = fold_build2 (MEM_REF,
+ build_array_type (char_type_node,
+ build_range_type (sizetype,
+ size_one_node,
len)),
+
On Wed, Jun 20, 2012 at 5:09 PM, Michael Matz m...@suse.de wrote:
Hi,
On Wed, 20 Jun 2012, Richard Guenther wrote:
+ exp = fold_build2 (MEM_REF,
+ build_array_type (char_type_node,
+ build_range_type (sizetype,
+
On Wed, 2012-06-20 at 13:11 +0200, Richard Guenther wrote:
On Thu, Jun 14, 2012 at 3:21 PM, William J. Schmidt
wschm...@linux.vnet.ibm.com wrote:
Pro forma ping. :)
;)
I notice (with all of these functions)
+unsigned
+negate_cost (enum machine_mode mode, bool speed)
+{
+ static
On 06/20/2012 12:57 AM, Ville Voutilainen wrote:
If a single pipe is indeed to be used, perhaps we want to correct that
piece of documentation, lest fools follow its advice. :)
Done, thanks.
Jason
On Mon, Jun 18, 2012 at 9:34 AM, Roland McGrath mcgra...@google.com wrote:
OK then. If you like the original patch, would you like to commit it for me?
ping?
On 20/06/12 15:41, Sebastian Huber wrote:
On 06/13/2012 02:51 PM, Richard Earnshaw wrote:
This patch is the first of a series to remove support for the now
obsolete FPA and Maverick co-processors.
This patch removes those targets and configuration options that were
marked as deprecated in
Hello!
2012-06-20 Uros Bizjak ubiz...@gmail.com
* config/i386/i386.md (rounding_insnmode2): Macroize expander
from {floor,ceil,btrunc}mode2 using FIST_ROUNDING int iterator.
(lrounding_insnMODEF:modeSWI48:mode2): Macroize expander
from
On 12-06-20 13:43 , Rüdiger Sonderfeld wrote:
The patch is extremely trivial and probably doesn't need copyright assignment.
However I have signed copyright assignment for Emacs and maybe that will work
too (not sure if this has to be signed for every project).
It does, unfortunately.
On 12-06-20 13:50 , Diego Novillo wrote:
OK. I suppose you do not have write to the repo, so I will commit it
for you.
Committed r188841.
Diego.
On 06/20/2012 04:11 AM, Richard Guenther wrote:
I notice (with all of these functions)
+unsigned
+negate_cost (enum machine_mode mode, bool speed)
+{
+ static unsigned costs[NUM_MACHINE_MODES];
+ rtx seq;
+ unsigned cost;
+
+ if (costs[mode])
+return costs[mode];
+
+
On Wed, 20 Jun 2012, Richard Guenther wrote:
On Sun, Jun 10, 2012 at 4:16 PM, Marc Glisse marc.gli...@inria.fr wrote:
Hello,
currently, tree-ssa-ifcombine handles pairs of imbricated ifs that share
the same then branch, or the same else branch. There is no particular reason
why it couldn't
Hello!
2012-06-20 Uros Bizjak ubiz...@gmail.com
* config/i386/i386.md (SINCOS): New int iterator.
(sincos): New int attribute.
(*sincosxf2_i387): Macroize insn from *{sin,cos}xf2_i387 using
SINCOS int iterator.
(*sincos_extendmodexf2_i387): Macroize insn
Hi,
A few weeks ago I submitted a fix for a garbage collection issue I ran
into involving -Wunused-local-typedefs [1]. The analysis for that patch
still stands, but unfortunately the patch is wrong.
The problem is that the allocation reuse can't be removed otherwise the
information about local
On Wed, 2012-06-20 at 11:52 -0700, Richard Henderson wrote:
On 06/20/2012 04:11 AM, Richard Guenther wrote:
I notice (with all of these functions)
+unsigned
+negate_cost (enum machine_mode mode, bool speed)
+{
+ static unsigned costs[NUM_MACHINE_MODES];
+ rtx seq;
+ unsigned
This patch addresses the problem of building GCC for mips with the
'--with-synci' configure option. If you do that and then compile a
program with GCC and specify an architecture that does not support synci
(such as the -mips32 option), GCC will issue a warning that synci
is not supported. This
Hi,
the attached patch makes the FTELL intrinsic function work on offsets
larger than 2 GB on 32-bit systems that support large files. As this
is an ABI change the old library function is left untouched, to be
removed when/if the library ABI is bumped.
Regtested on x86_64-unknown-linux-gnu, Ok
+ /* If we're putting types in their own .debug_types sections,
+ the .debug_pubtypes table will still point to the compile
+ unit (not the type unit), so we want to use the offset of
+ the skeleton DIE (if there is one). */
+ if
One of the most common runtime errors we have seen in gcc-4_7 is
caused by dangling references to temporaries whole life time have
ended
e.g,
const A a = foo();
or
foo (A());// where temp's address is saved and used after foo.
Of course this is user error according to the standard, triaging
Applied.
Gerald
Index: gcc-4.7/changes.html
===
RCS file: /cvs/gcc/wwwdocs/htdocs/gcc-4.7/changes.html,v
retrieving revision 1.119
diff -u -3 -p -r1.119 changes.html
--- gcc-4.7/changes.html14 Jun 2012 17:57:18 -
On Jun 20, 2012, Jakub Jelinek ja...@redhat.com wrote:
On Wed, Jun 20, 2012 at 12:39:29AM -0300, Alexandre Oliva wrote:
When promote_debug_loc was first introduced, it would never be called
with a NULL loc list. However, because of the strategy of temporarily
resetting loc lists before
On 10 April 2012 10:11, Ramana Radhakrishnan
ramana.radhakrish...@linaro.org wrote:
The patch with correct configure output is ok.
Thanks - this is what I committed.
Is this something that can be considered for backporting to release
branches ? This patch technically doesn't fix a regression
OK.
Jason
Lawrence,
you ask a number of awfully good questions. :-)
First of all you made me realize that we were missing a
cross-link from http://gcc.gnu.org/projects/web.html to
http://gcc.gnu.org/contribute.html#webchanges which the
first patch included below does now.
On Tue, 5 Jun 2012, Lawrence
The documentation needs to explain more what the option controls, and
why you might want it on or off. Other than that it looks fine.
Jason
Hi,
This patch creates basic gimple_assign statements. It is a little raw
not considering all types of gimple_assign statements for which I have
already started working.
Here is the Changelog.
2012-06-09 Sandeep Soni soni.sande...@gmail.com
* parser.c (gimple_symtab_get): New.
I modified the documentation and it now looks like this:
@item -ftemp-stack-reuse
@opindex ftemp_stack_reuse
This option enables stack space reuse for temporaries. The default is on.
The lifetime of a compiler generated temporary is well defined by the C++
standard. When a lifetime of a temporary
On Jun 20, 2012, at 12:26 AM, Jason Merrill ja...@redhat.com wrote:
The recent regression in libgomp leads me to want to add libgomp tests to the
check-c++ target.
I'm fine with the idea...
79 matches
Mail list logo