On Sun, Nov 6, 2011 at 12:16 AM, Aldy Hernandez al...@redhat.com wrote:
[rth, see below]
Index: gcc/attribs.c
===
--- gcc/attribs.c (.../trunk) (revision 180744)
+++ gcc/attribs.c
On Sun, Nov 6, 2011 at 4:41 AM, Aldy Hernandez al...@redhat.com wrote:
Well - we usually don't grab bits off the tree nodes lightly. Especially
if
the cgraph seems to be more fit.
If this is a suggestion, I can put it on my laundry list of future things
todo (after merge, 4.8?, etc).
Hi,
2011/11/5 Richard Henderson r...@redhat.com:
+ if (!TARGET_OPT_AGU || optimize_function_for_size_p (cfun))
Surely optimize_insn_for_size_p (), so that cold blocks are optimized for
size.
OK.
+ else if (ix86_use_lea_for_mov(insn, operands))
+ return lea{q}\t{%a1, %0|%0, %a1};
On Sat, 5 Nov 2011, Aldy Hernandez wrote:
I believe this should become 2009, 2011 or 2009, 2010, 2011
when it's applied to trunk.
I assume the same thing goes for the rest of similar files.
Yes, that's my understanding, I just didn't want to bother you with
a response to every single patch.
Hello,
2011/11/5 Eric Botcazou ebotca...@adacore.com:
Here is a patch which fixes redundant zero extensions problem. Issue
is resolved by expanding implicit_zee pass functionality to cover zero
and sign extends of different modes. Could please someone review it?
Could you explain the
Hi,
In ChangeLog is mentioned a change to config/config.gcc, which doesn't
exist. I assume the change is meant for config.gcc within gcc/
folder, but also for this file no change was applied. By this
f16cintrin.h header isn't installed at all.
Cuprit patch is:
2011-11-05 Quentin Neill
On Sun, Nov 6, 2011 at 11:50 AM, Ilya Enkovich enkovich@gmail.com wrote:
Hello,
2011/11/5 Eric Botcazou ebotca...@adacore.com:
Here is a patch which fixes redundant zero extensions problem. Issue
is resolved by expanding implicit_zee pass functionality to cover zero
and sign extends of
Sounds like what everything needs is a differently named enum: say
three_way_logic.
Well, one might just rename the present enum 'match' (which anyway
lacks the usual 'gfc' prefix), to something like 'gfc_three_way_logic'
(or whatever name you prefer), with appropriately named values. Then
one
2011/11/3 Steve Kargl s...@troutmask.apl.washington.edu:
On Thu, Nov 03, 2011 at 10:56:47PM +0100, Janus Weil wrote:
At least add a comment about the re-use (abuse?) of the
enum.
Updated patch attached, which adds a short comment on the usage of 'match'.
Thanks.
This should reduce
On 11/06/2011 03:19 AM, Dimitrios Apostolou wrote:
I understand major hassle is when the register file is big, too much
data is being copied on a function call, when it has a HARD_REG_SET as a
pass by value parameter. So I did some testing on SPARC, which has the
biggest register file I know
On Sun, Nov 06, 2011 at 01:12:21PM +0100, Paolo Bonzini wrote:
On 11/06/2011 03:19 AM, Dimitrios Apostolou wrote:
I understand major hassle is when the register file is big, too much
data is being copied on a function call, when it has a HARD_REG_SET as a
pass by value parameter. So I did
This fixes ipa-prop to properly re-compute inlinability which
can change from false to true if a mismatched argument is dropped.
Bootstrapped and tested onx 86_64-unknown-linux-gnu, applied.
Richard.
2011-11-6 Richard Guenther rguent...@suse.de
* ipa-prop.c
2011/11/6 Richard Guenther richard.guent...@gmail.com:
On Sun, Nov 6, 2011 at 11:50 AM, Ilya Enkovich enkovich@gmail.com wrote:
Hello,
2011/11/5 Eric Botcazou ebotca...@adacore.com:
Here is a patch which fixes redundant zero extensions problem. Issue
is resolved by expanding implicit_zee
There is rolled loop algorithm, that doesn't use SSE-modes - such
architectures could use it instead of unrolled_loop. I think the
performance wouldn't suffer much from that.
For the most of modern processors, SSE-moves are faster than several
word-sized moves, so this change in
On Fri, 2011-11-04 at 15:34 +0100, Michael Matz wrote:
Hi,
On Thu, 3 Nov 2011, Aldy Hernandez wrote:
+2010-05-28 Aldy Hernandez al...@redhat.com
+
+ * target-def.h (TARGET_VECTORIZE_BUILTIN_TM_LOAD): Define.
+ (TARGET_VECTORIZE_BUILTIN_TM_STORE): Same.
+ (TARGET_VECTORIZE):
On 11/06/11 06:05, Torvald Riegel wrote:
The patch adds unified changelogs. (Before merging, we would remove the
ChangeLog.tm-merge files and add their contents to the respective
ChangeLog files). libitm/ChangeLog would stay as is, I suppose.
OK for branch?
Thanks so much.
OK.
On Sat, 5 Nov 2011, amyl...@gcc.gnu.org wrote:
Propchange: trunk/gcc/config/epiphany/epiphany-modes.def
('svn:executable' added)
Propchange: trunk/gcc/config/epiphany/epiphany-protos.h
('svn:executable' added)
Propchange: trunk/gcc/config/epiphany/epiphany.c
Quoting Joseph S. Myers jos...@codesourcery.com:
All these properties look wrong; I see no reason for any of those files to
be executable.
Indeed. I didn't know svn add would do that.
I've removed these spurious properties.
Just checked in the patches. I will also post the exact patches now.
I am also checking out mainline and building it again.
Andrew
On 11/06/11 07:28, Torvald Riegel wrote:
On Fri, 2011-11-04 at 22:48 +0100, Torvald Riegel wrote:
On Thu, 2011-11-03 at 20:38 +, Joseph S. Myers wrote:
Make sure that you do need each #include present in this and any other new
file. Since 2008 a lot of includes of tm.h and toplev.h have
On Thu, 2011-11-03 at 20:15 +, Joseph S. Myers wrote:
Do you need a FLAGS_TO_PASS setting as in
http://gcc.gnu.org/ml/gcc-patches/2011-09/msg01187.html? (The way to
test is to do a multilib build and install, passing infodir=/some/where on
the make install line, and see if the manual
I just realized that my patch email did not come through - however, I
did not get any reject email.
Let's try first without patch - it's available at
http://users.physik.fu-berlin.de/~tburnus/tmp/constructor.diff
Tobias
Rouson, Damian wrote:
Bravo! Thanks for all the hard work, Tobias.
Also the following failed. Thus, the patch links again:
- http://users.physik.fu-berlin.de/~tburnus/tmp/constructor.diff
- Release notes:
http://users.physik.fu-berlin.de/~tburnus/tmp/releasenotes.diff
Tobias
PS: I really hate that the email get's simply dropped without any reject
email or
On Sun, 2011-11-06 at 06:09 -0800, Aldy Hernandez wrote:
On 11/06/11 06:05, Torvald Riegel wrote:
The patch adds unified changelogs. (Before merging, we would remove the
ChangeLog.tm-merge files and add their contents to the respective
ChangeLog files). libitm/ChangeLog would stay as is, I
On Fri, Nov 4, 2011 at 9:43 AM, Alan Modra amo...@gmail.com wrote:
This patch fixes PR30282, caused by instructions being scheduled over
the stack reset. Note that only for ABI_V4 do we currently have
frame_reg_rtx != sp_reg_rtx in rs6000_emit_stack_reset, so the patch
doesn't emit *less*
As has been suggested by Alexandre in the PR, this patch allows merging
basic blocks which couldn't be merged before because of user (non-forced)
labels at the beginning of the second basic blocks.
With this patch the user label is thrown away (for -g0 or -g
-fno-var-tracking-assignments) or
Hi,
duplicate diagnostics for (a pretty common error, I would guess):
void f() = delete;
void g() { f(); }
47695.C: In function ‘void g()’:
47695.C:2:12: error: use of deleted function ‘void f()’
47695.C:1:6: error: declared here
47695.C:2:14: error: use of deleted function ‘void f()’
Hi Tobias,
I'm just back from holiday, which it took me a bit longer to reply.
Actually, the test case is *not* OK.
If one compiles the original test case of the PR (or your
workshare2.f90) with -O and looks at -fdump-tree-original, one finds:
#pragma omp parallel default(shared)
Am 06.11.2011 17:26, schrieb Tobias Burnus:
I just realized that my patch email did not come through - however, I
did not get any reject email.
Let's try first without patch - it's available at
http://users.physik.fu-berlin.de/~tburnus/tmp/constructor.diff
Possibly it is too large - it's
Last try: Also gzip the release notes - let's see whether it mailserver
accepts that email.
Tobias
PS: I really hate that the email get's simply dropped without any
reject email or any other status. Seemingly, my other emails without
patches go through!
I just realized that my patch email
On Fri, 2011-11-04 at 22:48 +0100, Torvald Riegel wrote:
On Thu, 2011-11-03 at 20:38 +, Joseph S. Myers wrote:
Make sure that you do need each #include present in this and any other new
file. Since 2008 a lot of includes of tm.h and toplev.h have been removed
and diagnostic-core.h
Am 06.11.2011 17:26, schrieb Tobias Burnus:
I just realized that my patch email did not come through - however, I
did not get any reject email. Let's try first without patch - it's
available at
http://users.physik.fu-berlin.de/~tburnus/tmp/constructor.diff
I wondered whether the patch
Attached patch applied.
2011-11-06 François Dumont fdum...@gcc.gnu.org
* testsuite/performance/23_containers/insert_erase/41975.cc: Add
tests to check performance with or without cache of hash code
and with
string type that has a costlier hash functor than int type.
This is fixes a trunk build error noticed on hppa-linux at -j4. Tested on
hppa-unknown-linux-gnu and hppa64-hp-hpux11.11.
Ok?
Dave
--
J. David Anglin dave.ang...@nrc-cnrc.gc.ca
National Research Council of Canada (613) 990-0752 (FAX: 952-6602)
Last try: Also gzip the release notes - let's see whether it mailserver
accepts that email.
Tobias
PS: I really hate that the email get's simply dropped without any
reject email or any other status. Seemingly, my other emails without
patches go through!
I just realized that my patch email
On 11/06/2011 09:58 AM, Andrew MacLeod wrote:
Just checked in the patches. I will also post the exact patches
now. I am also checking out mainline and building it again.
Andrew
2011-11-06 Andrew MacLeod amacl...@redhat.com
Aldy Hernandez al...@redhat.com
On 11/06/2011 09:58 AM, Andrew MacLeod wrote:
Just checked in the patches. I will also post the exact patches
now. I am also checking out mainline and building it again.
Andrew
Index: gcc/testsuite/gcc.dg/gomp/atomic-14.c
On 11/06/2011 09:58 AM, Andrew MacLeod wrote:
Just checked in the patches. I will also post the exact patches
now. I am also checking out mainline and building it again.
Andrew
2011-11-06 Andrew MacLeod amacl...@redhat.com
Richard Henderson r...@redhat.com
Hi,
duplicate diagnostics for (a pretty common error, I would guess):
void f() = delete;
void g() { f(); }
47695.C: In function ‘void g()’:
47695.C:2:12: error: use of deleted function ‘void f()’
47695.C:1:6: error: declared here
47695.C:2:14: error: use of deleted function ‘void f()’
Ping .. (the Nov 7 cutoff date is close ..).
thanks,
David
On Sat, Nov 5, 2011 at 12:22 PM, Xinliang David Li davi...@google.com wrote:
thanks. The attached is the revised patch.
David
On Sat, Nov 5, 2011 at 11:52 AM, Paolo Carlini paolo.carl...@oracle.com
wrote:
On 11/05/2011 07:32 PM,
On Wed, Nov 2, 2011 at 3:42 PM, Richard Guenther rguent...@suse.de wrote:
On Sat, 29 Oct 2011, Sergey Ostanevich wrote:
On Fri, Oct 28, 2011 at 7:25 PM, Sergey Ostanevich sergos@gmail.com
wrote:
On Fri, Oct 28, 2011 at 4:52 PM, Richard Guenther rguent...@suse.de
wrote:
On Fri, 28
Hi,
This patch adds a support of conditions in SLP.
It also fixes a bug in pattern handling in SLP (we should put pattern
statements instead of original statements in the root), and allows
pattern def-stmts in SLP.
Bootstrapped on powerpc64-suse-linux and tested on
powerpc64-suse-linux and
[rth, more comments for you below]
On 11/04/11 04:14, Richard Guenther wrote:
new_version = cgraph_create_node (new_decl);
- new_version-analyzed = true;
+ new_version-analyzed = old_version-analyzed;
Hm? analyzed means with body, sure you have a body if you clone.
On 11/05/2011 07:39 PM, David Miller wrote:
Richard, is there a better way to represent this in RTL? These
instructions basically load a single byte or half-word into the bottom
of a 64-bit float register, and clear the rest of that register with
zeros. So the v4hi one is essentially loading
On 11/06/2011 02:12 AM, Ilya Enkovich wrote:
It would be great to have computed type here but ix86_use_lea_for_mov
will check types of other instructions and then call
extract_insn_cached. It will cause infinite loop, right?
Hum. That does cause a problem.
No objection to the patch approved
Xinliang David Li davi...@google.com writes:
Ping .. (the Nov 7 cutoff date is close ..).
The cutoff date is typically not rigorously applied to patches submitted
before the deadline but approved after the deadline.
However, this patch is OK.
Thanks.
Ian
On Sat, Nov 5, 2011 at 12:22 PM,
On 11/06/2011 06:43 PM, John David Anglin wrote:
This is fixes a trunk build error noticed on hppa-linux at -j4. Tested on
hppa-unknown-linux-gnu and hppa64-hp-hpux11.11.
Ok?
Ok.
Paolo
This update is to allow libfunc initialization for other target
features.
Tested on hppa64-hp-hpux11.11. Committed to trunk.
Dave
--
J. David Anglin dave.ang...@nrc-cnrc.gc.ca
National Research Council of Canada (613) 990-0752 (FAX: 952-6602)
On Thu, 2011-11-03 at 20:15 +, Joseph S. Myers wrote:
On Thu, 3 Nov 2011, Aldy Hernandez wrote:
Index: libitm/acinclude.m4
===
--- libitm/acinclude.m4 (.../trunk) (revision 0)
+++ libitm/acinclude.m4
On Sun, Nov 06, 2011 at 06:14:18PM +0100, Eric Botcazou wrote:
As has been suggested by Alexandre in the PR, this patch allows merging
basic blocks which couldn't be merged before because of user (non-forced)
labels at the beginning of the second basic blocks.
With this patch the user
As per your comment in the PR...
OK for branch?
* opts.c (finish_options): Error out when using -flto and
-fgnu-tm.
Index: opts.c
===
--- opts.c (revision 181028)
+++ opts.c (working copy)
@@ -784,6 +784,8
Following http://gcc.gnu.org/ml/gcc-patches/2011-10/msg02901.html, I have
applied
the following patch on x86_64-apple-darwin10
--- ../_clean/gcc/config.gcc2011-11-05 22:25:37.0 +0100
+++ gcc/config.gcc 2011-11-06 12:35:57.0 +0100
@@ -350,7 +350,7 @@ i[34567]86-*-*)
On 11/06/2011 10:53 AM, Aldy Hernandez wrote:
Did you check bootstrapping GCC with TM enabled and address-space
randomization turned on?
Actually, the table organization is irrelevant, because upon
registering of the table in the runtime, we qsort the entire thing.
False. You get the
OK.
Jason
On Sun, Nov 6, 2011 at 15:03, Aldy Hernandez al...@redhat.com wrote:
As per your comment in the PR...
OK for branch?
LTO support is currently not supported with transactional memory
'support' mentioned one too many times. Maybe 'LTO is currently not
supported with transactional memory'?
On Sun, Nov 06, 2011 at 03:48:16PM -0500, Joern Rennecke wrote:
Or keep HARD_REG_SET type as is and just use a new struct type which
contains HARD_REG_SET or HARD_REG_SET * in it.
struct hard_reg_set_ptr;
void (*live_on_entry) (struct hard_reg_set_ptr *);
in the target* headers and
struct
LTO support is currently not supported with transactional memory
'support' mentioned one too many times. Maybe 'LTO is currently not
supported with transactional memory'?
Diego.
How is this?
* opts.c (finish_options): Error out when using -flto and
-fgnu-tm.
Index: opts.c
On 11-11-06 16:05 , Aldy Hernandez wrote:
LTO support is currently not supported with transactional memory
'support' mentioned one too many times. Maybe 'LTO is currently not
supported with transactional memory'?
Diego.
How is this?
OK. Thanks.
Diego.
On 11/04/11 04:14, Richard Guenther wrote:
new_version = cgraph_create_node (new_decl);
- new_version-analyzed = true;
+ new_version-analyzed = old_version-analyzed;
Hm? analyzed means with body, sure you have a body if you clone.
Incidentally, for TM we also clone functions that
Actually, the table organization is irrelevant, because upon
registering of the table in the runtime, we qsort the entire thing.
False. You get the equivalent of bootstrap comparison mismatches.
If we actually used tm during the bootstrap.
The simplest thing to do is to change the hash this
I think the patch is fine and can be committed. But, give
Steven a chance to respond before committing.
Thanks, Steve. I think three days should be long enough. Will commit
later today (if no one protests in the meantime).
Committed as r181044.
Cheers,
Janus
Quoting Jakub Jelinek ja...@redhat.com:
The middle-end uses HARD_REG_SET in lots of places, so supposedly
when you want to support more than one target, you need to ensure
that hard-reg-set.h header won't use FIRST_PSEUDO_REGISTER of one randomly
selected target you want to support, but instead
On Sun, 6 Nov 2011, Joern Rennecke wrote:
But where HARD_REG_SETS make no material difference in speed, and the
compilation unit has no other tight coupling with tm.h, it would really
be cleaner to move from HARD_REG_SETS to a target-independent type,
like sbitmap or bitmap. Maybe we want
We don't provide cstdalign or cuchar - the latter is listed as C
library dependency but AFAIK we don't provide cuchar even if
uchar.h is present.
* doc/xml/manual/status_cxx2011.xml: Document cuchar and
cstdalign as missing.
Committed to trunk.
Index:
Hello,
By this patch branch-cost optimization is moved from tree AST to cfgexpand from
gimple to RTL. By this we are able to do better optimization on conditionals
simliar for all targets and do the final transition for branch-cost that late
it shows best effect.
This patch is splitted up
Hello,
the second patch extends the tree-ssa-ifcombine pass so, that it chains up
simple if-and/or-if patterns via associative bitwise-and/or operations. This
allows for example optimization for cases like:
if (c == 0) return 2;
if (c == 1) return 2;
if (c == 2) return 2;
...
as now
Quoting Dimitrios Apostolou ji...@gmx.net:
Working with bitmaps in gcc made me really miss the simple bitmap.h of
the linux kernel. I'm not sure if we can borrow from there. More info
at:
http://lxr.linux.no/#linux+v3.1/include/linux/bitmap.h
You would need the Copyright holders to assign
On Mon, Nov 07, 2011 at 12:01:29AM +0200, Dimitrios Apostolou wrote:
On Sun, 6 Nov 2011, Joern Rennecke wrote:
But where HARD_REG_SETS make no material difference in speed, and the
compilation unit has no other tight coupling with tm.h, it would really
be cleaner to move from HARD_REG_SETS to
On Sun, 6 Nov 2011, Torvald Riegel wrote:
Is the attached patch what you'd like to see? It doesn't yet use the
It's plausible, but really a build system maintainer should look at it.
--
Joseph S. Myers
jos...@codesourcery.com
Hi all,
up to now we call all type-bound procedures in a dynamic way, i.e.
through their entry in the vtable. However, for non-overridable
procedures this is not necessary. Since they can not be overridden, a
call to those can be resolved at compile time to an ordinary function
call, without the
The autoconf examples at
http://gcc.gnu.org/onlinedocs/libstdc++/manual/backwards.html#id541888
are incorrect, relying on pre-standard semantics for rvalue
references, and missing out several C++11 headers. This patch
corrects those examples.
This also tweaks some markup in the file, and adds
On Mon, 7 Nov 2011, Jakub Jelinek wrote:
On Mon, Nov 07, 2011 at 12:01:29AM +0200, Dimitrios Apostolou wrote:
On Sun, 6 Nov 2011, Joern Rennecke wrote:
But where HARD_REG_SETS make no material difference in speed, and the
compilation unit has no other tight coupling with tm.h, it would really
On Sun, 2011-11-06 at 23:04 +, Joseph S. Myers wrote:
On Sun, 6 Nov 2011, Torvald Riegel wrote:
Is the attached patch what you'd like to see? It doesn't yet use the
It's plausible, but really a build system maintainer should look at it.
So, can we keep this as-is then and fix this
This patch adds support for most of the C1X alignment features
(_Alignas, _Alignof, max_align_t, stdalign.h).
_Alignof is essentially GNU __alignof__, but restricted to be applied
to types not expressions and not allowed on function types. It goes
through the existing code, but with diagnostics
On Sun, Nov 6, 2011 at 12:03 PM, Aldy Hernandez al...@redhat.com wrote:
As per your comment in the PR...
OK for branch?
Why not just fix the issue instead of erroring out? No other option
has issues with LTO other than TM. In fact I think this should have
been a merge blocker really.
I first posted this a month ago, this adjusts the exports and moves
the tests under separate char and wchar_t directories as requested by
Paolo.
* include/bits/basic_string.h (basic_string::at): Move adjacent to other
overload.
(basic_string::pop_back): Define.
*
On Sun, Nov 6, 2011 at 18:56, Andrew Pinski pins...@gmail.com wrote:
Why not just fix the issue instead of erroring out? No other option
has issues with LTO other than TM. In fact I think this should have
been a merge blocker really.
I disagree. TM is a new experimental feature. It is
On 6 November 2011 23:53, Joseph S. Myers wrote:
As with stdnoreturn.h, the contents of stdalign.h are conditioned out
for C++; I'll leave it to C++ people to work out what's most useful
there if something nonempty is wanted (stdnoreturn.h is empty for C++,
stdbool.h defines _Bool and bool to
On Mon, 7 Nov 2011, Jonathan Wakely wrote:
On 6 November 2011 23:53, Joseph S. Myers wrote:
As with stdnoreturn.h, the contents of stdalign.h are conditioned out
for C++; I'll leave it to C++ people to work out what's most useful
there if something nonempty is wanted (stdnoreturn.h is
From: Andrew MacLeod amacl...@redhat.com
Date: Fri, 4 Nov 2011 00:50:47 +0100
These are the changes to libstdc++ to make use of the new atomics. I
changed the files to use the new atomics, and bkoz did a shuffling of
the include file layout to better suit the new c++ approach.
Hi list,
The win32 crtbegin code needs to know the full name of the libgcj DLL,
including in particular the trailing version suffix generated from the libtool
version info, and so far it's not autogenerated but needs to be manually
synced. This patch updates the two places where we have
On 6 November 2011 22:10, Jonathan Wakely wrote:
We don't provide cstdalign or cuchar - the latter is listed as C
library dependency but AFAIK we don't provide cuchar even if
uchar.h is present.
* doc/xml/manual/status_cxx2011.xml: Document cuchar and
cstdalign as missing.
From: Richard Henderson r...@redhat.com
Date: Sun, 06 Nov 2011 09:55:17 -0800
On 11/05/2011 07:39 PM, David Miller wrote:
Richard, is there a better way to represent this in RTL? These
instructions basically load a single byte or half-word into the bottom
of a 64-bit float register, and
On 11/06/11 12:20, Richard Henderson wrote:
- if (!computed_goto_p (stmt))
+ if (!computed_goto_p (stmt))
{
- tree new_dest = main_block_label (gimple_goto_dest (stmt));
- gimple_goto_set_dest (stmt, new_dest);
+ label =
On 11/04/2011 03:30 PM, Sebastian Huber wrote:
On 11/04/2011 01:57 PM, Sebastian Huber wrote:
It builds well and the test suite runs currently.
http://gcc.gnu.org/ml/gcc-testresults/2011-11/msg00407.html
The second version of your patch is OK with me and seems to work fine.
Patch commited
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1
On 11/03/11 17:40, Aldy Hernandez wrote:
On 11/03/11 18:30, Andrew Pinski wrote:
On Thu, Nov 3, 2011 at 3:12 PM, Aldy Hernandezal...@redhat.com
wrote:
These are known failures, mostly missed optimizations.
XFAILing them.
I think you should
On 11/06/2011 07:38 PM, Hans-Peter Nilsson wrote:
This (formally a change in the range 181027:181034) got me three
libstdc++ regressions for cris-elf, which has no atomic
support whatsoever (well, not the version represented in
cris-elf), so something is amiss at the bottom of the default
path:
The function constrain_visibility_for_template tries to set the
visibility of a template instantiation properly by giving it the minimum
visibility of the template itself and the template arguments. But this
PR points out that we were failing to do that in the case that the
template is within
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1
On 11/04/11 17:37, DJ Delorie wrote:
The only way I can think of to have two pseudos assigned the
same hard reg at the same point in the insn stream is if the two
pseudos are known to have the same value.
Since all we're doing is figuring out
The lookup rules require that all the operators for a given name be
searched for one that has the exact same arguments implied by the literal.
Also, an error concerning the length of raw operator strings was fixed.
Index: gcc/testsuite/g++.dg/cpp0x/udlit-raw-length.C
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1
On 11/04/11 14:23, Peter Bergner wrote:
On Fri, 2011-11-04 at 12:25 -0600, Jeff Law wrote:
The only way I can think of to have two pseudos assigned the same
hard reg at the same point in the insn stream is if the two
pseudos are known to have the
On Sun, Nov 6, 2011 at 2:13 PM, Dominique Dhumieres domi...@lps.ens.fr wrote:
Following http://gcc.gnu.org/ml/gcc-patches/2011-10/msg02901.html, I have
applied
the following patch on x86_64-apple-darwin10
--- ../_clean/gcc/config.gcc 2011-11-05 22:25:37.0 +0100
+++ gcc/config.gcc
False. You get the equivalent of bootstrap comparison mismatches.
If we actually used tm during the bootstrap.
The simplest thing to do is to change the hash this table uses.
E.g. use the DECL_UID right from the start, rather than the pointer.
Woah! Can it be that easy? That's as easy as
Sorry about the delay. I have updated the patch to output demangled
names under a new option (-m) and added a test case. Okay for trunk?
Sharad
2011-11-06 Sharad Singhai sing...@google.com
* doc/gcov.texi: Document gcov intermediate format.
* gcov.c (print_usage): Handle new
Dear Janus,
On Mon, Nov 7, 2011 at 12:14 AM, Janus Weil ja...@gcc.gnu.org wrote:
The patch actually consists of two parts:
1) The resolve.c part prevents the conversion to a PPC call via the
_vptr (for functions and subroutines).
This is obviously OK
2) The class.c parts prevents adding
I have seen compiler build error (segmentation fault) in libstdc++-v3.
It turns out that a vector allocated in gc memory is GCed before the
vector is released. The gc call is from a call to synethesize_method
from cp_finish_decl.
The following patch fixes the problem. Compiler bootstraps and
This is necessary for systems that treat all system headers as
implicitly extern C (at least AIX and OpenBSD) because otherwise the
empty parameter list is treated as declaring void __once_proxy(...)
This has come up recently because we are now enabling mutex etc. on
additional platforms.
97 matches
Mail list logo
