On Thu, Mar 15, 2012 at 22:28, Janne Blomqvist
blomqvist.ja...@gmail.com wrote:
On Thu, Mar 15, 2012 at 22:14, Tobias Burnus bur...@net-b.de wrote:
Janne Blomqvist wrote:
since some time GCC has BUILT_IN_IROUND{F,,L}, similar to lround() and
llround() but the result is returned as an integer.
Hi,
all the three VMS entries are consolidated into one with this patch.
Committed on trunk.
Tristan.
2012-03-16 Tristan Gingold ging...@adacore.com
* config.host (alpha64-dec-*vms*, alpha*-dec-*vms*)
(ia64-hp-*vms*): Replaced by ...
(*-*-*vms*): ... This new entry.
Hi,
this patch makes gcc provide the C99 compliant stdint.h header.
(required by libquadmath).
Committed on trunk.
Tristan.
2012-03-16 Tristan Gingold ging...@adacore.com
* config.gcc (*-*-*vms*): Define use_gcc_stdint and tm_file.
* config/vms/vms-stdint.h: New file.
On Thu, 15 Mar 2012, Rainer Orth wrote:
Gerald Pfeifer ger...@pfeifer.com writes:
On Mon, 12 Mar 2012, Rainer Orth wrote:
Tested with make doc/gccinstall.info doc/gccinstall.pdf, ok for mainline
and 4.7 branch?
+Sun does not ship a C compiler with Solaris 2 before Solaris 10, though
On Thu, 15 Mar 2012, Eric Botcazou wrote:
Computing the offset in stor-layout.c and storing it in DECL_INITIAL?
Ugh. I just realized that the DECL_BIT_FIELD_REPRESENTATIVE is built during
layout... but is overloaded with DECL_QUALIFIER. That's probably the source
of the miscompilation
Noticed during the VECTOR_CST representation change.
Bootstrapped and tested on x86_64-unknown-linux-gnu, applied to trunk.
Richard.
2012-03-16 Richard Guenther rguent...@suse.de
* tree-vect-loop.c (get_initial_def_for_induction): Use
build_constructor directly.
*
Hi,
these two macros are used by system headers to enable some features.
If they aren't defined, the headers assume the system is the most recent one,
and may use some functions that aren't present in the libraries.
Let's be conservative for cross compiler (and even for native one).
Committed
On Thu, Mar 15, 2012 at 08:35:47PM +0100, Jakub Jelinek wrote:
On Thu, Mar 15, 2012 at 05:56:32PM +0100, Bernhard Reutner-Fischer wrote:
On Thu, Mar 15, 2012 at 04:57:12PM +0100, Richard Guenther wrote:
On Thu, Mar 15, 2012 at 1:39 PM, Bernhard Reutner-Fischer
rep.dot@gmail.com wrote:
On Fri, Mar 16, 2012 at 1:13 AM, Andrew Pinski pins...@gmail.com wrote:
On Thu, Mar 15, 2012 at 5:09 PM, Bernd Schmidt ber...@codesourcery.com
wrote:
On 03/16/2012 12:44 AM, Jakub Jelinek wrote:
For pointer arithmetics in the IL we assume the C
requirements, pointer arithmetics can be
On Fri, Mar 16, 2012 at 1:29 AM, Jonathan Wakely jwakely@gmail.com wrote:
On 15 March 2012 15:40, Richard Guenther wrote:
On Thu, Mar 15, 2012 at 4:22 PM, Kai Tietz ktiet...@googlemail.com wrote:
Richard,
ping. I think now could be a good time for applying the patch you
have for this
Hi,
currently sizetype precision (cf store-layout.c:initialize_sizetypes) is the
same as size_t.
This is an issue on VMS, where size_t is 'unsigned int', but we'd like to have
a 64 bit sizetype
for Ada. My understanding is that ISO-C doesn't require size_t precision to
match the one of
void
Hi,
use of 'long' in vms-unwind.h for registers isn't correct as 'long' should be a
32 bit type on vms.
Let replace it with unw_reg, now defined as unsigned __int64.
Manually tested on ia64-hp-openvms.
Committed on trunk.
Tristan.
libgcc/
2012-03-16 Tristan Gingold ging...@adacore.com
On Fri, Mar 16, 2012 at 11:39 AM, Tristan Gingold ging...@adacore.com wrote:
Hi,
currently sizetype precision (cf store-layout.c:initialize_sizetypes) is the
same as size_t.
This is an issue on VMS, where size_t is 'unsigned int', but we'd like to
have a 64 bit sizetype
for Ada. My
On Fri, Mar 16, 2012 at 9:17 AM, Janne Blomqvist
blomqvist.ja...@gmail.com wrote:
On Thu, Mar 15, 2012 at 22:28, Janne Blomqvist
blomqvist.ja...@gmail.com wrote:
On Thu, Mar 15, 2012 at 22:14, Tobias Burnus bur...@net-b.de wrote:
Janne Blomqvist wrote:
since some time GCC has
On Mar 16, 2012, at 12:02 PM, Richard Guenther wrote:
On Fri, Mar 16, 2012 at 11:39 AM, Tristan Gingold ging...@adacore.com wrote:
Hi,
currently sizetype precision (cf store-layout.c:initialize_sizetypes) is the
same as size_t.
This is an issue on VMS, where size_t is 'unsigned int', but
On Thu, Mar 15, 2012 at 3:34 PM, Kai Tietz ktiet...@googlemail.com wrote:
2012/3/15 Richard Guenther richard.guent...@gmail.com:
On Thu, Mar 15, 2012 at 3:00 PM, Jakub Jelinek ja...@redhat.com wrote:
On Thu, Mar 15, 2012 at 02:53:10PM +0100, Kai Tietz wrote:
This looks like to match unbound
On Fri, Mar 16, 2012 at 12:33 PM, Tristan Gingold ging...@adacore.com wrote:
On Mar 16, 2012, at 12:02 PM, Richard Guenther wrote:
On Fri, Mar 16, 2012 at 11:39 AM, Tristan Gingold ging...@adacore.com
wrote:
Hi,
currently sizetype precision (cf store-layout.c:initialize_sizetypes) is
This fixes issue No. 1, easy.
Bootstrapped and tested on x86_64-unknown-linux-gnu, applied to trunk.
Richard.
2012-03-16 Richard Guenther rguent...@suse.de
* stor-layout.c (finish_bitfield_representative): Fall back
to the conservative maximum size if we cannot compute the
Hi,
I wanted to use 32 bit jump tables for VMS on ia64 (just to reduce executable
size), as
we know 64 offsets aren't necessary.
I override CASE_VECTOR_MODE in vms.h, I had to adjust ASM_OUTPUT_ADDR_DIFF_ELT.
I also adjust ADDR_VEC_ALIGN to reduce the alignment in the SImode case.
But this
On Mar 16, 2012, at 12:38 PM, Richard Guenther wrote:
On Fri, Mar 16, 2012 at 12:33 PM, Tristan Gingold ging...@adacore.com wrote:
On Mar 16, 2012, at 12:02 PM, Richard Guenther wrote:
On Fri, Mar 16, 2012 at 11:39 AM, Tristan Gingold ging...@adacore.com
wrote:
Hi,
currently
On 03/16/2012 11:08 AM, Richard Guenther wrote:
Your patch as-is is not safe at the moment. But why is casting a pointer
to an integer prohibitly expensive? That fact should be an implementation
detail of GIMPLE. Or is it the issue that IVOPTs chooses an integer
type that does not
At least generic vector lowering can end up using BIT_FIELD_REF
to generate sub-vector selects such as low/high part of vector
constants/constructors. We do not fold these currently, leading
to quite absymal code. Thus the following patch which teaches
fold to do this.
Bootstrap and regtest
This makes use of native_encode/interpret_expr to do $subject.
Pretty straight-forward and something value-numbering already
does. Not sure what a good upper bound for the encode buffer
size should be - I suppose at some point we should teach
encode to skip uninteresting start bytes (and teach
On Fri, Mar 16, 2012 at 1:36 PM, Bernd Schmidt ber...@codesourcery.com wrote:
On 03/16/2012 11:08 AM, Richard Guenther wrote:
Your patch as-is is not safe at the moment. But why is casting a pointer
to an integer prohibitly expensive? That fact should be an implementation
detail of GIMPLE.
On 03/16/2012 01:55 PM, Richard Guenther wrote:
On Fri, Mar 16, 2012 at 1:36 PM, Bernd Schmidt ber...@codesourcery.com
wrote:
The machine is special. Pointer addition is a different operation than
integer addition. It'll also need a new ptr_plus rtx code which takes a
Pmode and an SImode
On Fri, Mar 16, 2012 at 2:05 PM, Bernd Schmidt ber...@codesourcery.com wrote:
On 03/16/2012 01:55 PM, Richard Guenther wrote:
On Fri, Mar 16, 2012 at 1:36 PM, Bernd Schmidt ber...@codesourcery.com
wrote:
The machine is special. Pointer addition is a different operation than
integer
With recent disturbed testing this FAIL got unnoticed. Fixed as follows
(we have a constantthat we'd like to replicate in a vector).
Bootstrapped and tested on x86_64-unknown-linux-gnu, applied to trunk.
Richard.
2012-03-16 Richard Guenther rguent...@suse.de
*
On 03/16/2012 02:15 PM, Richard Guenther wrote:
Hmm, ok. So to have your lea represented on RTL you cannot use
a fancy pattern using add because it would not be an add. Hmm.
If it merely does not affect upper bits then
(set (reg:SI ptr) (add (reg:SI ptr) (reg:SI off))
might work (if
On Fri, Mar 16, 2012 at 2:18 PM, Bernd Schmidt ber...@codesourcery.com wrote:
On 03/16/2012 02:15 PM, Richard Guenther wrote:
Hmm, ok. So to have your lea represented on RTL you cannot use
a fancy pattern using add because it would not be an add. Hmm.
If it merely does not affect upper bits
Hi,
I noticed a bug in the __ARM_EABI_UNWINDER__ case that shows up
as a loop in backtrace():
=== Backtrace: =
/lib/libc.so.6[0x4c8c11cc]
/lib/libc.so.6[0x4c8c62a8]
/lib/libc.so.6(cfree+0x38)[0x4c8c63a8]
./bt(_Z6nqueenPiii+0xf0)[0xa8e0]
./bt(_Z6nqueenPiii+0xf0)[0xa8e0]
Sorry this is a resend with corrected email addresses,
I noticed a bug in the __ARM_EABI_UNWINDER__ case that shows up
as a loop in backtrace():
=== Backtrace: =
/lib/libc.so.6[0x4c8c11cc]
/lib/libc.so.6[0x4c8c62a8]
/lib/libc.so.6(cfree+0x38)[0x4c8c63a8]
Yes, that could work. ?Though it might end up being incredibly ugly ;)
In the code? Should really only change a few lines in the patch I would
have thought. I'll get back to you when I have a new version - thanks
for the tip.
No, in the GIMPLE IL.
Richard.
And I can just imagine
On 03/16/2012 02:33 PM, Zdenek Dvorak wrote:
Yes, that could work. ?Though it might end up being incredibly ugly ;)
In the code? Should really only change a few lines in the patch I would
have thought. I'll get back to you when I have a new version - thanks
for the tip.
No, in the GIMPLE
On Fri, Mar 16, 2012 at 2:37 PM, Bernd Schmidt ber...@codesourcery.com wrote:
On 03/16/2012 02:33 PM, Zdenek Dvorak wrote:
Yes, that could work. ?Though it might end up being incredibly ugly ;)
In the code? Should really only change a few lines in the patch I would
have thought. I'll get back
On 03/16/2012 02:46 PM, Richard Guenther wrote:
We do have three pointer offsetting operations at the moment,
POINTER_PLUS_EXPR, MEM_REF and TARGET_MEM_REF.
All of which expect C pointer semantics (if they can see the base
pointer, which is why I suggested to create a TARGET_MEM_REF
with a
On Wed, 14 Mar 2012, Martin Jambor wrote:
Hi,
On Mon, Mar 12, 2012 at 03:09:04PM +0100, Martin Jambor wrote:
Hi,
this patch is very similar to the one I posted before
(http://gcc.gnu.org/ml/gcc-patches/2012-02/msg01377.html) except that
it is now adjusted to sit on top of the new
2012/3/16 Paweł Sikora:
Hi,
during gcc build process there's some sed magic
(libstdc++-v3/include/Makefile.{am,in}) in action
which modifies libstdc++/include/bits/c++config.h (replaces '#define
_GLIBCXX_EXTERN_TEMPLATE'
according to --enable-extern-template=yes/no settings). in fact,
While reading through the GCC 4.7 changes.html, I noticed a couple of
nits that this patch fixes. It's mostly wording and grammar changes (so
I may well be wrong and would appreciate a native speaker having a look ;-)
Ok to install?
Beyond what I've changed, two items could perhaps do with
On Fri, Mar 16, 2012 at 11:04:45AM +0100, Bernhard Reutner-Fischer wrote:
The underlying problem is that dejagnu's runtest.exp only allows for a
single libdir where it searches for includes -- see comment in
libgomp.exp and libitm.exp
While just adding more and more load_gcc_lib calls to users
On 03/16/12 04:56, Tristan Gingold wrote:
2012-03-16 Tristan Gingold ging...@adacore.com
* config/ia64/vms.h (CASE_VECTOR_MODE): Define.
* config/ia64/ia64.md: Remove mode in template.
Sign extend operand in expand_simple_binop.
* config/ia64/ia64.h
On 03/16/2012 03:29 PM, Jonathan Wakely wrote:
Apart from that the patch looks good, I've CC'd gcc-patches, could you
provide a changelog entry?
If this kind of change to the makefile turns out to be applied I think
should go essentially everywhere, at least 4.6 and 4.7(.1) besides mainline.
On 2012/03/16 02:07:02, Cary wrote:
For google/gcc-4_6 branch.
This patch fixes several problems with -gfission:
- Bad index for range list in the compile unit DIE.
- DW_AT_ranges attribute for compile unit in the wrong file.
- Incorrect size for skeleton type unit DIEs.
- Wrote
On 03/16/2012 02:46 PM, Richard Guenther wrote:
In the end what we want is a POINTER_PLUS_EXPR variant
that does not make alias-analysis assume the result still points
to within the objects the pointer pointed to before the increment/decrement.
Hold on, is alias analysis really affected by
On Friday 16 of March 2012 14:29:21 Jonathan Wakely wrote:
2012/3/16 Paweł Sikora:
Hi,
during gcc build process there's some sed magic
(libstdc++-v3/include/Makefile.{am,in}) in action
which modifies libstdc++/include/bits/c++config.h (replaces '#define
_GLIBCXX_EXTERN_TEMPLATE'
Hello,
On 03/16/2012 02:46 PM, Richard Guenther wrote:
In the end what we want is a POINTER_PLUS_EXPR variant
that does not make alias-analysis assume the result still points
to within the objects the pointer pointed to before the increment/decrement.
Hold on, is alias analysis really
Hi,
On Mon, Mar 12, 2012 at 11:51:05AM +0100, Richard Guenther wrote:
On Thu, Mar 8, 2012 at 12:18 PM, Jakub Jelinek ja...@redhat.com wrote:
On Thu, Mar 08, 2012 at 12:06:46PM +0100, Martin Jambor wrote:
/* For local statics lookup proper context die. */
- if (TREE_STATIC
On Fri, Mar 16, 2012 at 4:49 PM, Zdenek Dvorak rakd...@iuuk.mff.cuni.cz wrote:
Hello,
On 03/16/2012 02:46 PM, Richard Guenther wrote:
In the end what we want is a POINTER_PLUS_EXPR variant
that does not make alias-analysis assume the result still points
to within the objects the pointer
ok for google-46 after the minor changes below. Make sure default lipo
testing is well covered too.
David
http://codereview.appspot.com/5746044/diff/11001/cgraph.c
File cgraph.c (right):
http://codereview.appspot.com/5746044/diff/11001/cgraph.c#newcode2887
cgraph.c:2887:
The comment before the hunk I moved wasn't helpful, so I moved the
flag_section_anchor check in error. Fixed by reverting that change
and clarifying the comment.
Richard.
2012-03-16 Richard Guenther rguent...@suse.de
PR tree-optimization/52603
* tree-vect-data-refs.c
I'd like to re-ping this patch backport:
http://gcc.gnu.org/ml/gcc-patches/2012-02/msg01428.html
Peter
2012/3/16 Paweł Sikora:
Should the addition be \$$ to escape it for the shell as well as for make?
(I know it works, but that might not be true for all shells.)
i don't think that $, could be expaneded by any shell.
I'm not worried about it not expanding, but rather being rejected as
invalid
One spelling nit.
May also want to explain the 'ripa' term more clearly somewhere.
http://codereview.appspot.com/5746044/diff/11001/ipa.c
File ipa.c (right):
http://codereview.appspot.com/5746044/diff/11001/ipa.c#newcode811
ipa.c:811: (2) the VNODE is neede, and
s/neede/needed/
On Fri, 16 Mar 2012, Rainer Orth wrote:
While reading through the GCC 4.7 changes.html, I noticed a couple of
nits that this patch fixes. It's mostly wording and grammar changes (so
I may well be wrong and would appreciate a native speaker having a look
This looks good in general, thanks.
Rainer == Rainer Orth r...@cebitec.uni-bielefeld.de writes:
Rainer Continuing with the patches to remove obsolete ports, here's the
Rainer IRIX 6.5 removal patch.
Rainer * libjava/gnu/classpath/natSystemProperties.cc (getpwuid_adaptor) claims
Rainer to be for IRIX, too, but I cannot tell for
Hi,
On Fri, Mar 16, 2012 at 05:14:38PM +0100, Martin Jambor wrote:
On Mon, Mar 12, 2012 at 11:51:05AM +0100, Richard Guenther wrote:
On Thu, Mar 8, 2012 at 12:18 PM, Jakub Jelinek ja...@redhat.com wrote:
On Thu, Mar 08, 2012 at 12:06:46PM +0100, Martin Jambor wrote:
/* For local
On 03/16/12 05:36, Bernd Schmidt wrote:
The machine is special. Pointer addition is a different operation than
integer addition. It'll also need a new ptr_plus rtx code which takes a
Pmode and an SImode operand. Pmode is larger than SImode but fits in a
single register; intptr_t (which is what
Attached patch applied.
2012-03-16 François Dumont fdum...@gcc.gnu.org
PR libstdc++/52476
* include/bits/hashtable.h (_Hashtable::_M_rehash_aux): Add.
(_Hashtable::_M_rehash): Use the latter.
* testsuite/23_containers/unordered_multimap/insert/52476.cc: New.
I am pinging this post:
http://gcc.gnu.org/ml/gcc-patches/2012-03/msg00430.html
Thanks
Edmar
On 03/15/12 12:29, Tom Tromey wrote:
2012-03-15 Tom Tromey tro...@redhat.com
* dwarf2out.c (dwarf_stack_op_name): Use get_DW_OP_name.
(dwarf_tag_name): Use get_DW_TAG_name.
(dwarf_attr_name): Use get_DW_AT_name.
(dwarf_form_name): Use get_DW_FORM_name.
*
Hi,
this is what I did to add the warning requested in the PR. Among the
slightly less trivial bits:
1- In order not to risk warning *before* an hard error is produced by
build_*_cast_1 code, I'm warning only if the cast is actually
successful, thus at the end of each build_*_cast function.
This removes some wrong code.
Ok?
Index: gcc/emit-rtl.c
===
--- gcc/emit-rtl.c (revision 184563)
+++ gcc/emit-rtl.c (working copy)
@@ -540,8 +540,6 @@ immed_double_const (HOST_WIDE_INT i0, HO
if (GET_MODE_BITSIZE
On Fri, Mar 16, 2012 at 10:54 PM, Mike Stump mikest...@comcast.net wrote:
This removes some wrong code.
Ok?
ChangeLog is missing. Tested how?
And why is this wrong anyway?
Ciao!
Steven
Index: gcc/emit-rtl.c
===
---
Ping - now that stage 1 is open, could someone review?
Thanks,
Teresa
On Sun, Dec 4, 2011 at 10:26 PM, Teresa Johnson tejohn...@google.com wrote:
Latest patch which improves the efficiency as described below is
included here. Boostrapped and checked again with
x86_64-unknown-linux-gnu. Could
Hello Everyone,
This patch is for the Cilkplus branch, mainly affecting the C++ compiler.
This patch will fix a bug when spawn is used inside template functions.
Thanks,
Balaji V. Iyer.
diff --git a/gcc/ChangeLog.cilk b/gcc/ChangeLog.cilk
index 3667c75..f1935d2 100644
---
On 03/16/2012 09:49 PM, Richard Henderson wrote:
On 03/16/12 05:36, Bernd Schmidt wrote:
The machine is special. Pointer addition is a different operation than
integer addition. It'll also need a new ptr_plus rtx code which takes a
Pmode and an SImode operand. Pmode is larger than SImode but
On Mar 16, 2012, at 3:03 PM, Steven Bosscher wrote:
On Fri, Mar 16, 2012 at 10:54 PM, Mike Stump mikest...@comcast.net wrote:
This removes some wrong code.
Ok?
ChangeLog is missing.
* emit-rtl.c (immed_double_const): Remove bogus assert.
Tested how?
Compiles a user program for
This patch addresses two main problems.
(1) We were mishandling tagless types. The root of the problem was that
tagless types were given tags based on a sequence number. Two different PPH
files would thus have the same manufactured tag name for different types,
leading to inappropriate
On Tue, 13 Mar 2012, Uros Bizjak wrote:
A small no-op change - there is no need for a constraint in an expand
pattern. Plus some formatting.
If you want to remove it, then remove it, don't just empty it. ;)
2012-03-13 Uros Bizjak ubiz...@gmail.com
* config/i386/i386.md (xbegin):
68 matches
Mail list logo