On Thu, Mar 15, 2012 at 22:28, Janne Blomqvist
wrote:
> On Thu, Mar 15, 2012 at 22:14, Tobias Burnus 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.
>>>
>>> Regtested on x86_64-
Hi,
all the three VMS entries are consolidated into one with this patch.
Committed on trunk.
Tristan.
2012-03-16 Tristan Gingold
* config.host (alpha64-dec-*vms*, alpha*-dec-*vms*)
(ia64-hp-*vms*): Replaced by ...
(*-*-*vms*): ... This new entry.
Index: gcc/config.h
Hi,
this patch makes gcc provide the C99 compliant stdint.h header.
(required by libquadmath).
Committed on trunk.
Tristan.
2012-03-16 Tristan Gingold
* config.gcc (*-*-*vms*): Define use_gcc_stdint and tm_file.
* config/vms/vms-stdint.h: New file.
Index: gcc/config.gcc
===
On Thu, 15 Mar 2012, Rainer Orth wrote:
> Gerald Pfeifer 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 miscompila
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
* tree-vect-loop.c (get_initial_def_for_induction): Use
build_constructor directly.
* tree-vect-stmts.c (v
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
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
>> > wrote:
>>
>> >> committe
On Fri, Mar 16, 2012 at 1:13 AM, Andrew Pinski wrote:
> On Thu, Mar 15, 2012 at 5:09 PM, Bernd Schmidt
> 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 performed only within the same
>>> ob
On Fri, Mar 16, 2012 at 1:29 AM, Jonathan Wakely wrote:
> On 15 March 2012 15:40, Richard Guenther wrote:
>> On Thu, Mar 15, 2012 at 4:22 PM, Kai Tietz wrote:
>>> Richard,
>>>
>>> ping. I think now could be a good time for applying the patch you
>>> have for this issue as we are in stage 1.
>>
>
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
* config/ia64/v
On Fri, Mar 16, 2012 at 11:39 AM, Tristan Gingold 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 understanding is t
On Fri, Mar 16, 2012 at 9:17 AM, Janne Blomqvist
wrote:
> On Thu, Mar 15, 2012 at 22:28, Janne Blomqvist
> wrote:
>> On Thu, Mar 15, 2012 at 22:14, Tobias Burnus wrote:
>>> Janne Blomqvist wrote:
since some time GCC has BUILT_IN_IROUND{F,,L}, similar to lround() and
llround() but
On Mar 16, 2012, at 12:02 PM, Richard Guenther wrote:
> On Fri, Mar 16, 2012 at 11:39 AM, Tristan Gingold 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 lik
On Thu, Mar 15, 2012 at 3:34 PM, Kai Tietz wrote:
> 2012/3/15 Richard Guenther :
>> On Thu, Mar 15, 2012 at 3:00 PM, Jakub Jelinek wrote:
>>> On Thu, Mar 15, 2012 at 02:53:10PM +0100, Kai Tietz wrote:
> This looks like to match unbound pattern sizes and thus does not fit
> into the forw
On Fri, Mar 16, 2012 at 12:33 PM, Tristan Gingold wrote:
>
> On Mar 16, 2012, at 12:02 PM, Richard Guenther wrote:
>
>> On Fri, Mar 16, 2012 at 11:39 AM, Tristan Gingold
>> wrote:
>>> Hi,
>>>
>>> currently sizetype precision (cf store-layout.c:initialize_sizetypes) is
>>> the same as size_t.
>>
This fixes issue No. 1, easy.
Bootstrapped and tested on x86_64-unknown-linux-gnu, applied to trunk.
Richard.
2012-03-16 Richard Guenther
* stor-layout.c (finish_bitfield_representative): Fall back
to the conservative maximum size if we cannot compute the
size of 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 resu
On Mar 16, 2012, at 12:38 PM, Richard Guenther wrote:
> On Fri, Mar 16, 2012 at 12:33 PM, Tristan Gingold wrote:
>>
>> On Mar 16, 2012, at 12:02 PM, Richard Guenther wrote:
>>
>>> On Fri, Mar 16, 2012 at 11:39 AM, Tristan Gingold
>>> wrote:
Hi,
currently sizetype precision (c
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 nec
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 pen
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 it
On Fri, Mar 16, 2012 at 1:36 PM, Bernd Schmidt 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. Or is it the i
On 03/16/2012 01:55 PM, Richard Guenther wrote:
> On Fri, Mar 16, 2012 at 1:36 PM, 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
On Fri, Mar 16, 2012 at 2:05 PM, Bernd Schmidt wrote:
> On 03/16/2012 01:55 PM, Richard Guenther wrote:
>> On Fri, Mar 16, 2012 at 1:36 PM, Bernd Schmidt
>> wrote:
>>>
>>> The machine is "special". Pointer addition is a different operation than
>>> integer addition. It'll also need a new ptr_plu
With recent disturbed testing this FAIL got unnoticed. Fixed as follows
(we have a constant " " that 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
* tree-vect-stmts.c (vect_get_v
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
On Fri, Mar 16, 2012 at 2:18 PM, Bernd Schmidt 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 then
>>
>> (set
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]
./bt(_Z6nqueenP
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]
./bt(_Z6nqueenPiii+0xf0)[0xa8
> >> 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
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.
>>
>> N
On Fri, Mar 16, 2012 at 2:37 PM, Bernd Schmidt 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 to yo
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
> wit
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
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 fac
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 bette
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 u
On 03/16/12 04:56, Tristan Gingold wrote:
> 2012-03-16 Tristan Gingold
>
> * 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 (ASM_OUTPUT_ADDR_DIFF_ELT): U
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.
P
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 locati
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 t
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_T
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
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 wrote:
> > On Thu, Mar 08, 2012 at 12:06:46PM +0100, Martin Jambor wrote:
> >> /* For local statics lookup proper context die. */
> >> - if (TREE_STATIC (decl) && de
On Fri, Mar 16, 2012 at 4:49 PM, Zdenek Dvorak 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 pointed to be
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: && !(L_IPO_STREAM_IN_L
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
PR tree-optimization/52603
* tree-vect-data-refs.c (vect_can_force_dr_alignm
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
inv
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/
http://coderev
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 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 certain.
It doesn'
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 wrote:
> > > On Thu, Mar 08, 2012 at 12:06:46PM +0100, Martin Jambor wrote:
> > >> /* For local statics
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
Attached patch applied.
2012-03-16 François Dumont
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.
* testsuite
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
>
> * 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.
> * dwarf2cfi.c (d
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 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
> ===
> --- gcc/emit-rtl.c (rev
Ping - now that stage 1 is open, could someone review?
Thanks,
Teresa
On Sun, Dec 4, 2011 at 10:26 PM, Teresa Johnson wrote:
> Latest patch which improves the efficiency as described below is
> included here. Boostrapped and checked again with
> x86_64-unknown-linux-gnu. Could someone review?
>
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
--- a/gcc/ChangeLog.cilk
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 SImo
On Mar 16, 2012, at 3:03 PM, Steven Bosscher wrote:
> On Fri, Mar 16, 2012 at 10:54 PM, Mike Stump 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 my port.
>
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 collision
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
>
> * config/i386/i386.md (xbegin): Remove constrai
68 matches
Mail list logo
