2011/5/28 Georg-Johann Lay :
> Georg-Johann Lay wrote:
>>
>> Georg-Johann Lay wrote:
>>
>>> Richard Henderson wrote:
>>>
>>>
Why are you adding "optimize" to all these insns? None of them will
be matched unless combine is run, which implies optimization.
>>>
>>> Here is a patch without o
Hi!
f4/f5/f6 were copied from f1/f2/f3, just with s/long/int/,
but only in f4 I've adjusted the shift count.
Fixed thusly, committed to trunk.
2011-06-04 Jakub Jelinek
* gcc.dg/guality/rotatetest.c (f5, f6): Fix up pastos.
--- gcc/testsuite/gcc.dg/guality/rotatetest.c (revision 174
On Mon, May 30, 2011 at 12:26:50PM +0200, Bernd Schmidt wrote:
> On 05/30/2011 11:51 AM, Alexandre Oliva wrote:
> > Bootstrap-O3 complains about uninitialized uses of this variable in
> > gengtype-state.c. The variable can never actually be used before
> > initialization, but GCC is not smart enou
Hi,
I've applied the attached patch under the obvious rule
to fix the build failure on sh4-unknown-linux-gnu.
Regards,
kaz
--
[libgcc]
2011-06-04 Kaz Kojima
* config.host (sh*-*-linux*): Fix typo.
--- ORIG/trunk/libgcc/config.host 2011-06-04 10:16:44.0 +0900
+++
Hi ARM maintainers
Could you help to review the following patches?
thanks
Carrot
On Tue, May 24, 2011 at 9:31 AM, Carrot Wei wrote:
> Hi
>
> http://gcc.gnu.org/ml/gcc-patches/2011-03/msg01973.html
> Use ldrd and strd to access two consecutive words
>
> http://gcc.gnu.org/ml/gcc-patches/2011-05/
On 31 May 2011 11:03, Nathan Sidwell wrote:
> On 05/27/11 01:47, Ramana Radhakrishnan wrote:
>
>> Could you consider adding a check in the configury to test if a binutils
>> version
>> of recent vintage is being used when --with-tls=gnu is in ?
>
> I thought about that and it didn't seem worth it.
As discussed in http://gcc.gnu.org/ml/gcc-patches/2011-06/msg00025.html,
these assertions are no longer needed and they can be unified under
lto_is_streamable (before converting it into a streamer hook).
Tested with LTO profiledbootstrap on x86_64. Committed to trunk.
Diego.
* lto-str
On 06/03/2011 04:20 AM, Jonathan Wakely wrote:
On 3 June 2011 02:55, Jason Merrill wrote:
Do we really want to warn about the case where the class has no/trivial
destructor?
I think so. Definitely if it's an abstract class, since it's
definitely undefined behaviour. The user might have for
On Fri, Jun 3, 2011 at 13:28, Gabriel Charette wrote:
> 2011-06-03 Gabriel Charette
>
> * timevar.def (TV_PPH_CACHE_IN): Remove.
> (TV_PPH_CACHE_OUT): Remove.
> (TV_PPH_MANAGE): Remove.
OK. Committed.
Diego.
On Thu, Jun 2, 2011 at 21:09, Gabriel Charette wrote:
> 2011-06-02 Gabriel Charette
>
> * gcc/c-family/c.opt (fpph): Remove.
> Update all users.
> (fpph-decls): Remove.
> Update all users.
> (fpph-decls-debug): Remove.
> Update all users.
> (fpp
Hello,
I am sorry, my editor (vim) was not correctly configure (I used
http://gcc.gnu.org/ml/gcc/2011-03/msg00425.html to improve it). I guess
it is ok now.
If I still have issue, I will post on the mailing list if there is some
tips for vim otherway I will use Emacs (I am not very comfortable
Ok.
Jakub Jelinek wrote:
Hi!
This testcase got broken by the
http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=164719
changes, before that it really didn't matter at that spot
if DECL_CONTEXT or CP_DECL_CONTEXT is used, as write_prefix
immediately returns for both NULL and global_namespace.
But
This patch removes another couple of deprecated header fragments. Note that
this
starts to remove pieces of the Traditional API, which will be entirely removed
in GCC 4.7.0. Of course, that requires switching all testcases to use the
Modern
API when compiling for the GNU runtime, but we'll get
On Fri, Jun 03, 2011 at 09:38:31PM +0200, Rainer Orth wrote:
> Rainer Orth writes:
> Jakub, any suggestion how to properly test for -fpie/-fPIE support?
> Otherwise, I'll remove that part of the patch for now and just commit
> the -fpic/-fPIC one.
You want to compile/link the program with -pie -f
This patch removes more deprecated and obsolete hooks/functions from libobjc.
Committed to trunk.
Thanks
Index: Makefile.in
===
--- Makefile.in (revision 174616)
+++ Makefile.in (working copy)
@@ -132,7 +132,6 @@ OBJC_DEPRECATED_H =
Rainer Orth writes:
> As suggested by Joseph, the gcc.dg/torture/tls tests should be run with
> -fpic/-fPIC and -fpie/-fPIE if supported.
>
> The patch below implements part of this. Unlike
> gcc.dg/torture/stackalign/stackalign.exp, I'm using the code below to
> add -fpic etc. to the torture op
This patch removes the deprecated definition of METHOD_NULL.
Committed to trunk.
Thanks
Index: Makefile.in
===
--- Makefile.in (revision 174615)
+++ Makefile.in (working copy)
@@ -127,7 +127,6 @@ OBJC_H = \
# User-visible header fi
On Jun 3, 2011, at 9:25 AM, Jack Howarth wrote:
> Could someone commit this patch, which was already approved in
Done.
This patch removes another bunch of deprecated functions and hooks from libobjc,
in particular objc_valloc(), the various _objc_alloc and similar hooks and the
_objc_unexpected_exception hook.
Committed to trunk.
Thanks
Index: Makefile.in
=
ARM maintainers,
Is it ok to skip building target-libiberty for arm*-*-linux-androideabi target?
http://gcc.gnu.org/ml/gcc-patches/2011-05/msg02208.html
Thanks,
Jing
On Fri, Jun 3, 2011 at 11:26 AM, DJ Delorie wrote:
>
>> Ping.
>> http://gcc.gnu.org/ml/gcc-patches/2011-05/msg02208.html
>>
>> >
This trivial patch removes the deprecated definition of STR.
Committed to trunk.
Thanks
Index: Makefile.in
===
--- Makefile.in (revision 174610)
+++ Makefile.in (working copy)
@@ -131,7 +131,6 @@
MetaClass.h \
Object.h \
Pr
Hi!
This testcase got broken by the
http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=164719
changes, before that it really didn't matter at that spot
if DECL_CONTEXT or CP_DECL_CONTEXT is used, as write_prefix
immediately returns for both NULL and global_namespace.
But when DECL_CONTEXT is TRANSLA
On 06/03/2011 04:47 PM, Alexandre Oliva wrote:
> https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=677681 can be
> “fixed” by disabling the artificial dependency of a debug insn on its
> previous insn.
>
> Debug insns will often depend on their prevs anyway, in a use/def
> relationship, but if
> Ping.
> http://gcc.gnu.org/ml/gcc-patches/2011-05/msg02208.html
>
> > I don't have the bandwidth to work on the ideal patch. Thus I am
> > wondering if we can skip target-libiberty for androideabi target
> > before the ideal patch is out.
Target-specific changes in the build are up to the targ
> "Dodji" == Dodji Seketeli writes:
Dodji> I am pinging this patch first posted to
Dodji> http://gcc.gnu.org/ml/gcc-patches/2011-04/msg00697.html.
Thanks, and sorry for not noticing earlier.
Dodji> +context of 'P'. The problem is, if we are beeing
Typo, "being".
Ok with t
Ping.
http://gcc.gnu.org/ml/gcc-patches/2011-05/msg02208.html
On Tue, May 31, 2011 at 11:32 AM, Jing Yu wrote:
> Based on discussion on another thread
> (http://www.mail-archive.com/gcc-patches@gcc.gnu.org/msg06627.html),
> what Joseph recommended was ripping out all support for building
> libibe
On Jun 3, 2011, at 7:35 AM, Rainer Orth wrote:
> Apart from those changes, the patch addresses the PowerPC Darwin and
> Windows32 issues as suggested:
Darwin bits: Ok.
> Note that I am using r174594 sources with your patch *AND* with the patch
> Bernd proposed for PR 48673
> (http://gcc.gnu.org/ml/gcc-patches/2011-05/msg02470.html).
> So this regression may be due to the other change. I will look into it some
> more to see if I can figure out which change is
This patch removes objc/hash.h and objc/sarray.h, which had been deprecated in
GCC 4.6.0.
Of course, libobjc still has its own private versions, but they are (finally!)
private
so we have made another step towards being able to change the libobjc internals
at will
without having to change the p
On 06/03/2011 07:44 PM, Alexander Monakov wrote:
>
>
> On Thu, 2 Jun 2011, Bernd Schmidt wrote:
>
>> Ok. Although I wonder how sel-sched can end up reusing an entry in
>> h_d_i_d? How does it use this machinery? If it's not doing a normal
>> forward scan as in sched_analyze, the INSN_COND mechan
I am pinging this patch first posted to
http://gcc.gnu.org/ml/gcc-patches/2011-04/msg00697.html.
--- Begin Message ---
Hello,
While looking at something else, I noticed that preprocessing this
code snippet:
cat -n test.c
~=~
void foo (void)
{
int i1, j1, k1;
#define p parallel
#define P(x) pr
On Thu, 2 Jun 2011, Bernd Schmidt wrote:
> Ok. Although I wonder how sel-sched can end up reusing an entry in
> h_d_i_d? How does it use this machinery? If it's not doing a normal
> forward scan as in sched_analyze, the INSN_COND mechanism may break in
> other ways.
Indeed, that patch did not c
On Thu, 02 Jun 2011 16:35:01 +0100
Richard Earnshaw wrote:
> I see Paul has already approved this, but I've just spotted one
> potential problem that might cause latent bugs sometime in the future.
>
> The code to register the libcalls is only run once, the first time we
> try to look up a libca
Richard Henderson writes:
> On 06/01/2011 07:51 AM, Rainer Orth wrote:
>> + /* Set PC to the instruction after the faulting one to skip over it,
>> + otherwise we enter an infinite loop. 4 is the size of the stmxcsr
>> + instruction. */
> ...
>> + /* We need a single SSE instructi
On Thu, 2011-06-02 at 15:30 -0700, Steve Ellcey wrote:
> On Thu, 2011-06-02 at 15:29 +0400, Alexander Monakov wrote:
> > Bernd,
> >
> > The problem is INSN_COND should be reset when initializing a new deps
> > structure, otherwise instructions may get stale conditions from other
> > previously ana
Removed the 3 PPH timevars used in pph instrumentation. Those are not used
anymore following the instrumentation removal patch.
Bootstrap build and regression testing were successful.
2011-06-03 Gabriel Charette
* timevar.def (TV_PPH_CACHE_IN): Remove.
(TV_PPH_CACHE_OUT): Remo
Hi Doug,
On Fri, Jun 3, 2011 at 10:00 AM, Douglas B Rupp wrote:
>
> I've been working on a resurrection patch for Interix, one remaining problem
> is fixincludes/fixincl.c must be compiled on native Interix with
> -D_ALL_SOURCE in order to find the getpagesize prototype.
>
> This works (taking so
On 06/03/2011 05:31 AM, Richard Guenther wrote:
On Fri, Jun 3, 2011 at 6:31 AM, H.J. Lu wrote:
On Wed, May 18, 2011 at 8:57 AM, H.J. Lu wrote:
On Tue, Apr 26, 2011 at 6:05 AM, H.J. Lu wrote:
On Thu, Mar 31, 2011 at 7:57 AM, H.J. Lu wrote:
On Mon, Mar 21, 2011 at 11:40 AM, H.J. Lu wrote:
This patch removes -forward:: from Object. I was planning to do this later,
but the
previous patch (patch 3) removed -doesNotRecognize:, which was used in the
default
implementation of -forward::, which generates some ugly warnings when
compiling.
Removing -forward:: removes the warnings.
Pl
I've been working on a resurrection patch for Interix, one remaining
problem is fixincludes/fixincl.c must be compiled on native Interix with
-D_ALL_SOURCE in order to find the getpagesize prototype.
This works (taking some liberties with diff...):
fixincludes/Makefile.in
-FIXINC_CFLAGS = -D
On 06/03/2011 04:11 PM, Alexandre Oliva wrote:
> Trunk is now fixed, is the patch ok for the 4.6 branch too?
As far as I'm concerned, sure.
Bernd
Ok, windows part of the patch is ok.
Thanks,
Kai
Could someone commit this patch, which was already approved in
http://gcc.gnu.org/ml/gcc-patches/2011-05/msg00411.html
The absence of this patch in gcc trunk prevents --enable-build-with-cxx
from working on darwin. Thanks in advance.
Jack
- Forwarded message from Jack Howarth
Looks good to me, but you need an OK from a maintainer.
Thanks,
David
On Fri, Jun 3, 2011 at 7:17 AM, Alexandre Oliva wrote:
> A recent change introduced decl_uid in the “;; Function ” header in dump
> files. This breaks -fcompare-debug (and bootstrap-debug-lean), because
> decl uids aren't ke
On 06/03/2011 05:11 PM, Rainer Orth wrote:
Bootstrapped without regressions on i386-pc-solaris2.11, ok for
mainline? I suppose this is obvious.
The patch is okay, I'll look at the rest next week.
Paolo
On 06/03/2011 05:45 PM, Rainer Orth wrote:
Paolo Bonzini writes:
>> Seems like a plan, but I didn't go in this direction since I couldn't
>> test anything like this.
>
> As long as you test the general configury on 1-2 platforms, it's not any
> less tested than what you have now.
Unfortu
Actually, please consider this patch instead. It's the same but fixes a
mistake in ld.so pathname.
2011/6/2 Robert Millan :
> Hi,
>
> 2011/5/21 Joseph S. Myers :
>> Please send a patch against *current trunk* and CC *relevant target
>> architecture maintainers*. linux*.h headers are no longer us
On Fri, 03 Jun 2011 17:31:25 +0200
Pierre Vittet wrote:
> Thank you for your answer!
>
> I send you a new patch
> I have corrected the errors you raised.
>
> I have make my patch compatible with the old use of c_register_pragma
> and c_register_pragma_with_expansion.
I find the patch quite in
Paolo Bonzini writes:
>> Seems like a plan, but I didn't go in this direction since I couldn't
>> test anything like this.
>
> As long as you test the general configury on 1-2 platforms, it's not any
> less tested than what you have now.
Unfortunately, that's not true: my original patch just mov
On 06/03/2011 04:35 PM, Rainer Orth wrote:
Apart from those changes, the patch addresses the PowerPC Darwin and
Windows32 issues as suggested:
* rs6000/darwin-unwind.h is wrapped in !__LP64__, while removing the
need for the !DARWIN_LIBSYSTEM_HAS_UNWIND test (only defined on Darwin
9 and u
Thank you for your answer!
I send you a new patch
I have corrected the errors you raised.
I have make my patch compatible with the old use of c_register_pragma
and c_register_pragma_with_expansion.
I don't know what is the best solution, maybe changing every call
c_register_pragma allows to
When checking in the Solaris toplevel libgcc patch on wednesday, I
notice that I still had LIB2ADDEH and LIB2ADDEHDEP in gcc/config/t-sol2.
The latter is completely unused now that libgcc does automatic
dependency generation and I had already removed it in
libgcc/config/t-sol2. This patch removes
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=677681 can be
“fixed” by disabling the artificial dependency of a debug insn on its
previous insn.
Debug insns will often depend on their prevs anyway, in a use/def
relationship, but if the def was (re)moved or the use was reset, this
artificial
As discussed in http://gcc.gnu.org/ml/gcc-patches/2011-06/msg00063.html,
this patch moves decl registration in symbol tables to the LTO front
end. It makes type and symbol registration happen at the same time in
uniquify_nodes.
Tested with LTO profiledbootstrap on x86_64. Committed to trunk.
On Tue, May 31, 2011 at 6:15 AM, Richard Guenther wrote:
>
> This patch, now that Micha made us less dependent on preloading
> exactly the same from each FE, gets rid of two hacks regarding
> to the frontend (and option) dependent char_type_node and
> boolean_type_node. It does so by first decoup
According to http://gcc.gnu.org/ml/gcc-patches/2009-11/msg00999.html
on Nov 19, 2009, Richard Guenther wrote:
> On Thu, Nov 19, 2009 at 4:05 AM, Alexandre Oliva wrote:
>> On Nov 17, 2009, Richard Guenther wrote:
>>
> This looks odd. SSA DEF operand iteration should walk the PHI defs
>
Paolo Bonzini writes:
> On 05/30/2011 05:43 PM, Rainer Orth wrote:
>> +md-unwind-support.h: config.status
>> +if test -n "$(md_unwind_header)"; then \
>> + echo "#include \"config/$(md_unwind_header)\""> $@; \
>> +else \
>> + :> $@; \
>> +fi
>
> Can you add a default file
According to http://gcc.gnu.org/ml/gcc-patches/2009-11/msg01082.html
on Nov 20, 2009, Alexandre Oliva wrote:
> On Nov 19, 2009, Richard Guenther wrote:
>> In fact this exchanging of the LHS (or rather invalidating of the
>> SSA name value) should be a helper function that knows
>> the implementa
On 06/02/2011 02:52 PM, Aldy Hernandez wrote:
> On 06/02/11 14:25, Jakub Jelinek wrote:
>
>>> +case MEMMODEL_SEQ_CST:
>>> + op = sync_mem_exchange_seq_cst_optab;
>>> + break;
>>> +case MEMMODEL_ACQUIRE:
>>> + op = sync_mem_exchange_acq_optab;
>>> + break;
>>> +case
A recent change introduced decl_uid in the “;; Function ” header in dump
files. This breaks -fcompare-debug (and bootstrap-debug-lean), because
decl uids aren't kept in sync between -g and non-g compilations.
This patch rearranges the header so that decl_uid is omitted when
dumping with NOUID, an
On Jun 2, 2011, Alexandre Oliva wrote:
> Right you are, though it looks like leaving the @hook lines out makes no
> difference. Anyhow, here's the patch I'm checking in.
Trunk is now fixed, is the patch ok for the 4.6 branch too?
> for gcc/ChangeLog
> from Alexandre Oliva
> PR debu
Since we are not sure whether pre-tokenized headers are worth
adding at all (the performance gains are minimal), Gabriel has
started removing that code and the parser instrumentation code we
had in the pph branch.
This will simplify branch maintenance wrt merges, but we are
still not sure whether
Hi!
This limits this testcase to i?86/x86_64 (moving to gcc.target/ would
be harder because it relies on all the weirdo vectorization options to be
passed), because apparently on strict alignment targets we don't handle
aligned (1) non-aggregates correctly. Or should it be instead xfailed
just on
Hi!
If memcpy is folded into an assignment, that assignment can be for C++
folded into nothing (if it is copying of 1 byte from or to empty C++ class).
gimple-fold.c was changed to handle that case in some spots, but not all,
particularly if that memcpy is the last stmt in a basic block (which can
On Fri, Jun 3, 2011 at 5:31 AM, Richard Guenther
wrote:
> On Fri, Jun 3, 2011 at 6:31 AM, H.J. Lu wrote:
>> On Wed, May 18, 2011 at 8:57 AM, H.J. Lu wrote:
>>> On Tue, Apr 26, 2011 at 6:05 AM, H.J. Lu wrote:
On Thu, Mar 31, 2011 at 7:57 AM, H.J. Lu wrote:
> On Mon, Mar 21, 2011 at 11:
Hi,
The attached patch, which includes test cases, fixes this bug by eliminating the
code which used floating point instructions to determine the 'r' value as
outlined in the Fortran standard under G formatting.
Essentially, the code now examines the d and e values to determine the number of
On 05/31/2011 02:18 PM, David Edelsohn wrote:
On Tue, May 31, 2011 at 12:08 PM, Pat Haugen
wrote:
> The following patch fixes an issue I noticed where vr0..vr2 were
> inadvertently included in NON_FLOAT_REGS.
>
> Bootstrap/regtest on powerpc64-linux with no new regressions. Ok for trunk?
>
On Fri, Jun 3, 2011 at 6:31 AM, H.J. Lu wrote:
> On Wed, May 18, 2011 at 8:57 AM, H.J. Lu wrote:
>> On Tue, Apr 26, 2011 at 6:05 AM, H.J. Lu wrote:
>>> On Thu, Mar 31, 2011 at 7:57 AM, H.J. Lu wrote:
On Mon, Mar 21, 2011 at 11:40 AM, H.J. Lu wrote:
> On Mon, Mar 14, 2011 at 12:28 PM,
On Fri, Jun 3, 2011 at 4:17 AM, Alexandre Oliva wrote:
> Ping? This fixes a case in which -g might change the executable code,
> exposed with bootstrap-debug-lean.
>
> On Apr 2, 2011, Alexandre Oliva wrote:
>
>> PR debug/48333
>> * calls.c (emit_call_1): Prefer the __builtin declara
On Thu, Jun 2, 2011 at 6:06 PM, Ira Rosen wrote:
> On 26 May 2011 10:52, Ira Rosen wrote:
>> Hi,
>>
>> The vectorizer supports strided loads with gaps, e.g., when only a[4i]
>> and a[4i+2] are accessed, it generates a vector load a[4i:4i+3], i.e.,
>> creating an access to a[4i+3], which doesn't e
> Hmm... Maybe it could, I'm not sure. The problem is that DCE removes
> insns, and then DF associates remaining uses in debug insns to earlier
> DEFs. Adjusting debug insns in DCE is right per the VTA design motto:
> decide as if debug insns weren't there, adjust them as you would adjust
> non-
On Thu, 2011-06-02 at 17:41 +0100, Julian Brown wrote:
> On Wed, 01 Jun 2011 17:00:30 +0100
> Richard Earnshaw wrote:
>
> >
> > On Wed, 2011-06-01 at 16:49 +0100, Julian Brown wrote:
> > > This patch tweaks the behaviour of arm_final_prescan_insn when
> > > tuning for Cortex-A5 cores, since bra
On 06/02/11 18:13, Ian Lance Taylor wrote:
Works for me, but please add an @xref{Spec Files} (you might need a
document in there too, not sure) as a pointer to where the spec format
is documented.
ok, this is what I committed, cribbing from the rs6000 xref in the same file.
nathan
--
Nathan
On 01/06/11 18:27, Ramana Radhakrishnan wrote:
2011-05-31 Ramana Radhakrishnan
* config/arm/neon.md (orndi3_neon): Actually split it.
This is what I committed in the end after a round of testing.
Ramana
Index: gcc/config/arm/neon.md
==
On 3 June 2011 02:55, Jason Merrill wrote:
> On 06/02/2011 06:02 PM, Jonathan Wakely wrote:
>>
>> + if (!dtor || !DECL_VINDEX (dtor))
>
> Do we really want to warn about the case where the class has no/trivial
> destructor?
I think so. Definitely if it's an abstract class, since it's
Latest results for 4.3.x.
-tgc
Testresults for 4.3.5:
sparc-sun-solaris2.7
Index: buildstat.html
===
RCS file: /cvs/gcc/wwwdocs/htdocs/gcc-4.3/buildstat.html,v
retrieving revision 1.34
diff -u -r1.34 buildstat.html
--- buildstat.ht
Latest results for 4.6.x
-tgc
Testresults for 4.6.0:
i686-pc-linux-gnu
x86_64-unknown-linux-gnu
Index: buildstat.html
===
RCS file: /cvs/gcc/wwwdocs/htdocs/gcc-4.6/buildstat.html,v
retrieving revision 1.3
diff -u -r1.3 buildstat.
Latest results for 4.5.x
-tgc
Testresults for 4.5.3:
armv7l-unknown-linux-gnueabi
hppa2.0w-hp-hpux11.00
hppa2.0w-hp-hpux11.11
hppa64-hp-hpux11.00
hppa64-hp-hpux11.11
i386-pc-solaris2.8
i686-pc-linux-gnu (2)
ia64-unknown-linux-gnu
sparc-sun-solaris2.8
x86_64-unknown-linux-gnu (
Latest results for 4.4.x.
-tgc
Testresults for 4.4.6:
alphaev68-dec-osf5.1a
hppa2.0w-hp-hpux11.11
hppa64-hp-hpux11.11
sparc-sun-solaris2.7
Index: buildstat.html
===
RCS file: /cvs/gcc/wwwdocs/htdocs/gcc-4.4/buildstat.html,v
r
79 matches
Mail list logo
