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
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 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.
>
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
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
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?
>
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
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 (
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.
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
I am pinging this post:
http://gcc.gnu.org/ml/gcc-patches/2012-03/msg00430.html
Thanks
Edmar
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
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
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
> "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'
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
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
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
I'd like to re-ping this patch backport:
http://gcc.gnu.org/ml/gcc-patches/2012-02/msg01428.html
Peter
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
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
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
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
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
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
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 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 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 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 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
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
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
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
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 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: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
> >> 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
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
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
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
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
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 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
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 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
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
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
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
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
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
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
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.
>>
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 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 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 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
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
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 *.
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.
>>
>
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 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
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
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
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
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
> > +
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
===
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
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-
68 matches
Mail list logo
