On Sat, 6 Aug 2011, Paolo Bonzini wrote:
> On 08/05/2011 02:31 PM, Richard Guenther wrote:
> > This extends VRP to handle BIT_NOT_EXPR by composing ~X as
> > -X - 1 (which should give us anti-range handling for free).
>
> Just a small detail, but why not -1 - X which saves the NEGATE_EXPR? :)
Ma
On Fri, 5 Aug 2011, Martin Jambor wrote:
> Hi,
>
> the patch below fixes PR 49923 by checking for misaligned accesses
> before doing IPA-SRA (on strict alignment targets). I have checked it
> fixes the issue on compile farm sparc64 and I also included this in a
> bootstrap and testsuite run on a
A recent change in ipa-inline-analysis.c introduced a comparison
between signed and unsigned values, which broke bootstrap due to
-Wsign-compare (PR50005). Fixed by casting VEC_length to int before
using it in a comparison, as is done in many other places.
Tested on i686-linux and sparc64-linux.
Hi Terry,
On Mon, 8 Aug 2011, Terry Guo wrote:
> This new branch intends to provide fixes and enhancements for GCC 4.6
> when used with ARM embedded cores.
>
> The attached patch adds documentation for this branch.
thanks for thinking to document this! This looks good modulo two
notes.
+ This
On 08/08/11 10:43, Gerald Pfeifer wrote:
> As for maintainership, we usually have individuals, not companies
> there. Could you name one or two (or three)? You can always adjust
> the list an any time later on, without any form of approval.
Often, but by no means always.
"This branch is mainta
2011/7/20 Diego Novillo :
> On Mon, Jul 18, 2011 at 03:06, Romain Geissler
> I will commit this patch shortly.
>
>
> Diego.
>
Ping !
Eric Botcazou writes:
>> 2011-07-27 Rainer Orth
>>
>> gcc:
>> * config/sparc/driver-sparc.c: New file.
>> * config/sparc/x-sparc: New file.
>> * config.host: Use driver-sparc.o, sparc/x-sparc on
>> sparc*-*-solaris2*.
>> * config/sparc/sparc.opt (native): New valu
Prompted by Eric's request for SPARC, I'm also updating Alpha and MIPS
options docs for recent changes. Tested in an (unrelated)
i386-pc-solaris2.11 bootstrap and by inspection with info.
Installed on mainline as obvious.
Rainer
2011-08-08 Rainer Orth
gcc:
* doc/inv
IRIX 6.5 bootstrap was broken this weekend: gengtype failed to link in
stage2 since it was linked with -lstdc++ (for graphite, from HOST_LIBS),
but LDFLAGS (which provides the necessary -L option) is missing.
With this patch, the bootstrap continues.
I suppose this is obvious?
Rainer
2
On Fri, 5 Aug 2011, Paolo Bonzini wrote:
> On 08/05/2011 01:04 PM, Richard Guenther wrote:
> > (I believe that's the only bit we know sth about
> > when both vr.min and vr.max are negative).
>
> Depends, if the value is between -2^16 and -1, we know something about all the
> bits to the left of b
While preparing the patch to move gthr* to toplevel libgcc, I noticed
that we still have half-hearted support for the posix95 thread model.
No port uses it as the default, and the posix95 subdir in libgomp/config
isn't referenced anywhere and cannot be used. Thus, I propose to remove
this model co
Hello,
this helps C++ compilers that do not define __GNUC__ compile gcc.
Bootstrapped:
1) CC=gcc with --disable-build-poststage1-with-cxx
2) CC=suncc
The second one required a fix for PR49907. I didn't build with sunCC
simply because it requires several more patches.
Issues were discussed th
On 08/08/2011 02:21 PM, Rainer Orth wrote:
IRIX 6.5 bootstrap was broken this weekend: gengtype failed to link in
stage2 since it was linked with -lstdc++ (for graphite, from HOST_LIBS),
but LDFLAGS (which provides the necessary -L option) is missing.
With this patch, the bootstrap continues.
I
On Mon, 8 Aug 2011, Richard Guenther wrote:
> On Fri, 5 Aug 2011, Paolo Bonzini wrote:
>
> > On 08/05/2011 01:04 PM, Richard Guenther wrote:
> > > (I believe that's the only bit we know sth about
> > > when both vr.min and vr.max are negative).
> >
> > Depends, if the value is between -2^16 and
Hello,
apparently the doc wasn't updated 10 years ago when the macro
CP_TYPE_QUALS was replaced with the function cp_type_quals. I assume the
behavior didn't change, so the documentation change should be minor. Or
are functions documented differently?
2011-08-08 Marc Glisse
* doc
This patch moves gthr*.h and related code to toplevel libgcc. Unlike
the other outstanding patches (libgcc1, libgcc2, crtstuff, shlib), it is
pretty independent from the rest, so I'm posting it first. It was
developed last, so there may be conflicts (gcc/Makefile.in). It also
depends on the posi
On Thu, Jul 28, 2011 at 3:26 AM, Gerald Pfeifer wrote:
> On Thu, 28 Jul 2011, Kirill Yukhin wrote:
>> Ping
>
> Oh, sure. I had somehow thought this had been applied already.
>
> Instead of just removing ix86/avx, would you mind moving it to
> the "Inactive Development Branches" section?
I am app
> "Jie" == Jie Liu writes:
Jie> + if(str == NULL)
Jie> + throw new NullPointerException();
Jie> +
Jie>int length = str->length();
Why doesn't 'str->length()' throw the NPE?
Tom
Oops, my last fix was still wrong. This patch just compares min/max
values, which should be safe. :)
Tested x86_64-pc-linux-gnu, applied to trunk.
commit 5efa9cf21a329bba89748ddbc8e3bf3c8ea4329e
Author: Jason Merrill
Date: Sun Aug 7 17:10:26 2011 -0400
PR c++/50011
* typeck2.c (c
Hi,
On Mon, 8 Aug 2011, Diego Novillo wrote:
> With these patches I'd like to:
>
> 1 Create independent facilities to stream trees, gimple and other
> data structures independently of LTO. This first patch creates
> the files data-streamer*.[ch], tree-streamer*.[ch] and
> gimple-streamer*
On 08/08/2011 03:47 PM, Rainer Orth wrote:
This patch was bootstrapped without regressions on i386-pc-solaris2.10
and i386-pc-solaris2.11. It probably needs more testing on other
non-posix platforms.
Did you include Ada?
Paolo
Paolo Bonzini writes:
> On 08/08/2011 03:47 PM, Rainer Orth wrote:
>>
>> This patch was bootstrapped without regressions on i386-pc-solaris2.10
>> and i386-pc-solaris2.11. It probably needs more testing on other
>> non-posix platforms.
>
> Did you include Ada?
sure, all languages, even Go and O
On Mon, Aug 8, 2011 at 10:35, Jack Howarth wrote:
> On Mon, Aug 08, 2011 at 10:23:34AM -0400, Diego Novillo wrote:
>>
>> This is the first patch in a series of refactoring patches to cleanup
>> the API for the LTO streamer. I need this cleanup so I can change the
>> way things like caching and ex
On Mon, Aug 8, 2011 at 10:52, Michael Matz wrote:
> Sound. ;) Looking forward to some bikeshedding about naming in (2) and
> overabstraction in (3) :)
Heh, yeah.
I am going to be sending the renaming patch later today or tomorrow.
In principle, the things I want to abstract are those that are
Hello!
Attached patch implements addr32 prefixed addresses for x86_64
targets, where memory locations are accessed with 32bit base and index
registers in the form (zero_extend:DI (... SImode registers ...)).
The optimization rarely (if at all) triggers on x86_64, but is very
important on x32 (see
On 08/08/2011 03:47 PM, Rainer Orth wrote:
This patch moves gthr*.h and related code to toplevel libgcc. Unlike
the other outstanding patches (libgcc1, libgcc2, crtstuff, shlib), it is
pretty independent from the rest, so I'm posting it first. It was
developed last, so there may be conflicts (g
On Mon, 8 Aug 2011, Diego Novillo wrote:
> On Mon, Aug 8, 2011 at 10:52, Michael Matz wrote:
>
> > Sound. ;) Looking forward to some bikeshedding about naming in (2) and
> > overabstraction in (3) :)
>
> Heh, yeah.
>
> I am going to be sending the renaming patch later today or tomorrow.
> In
On 08/08/2011 02:30 PM, Rainer Orth wrote:
While preparing the patch to move gthr* to toplevel libgcc, I noticed
that we still have half-hearted support for the posix95 thread model.
No port uses it as the default, and the posix95 subdir in libgomp/config
isn't referenced anywhere and cannot be u
Uros Bizjak wrote:
> Although, it would be nice for reload to subsequently fix CSE'd
> non-offsetable memory by copying address to temporary reg (*as said in
> the documentation*), we could simply require an XMM temporary for
> TImode reloads to/from integer registers, and this fixes ICE for x32.
The update to range-for lookup of 'begin' and 'end' added support for
COMPONENT_REF to finish_call_expr; we need to check for it here, too.
Tested x86_64-pc-linux-gnu, applied to trunk.
commit 5509a56f6699e727a27a8010a566027cb41191f7
Author: Jason Merrill
Date: Mon Aug 8 10:56:49 2011 -0400
On 08/08/2011 05:16 PM, Uros Bizjak wrote:
BTW: There is a strange optimization in combine pass, where
zero-extended address is converted on-the-fly to:
Trying 9 -> 10:
Failed to match this instruction:
(... (and:DI (subreg:DI (plus:SI (ashift:SI (reg/v:SI 63 [ i ])
(const_
On 8-Aug-11, at 9:47 AM, Rainer Orth wrote:
** _DCE_THREADS is used to select gthr-dce.h, but again dce is the
only/default model on hppa[12]*-*-hpux10* (pa-hpux10.h), so the
special-casing can be removed.
DCE threads and thread single are both supported. There are multilibs
for both. D
On 08/08/2011 05:42 PM, John David Anglin wrote:
** _DCE_THREADS is used to select gthr-dce.h, but again dce is the
only/default model on hppa[12]*-*-hpux10* (pa-hpux10.h), so the
special-casing can be removed.
DCE threads and thread single are both supported. There are
multilibs for both.
Paolo Bonzini writes:
>> ** The use of _PTHREADS to select gthr-posix.h is quite inconsistent:
>> only a few targets define that macro (m32r/linux.h, mn10300/linux.h,
>> netbsd.h, sh/linux.h, sol2.h), and none of them support alternative
>> thread models. Given that everything works
Paolo Bonzini writes:
> On 08/08/2011 05:42 PM, John David Anglin wrote:
>>
>>
>>> ** _DCE_THREADS is used to select gthr-dce.h, but again dce is the
>>> only/default model on hppa[12]*-*-hpux10* (pa-hpux10.h), so the
>>> special-casing can be removed.
>>
>> DCE threads and thread single are both
On 08/08/2011 05:51 PM, Rainer Orth wrote:
libgfortran/acinclude.m4 (LIBGFOR_GTHREAD_WEAK) tries to determine a
sensible value.
I think libgfortran's approach is the nicest to be added in
libgcc/configure.ac.
I'm not convinced: when it was introduced (or used more extensively), I
w
Paolo Bonzini writes:
> On 08/08/2011 05:51 PM, Rainer Orth wrote:
libgfortran/acinclude.m4 (LIBGFOR_GTHREAD_WEAK) tries to determine a
sensible value.
>>>
>>> I think libgfortran's approach is the nicest to be added in
>>> libgcc/configure.ac.
>>
>> I'm not convinced: when i
On Fri, Aug 5, 2011 at 18:49, Gabriel Charette wrote:
> This is the test I was talking would potentially fail with our current
> implementation.
>
> And it does.
>
> It has an assembly difference in the order of the global variables, like we
> used to have in simpler tests.
>
> Gab
>
> 2011-08-0
Richard Guenther wrote:
> Of course testing for the fold makes sure we also test for the crash ...
> so yes, the idea was to also test for the fold. As the LTO testing
> harness does not support scanning dumps (sigh...) link tests are
> the only possibility right now.
OK, so what about the follo
Michael Matz wrote:
> Hi,
>
> On Tue, 26 Jul 2011, Michael Matz wrote:
>
> > On Tue, 26 Jul 2011, Michael Matz wrote:
> >
> > > Hi,
> > >
> > > On Tue, 26 Jul 2011, Ulrich Weigand wrote:
> > >
> > > > > Well, REG_ATTRS->decl is again a decl, not an SSA name. I suppose
> > > > > we'd need to p
On 8-Aug-11, at 11:53 AM, Rainer Orth wrote:
Paolo Bonzini writes:
On 08/08/2011 05:42 PM, John David Anglin wrote:
** _DCE_THREADS is used to select gthr-dce.h, but again dce is the
only/default model on hppa[12]*-*-hpux10* (pa-hpux10.h), so the
special-casing can be removed.
DCE thre
John David Anglin writes:
>> I could find no indication that HP-UX 10 supported anything but DCE
>> threads (especially no POSIX threads), so _REENTRANT being defined must
>> meant DCE threads, unless I'm mistaken.
>
> _REENTRANT doesn't imply DCE threads. The reentrant routines were added
> to
John David Anglin writes:
>> I haven't moved two macros used in gthr*.h yet: GTHREAD_USE_WEAK and
>> SUPPORTS_WEAK. This seems to warrant a separate followup patch, but for
>> the record I report my findings:
>>
>> * GTHREAD_USE_WEAK must be defined explicitly. It needs to be poisoned
>> in sy
On Mon, Aug 8, 2011 at 8:16 AM, Uros Bizjak wrote:
> Hello!
>
> Attached patch implements addr32 prefixed addresses for x86_64
> targets, where memory locations are accessed with 32bit base and index
> registers in the form (zero_extend:DI (... SImode registers ...)).
> The optimization rarely (if
> I'm seeing this bootstrap failure on ia64 (configured with
> --with-system-libunwind):
> /usr/local/gcc/gcc-20110806/Build/./prev-gcc/libgcc_s.so: undefined referen=
> ce to `__libunwind__Unwind_Find_FDE'
> collect2: error: ld returned 1 exit status
> make[3]: *** [gengtype] Error 1
I get this
On 8-Aug-11, at 12:29 PM, Rainer Orth wrote:
John David Anglin writes:
I haven't moved two macros used in gthr*.h yet: GTHREAD_USE_WEAK and
SUPPORTS_WEAK. This seems to warrant a separate followup patch,
but for
the record I report my findings:
* GTHREAD_USE_WEAK must be defined explici
On Mon, Aug 8, 2011 at 5:30 PM, Ulrich Weigand wrote:
> Uros Bizjak wrote:
>
>> Although, it would be nice for reload to subsequently fix CSE'd
>> non-offsetable memory by copying address to temporary reg (*as said in
>> the documentation*), we could simply require an XMM temporary for
>> TImode r
On Mon, Aug 8, 2011 at 10:11 AM, Uros Bizjak wrote:
> On Mon, Aug 8, 2011 at 5:30 PM, Ulrich Weigand wrote:
>> Uros Bizjak wrote:
>>
>>> Although, it would be nice for reload to subsequently fix CSE'd
>>> non-offsetable memory by copying address to temporary reg (*as said in
>>> the documentation
"H.J. Lu" writes:
> #define OPTION_MASK_ISA_XOP (1 << 31)
1 << 31 is undefined.
Andreas.
--
Andreas Schwab, sch...@linux-m68k.org
GPG Key fingerprint = 58CA 54C7 6D53 942B 1756 01D3 44D5 214B 8276 4ED5
"And now for something completely different."
Andreas Schwab writes:
> I'm seeing this bootstrap failure on ia64 (configured with
> --with-system-libunwind):
>
> /usr/local/gcc/gcc-20110806/Build/./prev-gcc/g++
> -B/usr/local/gcc/gcc-20110806/Build/./prev-gcc/ -B/usr/ia64-suse-linux/bin/
> -nostdinc++
> -B/usr/local/gcc/gcc-20110806/Build
Hi
Le 16 juil. 2011 à 08:52, Ian Lance Taylor a écrit :
> I would like to propose this patch as a step toward building gcc using a
> C++ compiler. This patch builds stage1 with the C compiler as usual,
> and defaults to building stages 2 and 3 with a C++ compiler built during
> stage 1. This me
> > +/** @file bits/alloc_traits.h
> > + * This is an internal header file, included by other library
> > headers.
> > + * Do not attempt to use it directly.
> > @headername{scoped_allocator}
> > + */
> > +
> >
> > The correct header for allocator_traits is
Ack. Thanks for the review. Kind of
On 8 August 2011 19:09, Benjamin Kosnik wrote:
>
>> > +/** @file bits/alloc_traits.h
>> > + * This is an internal header file, included by other library
>> > headers.
>> > + * Do not attempt to use it directly.
>> > @headername{scoped_allocator}
>> > + */
>> > +
>> >
>> > The correct header for a
Romain Geissler writes:
> This new build behavior broke former plugins built with gcc. Indeed,
> all cc1 function symbols are now mangled and thus with the current
> trunk, plugins should also look for mangled symbols (and so built
> with g++).
>
> What's the new GCC policy about that ? Do plugin
Hello!
One notable omission from x87 rounding sequences is plain round(),
since x87 does not support round-to-nearest, away-from-zero directly.
However, this function can be synthesized as round(a) = sgn(a) *
floor(fabs(a) + 0.5). Fortunately, we have all building blocks ready
in i386.md, we shoul
Le 8 août 2011 à 20:49, Ian Lance Taylor a écrit :
>
> However, since we currently permit plugins to call anything in gcc, I
> think the answer is going to have to be that plugins which do that
> should be compiled with C++.
Ok, i'll move to C++ then, until a dedicated C plugin API comes out.
>
Hi.
This patch removes obsolete REGISTER_MOVE_COST macro from MMIX back end in
the GCC and introduces equivalent TARGET_REGISTER_MOVE_COST target hook.
Regression tested on mmix-knuth-mmixware.
OK to install?
* config/mmix/mmix.h (REGISTER_MOVE_COST): Remove macro.
* conf
Hello.
This patch remove unused REG_OK_FOR_BASE_P and REG_OK_FOR_INDEX_P macros
from the MMIX back end.
Regression tested on mmix-knuth-mmixware.
OK to install?
* config/mmix/mmix.h (REG_OK_FOR_BASE_P, REG_OK_FOR_INDEX_P): Remove.
Index: gcc/config/mmix/mmix.h
==
Hello,
Jason Merrill writes:
> On 08/06/2011 06:57 AM, Dodji Seketeli wrote:
> > @@ -4340,6 +4340,8 @@ c_sizeof_or_alignof_type (location_t loc,
> >value = fold_convert_loc (loc, size_type_node, value);
> >gcc_assert (!TYPE_IS_SIZETYPE (TREE_TYPE (value)));
> >
> > + maybe_record_local_
On Mon, 8 Aug 2011, Marc Glisse wrote:
> * include/obstack.h (obstack_free): Cast to char* instead of int
This header comes from glibc/gnulib. Although some local changes have
been made to it in the past, sending any fixes to upstream glibc is still
a good idea.
(ansidecl.h *claims* to
On Fri, 2011-08-05 at 11:45 -0700, Richard Henderson wrote:
> On 08/05/2011 10:54 AM, Steve Ellcey wrote:
> > - unsigned char checksum[16];
> >struct md5_ctx ctx;
> > + unsigned char checksum[16];
>
> How about
>
> struct md5_data
> {
> struct md5_ctx ctx;
> unsigned char checks
On Mon, 8 Aug 2011, Anatoly Sokolov wrote:
> Hi.
>
> This patch removes obsolete REGISTER_MOVE_COST macro from MMIX back end in
> the GCC and introduces equivalent TARGET_REGISTER_MOVE_COST target hook.
...and removed a comment about a moot issue (perfectly ok).
>
> Regression tested on mmix-
On 08/08/2011 02:24 PM, Steve Ellcey wrote:
> Do you mean a union, as Andreas suggested?
No, I mean struct. The union is wrong, because both data elements
are live at the same time.
r~
On Mon, 8 Aug 2011, Anatoly Sokolov wrote:
> Hello.
>
> This patch remove unused REG_OK_FOR_BASE_P and REG_OK_FOR_INDEX_P macros
> from the MMIX back end.
>
> Regression tested on mmix-knuth-mmixware.
>
> OK to install?
>
> * config/mmix/mmix.h (REG_OK_FOR_BASE_P, REG_OK_FOR_INDEX_P):
On Mon, 8 Aug 2011, Joseph S. Myers wrote:
On Mon, 8 Aug 2011, Marc Glisse wrote:
* include/obstack.h (obstack_free): Cast to char* instead of int
This header comes from glibc/gnulib. Although some local changes have
been made to it in the past, sending any fixes to upstream glibc i
On Fri, 2011-08-05 at 11:45 -0700, Richard Henderson wrote:
> On 08/05/2011 10:54 AM, Steve Ellcey wrote:
> > - unsigned char checksum[16];
> >struct md5_ctx ctx;
> > + unsigned char checksum[16];
>
> How about
>
> struct md5_data
> {
> struct md5_ctx ctx;
> unsigned char checks
It's already fixed, 'twas the missing ldflags.
Andreas.
--
Andreas Schwab, sch...@linux-m68k.org
GPG Key fingerprint = 58CA 54C7 6D53 942B 1756 01D3 44D5 214B 8276 4ED5
"And now for something completely different."
This is a ping for my patch to fix the C++ build of GCC on HP-UX and to
fix -static-libstdc.
The original patch is at:
http://gcc.gnu.org/ml/gcc-patches/2011-07/msg02704.html
I have updated the patch to deal with the changes that Rainer's top-level
libgcc patch made and included the new version
The following patch fixes PR49990. The problem is described on
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=49936
Reg classes which can not change modes for some pseudo were excluded
from the consideration. As I wrote recently, they should not. Instead
the correct cost for changing mode (by
Remove Xuepeng Guo and Joey Ye from owners of branches that we're no
longer maintaining.
Index: svn.html
===
RCS file: /cvs/gcc/wwwdocs/htdocs/svn.html,v
retrieving revision 1.163
diff -u -p -r1.163 svn.html
--- svn.html8 Aug 2011
This patch updates tests to clear conflicting commits.
The change needed for x6rtti.cc does not make sense to me.
Tested on x64.
Index: gcc/testsuite/ChangeLog.pph
2011-08-08 Lawrence Crowl
* (g++.dg/pph/x6dynarray4.cc): Remove dg-prune-output.
* (g++.dg/pph/x4keyed.cc): Ad
On Mon, 8 Aug 2011, Rainer Orth wrote:
> * SUPPORTS_WEAK is probably best detected with autoconf. We have
> libgfortran/acinclude.m4 (LIBGFOR_GTHREAD_WEAK), but should probably
> go for m4/ax_sys_weak_alias.m4 from the autoconf-archive instead. If
> we're lucky, that macro can detect suppor
On Mon, Aug 8, 2011 at 4:19 PM, Vladimir Makarov wrote:
> The following patch fixes PR49990. The problem is described on
>
> http://gcc.gnu.org/bugzilla/show_bug.cgi?id=49936
>
> Reg classes which can not change modes for some pseudo were excluded from
> the consideration. As I wrote recently, t
Hello list,
the attach simple patch saves reproducively a tiny bit of time on various
-O0 compilations I've performed, for example 5ms out of 627ms. Tested on
i386.
We trade a little bit of memory (maxmem2.sh from [1] reported 25240 KB
instead of 25080 KB without the patch) to allow our hash
Hello list,
this is the second part of my patch. It collects and prints some memory
statistics for hash tables that I've measured as the hottest ones. Tested
together with previous patch on i386. Example output:
libcpp symtab string pool:
identifiers 32644 (100.00%)
entri
Hi,
In vectorization of reduction we use a dummy def_type when getting a
copy of a vector operand. Therefore, instead of just using a constant
operand from a previous copy, we try to create a copy of it, causing
the failure. This patch adds a call to vect_is_simple_use () to get a
correct def_type
76 matches
Mail list logo