I have added a person from AMD to comment on the decision.
Otherwise, the patch looks OK, but please wait a couple of days for possible
comments.
Thank you Uros!
I am checking the changes with few tests and benchmarking them.
Please wait for a couple of days.
-Ganesh
On Sat, 6 Dec 2014, Jonathan Wakely wrote:
This adds recent libstdc++ updates to gcc-5/changes.html
Nice! Just a most minor change to end a list with a full stop
instead of a semi-colon. Applied.
Gerald
Index: changes.html
Am 04/08/2015 um 10:28 AM schrieb Denis Chertykov:
2015-04-07 15:34 GMT+03:00 Georg-Johann Lay a...@gjlay.de:
Am 04/06/2015 um 11:54 AM schrieb Sivanupandi, Pitchumani:
Hi Johann,
Did you try running g++ tests?
It seems xgcc is invoked to get multilibs (from gcc/testsuite/lib/g++.exp)
which
Hello,
Patch in the bottom fixes PR65676.
Bootstrapped, reg-testing is in progress.
I am going to commit if testing will pass.
I am also going back port the patch to 4.9.x.
gcc/
PR target/65676
* config/i386/i386.c (fixup_modeless_constant): New.
On 08/04/15 13:06 +0200, Gerald Pfeifer wrote:
On Sat, 6 Dec 2014, Jonathan Wakely wrote:
I'm also noting one old change in the GCC 4.5 page, and
removing/changing some links to the C++0x status table. The list of
features supported on trunk is fairly irrelevant to someone looking at
the 4.4
Le 07/04/2015 14:25, Mikael Morin a écrit :
Le 06/04/2015 20:26, Mikael Morin a écrit :
Regarding the patch, I don't understand why the existing symbol
restoration code doesn't work here (see
gfc_restore_last_undo_checkpoint, restore_old_symbol). I have to
investigate more.
I think the
On Thu, 10 Nov 2011, Hans-Peter Nilsson wrote:
I think I need someone with appropriate write privileges to
agree with that, and to also give 48h for someone to fix the
problem. Sorry for not forthcoming on the second point.
brgds, H-P
PS. where is the policy written down, besides the
On Sat, 6 Dec 2014, Jonathan Wakely wrote:
I'm also noting one old change in the GCC 4.5 page, and
removing/changing some links to the C++0x status table. The list of
features supported on trunk is fairly irrelevant to someone looking at
the 4.4 release notes, so I've linked to the docs for
On Mon, 6 Apr 2015, Sandra Loosemore wrote:
s/frontend/front end/ ??
(Since we have middle end and back end on the previous line.)
Absolutely -- I should have caught this. Is also is in line with
https://gcc.gnu.org/codingconventions.html.
Here is the updated patch I just committed.
Hi!
The following patches (included or attached) fix a regression on WebKit
compilation.
The first hunk basically reverts Honza's patch from December/January,
because layout_type when the variants are already linked in doesn't layout
just the current type, but also forcefully overwrites all the
Hi Sandra,
On Mon, 26 Jan 2015, Sandra Loosemore wrote:
OK, here is a patch that attempts to implement that convention. I'd
appreciate review from a target maintainer to check that I've correctly
disambiguated places where i386 was referring to both 32- and 64-bit
variants vs 32-bit only.
Le 08/04/2015 12:29, Mikael Morin a écrit :
Except that the following ... ahem ... doesn't work.
And it doesn't work because gfc_get_ha_symbol doesn't version
host-associated symbols.
So one has to call symbol.c's save_symbol_data by hand. And then, we
can as well keep the original
On Wed, 25 Feb 2015, Chung-Ju Wu wrote:
Committed as revision 1.82 of htdocs/gcc-5/changes.html with minor
adjustment.
Thanks for adding those release notes! I just applied a number
of editorial changes on top of the original patch. Let me know
if there are further changes you'd like to see
2015-04-07 15:34 GMT+03:00 Georg-Johann Lay a...@gjlay.de:
Am 04/06/2015 um 11:54 AM schrieb Sivanupandi, Pitchumani:
Hi Johann,
Did you try running g++ tests?
It seems xgcc is invoked to get multilibs (from gcc/testsuite/lib/g++.exp)
which failed to find specs file.
This is because
On 07/04/15 23:58 -0400, Hans-Peter Nilsson wrote:
I'd expect
alignof(ai): 4 .is_lock_free(): 1
No... wait, that's because atomic_base.h doesn't have the
natural-alignment fix, so it's still broken for
less-than-natural-alignment targets. But will be fixed?
Yes, with my uncommitted patch to
This disables incorrect optimization (mainly CSE) of calls to Pure functions
returning a discriminated record type. These functions allocate their return
value on the secondary stack and thus calls to them cannot be CSE'ed because
the stack can be reclaimed in between.
Tested on
On Wed, 8 Apr 2015, Gopalasubramanian, Ganesh wrote:
I have added a person from AMD to comment on the decision.
Otherwise, the patch looks OK, but please wait a couple of days for
possible comments.
Thank you Uros!
I am checking the changes with few tests and benchmarking them.
Please
On 04/08/2015 06:02 AM, Jakub Jelinek wrote:
(cp_build_qualified_type_real): Use check_base_type. Build a
variant and copy over even TYPE_CONTEXT and
TYPE_ALIGN/TYPE_USER_ALIGN if any of those are different.
This seems wrong. If there is an array with the same name,
On Wed, Apr 08, 2015 at 06:22:10PM +0200, Jan Hubicka wrote:
On 04/08/2015 06:02 AM, Jakub Jelinek wrote:
(cp_build_qualified_type_real): Use check_base_type. Build a
variant and copy over even TYPE_CONTEXT and
TYPE_ALIGN/TYPE_USER_ALIGN if any of those are different.
This
On Wed, Apr 08, 2015 at 06:22:10PM +0200, Jan Hubicka wrote:
On 04/08/2015 06:02 AM, Jakub Jelinek wrote:
(cp_build_qualified_type_real): Use check_base_type. Build a
variant and copy over even TYPE_CONTEXT and
TYPE_ALIGN/TYPE_USER_ALIGN if any of those are different.
This
On Tue, 7 Apr 2015 17:26:45 +0200
Jakub Jelinek ja...@redhat.com wrote:
On Mon, Apr 06, 2015 at 03:45:57PM +0300, Ilya Verbin wrote:
On Wed, Apr 01, 2015 at 15:20:25 +0200, Jakub Jelinek wrote:
LGTM with proper ChangeLog entry.
I've commited this patch into trunk.
Julian, you
On Wed, Apr 08, 2015 at 15:31:42 +0100, Julian Brown wrote:
This version is mostly the same as the last posted version but has a
tweak in GOACC_parallel to account for the new splay tree arrangement
for target functions:
- tgt_fn = (void (*)) tgt_fn_key-tgt-tgt_start;
+ tgt_fn =
Hi!
Attached patch updates baseline_symbols.txt for a couple of architectures.
Don't have 32-bit powerpc-linux around though this time.
Ok for trunk?
2015-04-07 Jakub Jelinek ja...@redhat.com
* config/abi/post/x86_64-linux-gnu/baseline_symbols.txt: Update.
*
Some AVX512DQ tests are failing on Solaris/x86 with gas:
FAIL: gcc.target/i386/avx512dq-vfpclasspd-2.c (test for excess errors)
FAIL: gcc.target/i386/avx512dq-vfpclassps-2.c (test for excess errors)
FAIL: gcc.target/i386/avx512vl-vfpclasspd-2.c (test for excess errors)
FAIL:
Ian Lance Taylor i...@golang.org writes:
On Mon, Nov 3, 2014 at 8:59 AM, Rainer Orth r...@cebitec.uni-bielefeld.de
wrote:
The recent godump changes broke Solaris 11.1+ bootstrap in libgo:
before, gen-sysinfo.so had
type _zone_net_addr_t struct { zna_family uint16; zna_plen uint16;
On 07 Apr 22:37, Jan Hubicka wrote:
OK, thanks!
Please put the comment about inline clones just before the global.inlined_to
test
and make the comment about thunks separate.
Probably Thunks can not call across section boundary
Honza
Here is a committed variant.
Thanks,
Ilya
--
gcc/
Nuno Diegues n...@ist.utl.pt writes:
What workloads did you test this on?
+static inline float fastLog(float x)
+{
+ union { float f; uint32_t i; } vx = { x };
+ float y = vx.i;
+ y *= 8.2629582881927490e-8f;
+ return y - 87.989971088f;
+}
+
+static inline float fastSqrt(float x)
On Wed, Apr 8, 2015 at 6:48 AM, Rainer Orth r...@cebitec.uni-bielefeld.de
wrote:
Ian Lance Taylor i...@golang.org writes:
On Mon, Nov 3, 2014 at 8:59 AM, Rainer Orth r...@cebitec.uni-bielefeld.de
wrote:
The recent godump changes broke Solaris 11.1+ bootstrap in libgo:
before,
[CC ing maintainers]
Ping.
On 27 March 2015 at 11:24, tbsaunde+...@tbsaunde.org wrote:
From: Trevor Saunders tbsaunde+...@tbsaunde.org
Hi,
Unfortunately when testing r217869 I didn't realize the modified code in
alpha.c was only used for some alpha targets. So testing alpha-linux wasn't
I see there is some documentation about build constraints in
go/go/build/doc.go.
There is no mention of netgo on this page now but this seems like an
appropriate
spot to mention the use of the netgo tag for gc and then how to achieve the
equivalent effect with gccgo.
On 04/07/2015 01:09 PM,
On Fri, Mar 27, 2015 at 11:24 AM, tbsaunde+...@tbsaunde.org wrote:
From: Trevor Saunders tbsaunde+...@tbsaunde.org
Hi,
Unfortunately when testing r217869 I didn't realize the modified code in
alpha.c was only used for some alpha targets. So testing alpha-linux wasn't
enough or even really
With the GCC 5 release approaching, it's time to update the Solaris
baselines again.
This patch does just that and is pretty much straightforward:
* With one exception, all new symbols are in the GLIBCXX_3.4.21 and
CXXABI_1.3.9 versions.
* On Solaris/x86 only (obviously), we have
Ian Lance Taylor i...@golang.org writes:
On Wed, Apr 8, 2015 at 6:48 AM, Rainer Orth r...@cebitec.uni-bielefeld.de
wrote:
Ian Lance Taylor i...@golang.org writes:
On Mon, Nov 3, 2014 at 8:59 AM, Rainer Orth
r...@cebitec.uni-bielefeld.de wrote:
The recent godump changes broke Solaris
On Wed, Apr 08, 2015 at 03:57:09PM +0200, Bernhard Reutner-Fischer wrote:
[CC ing maintainers]
Ping.
This is ok.
--- a/gcc/ChangeLog
+++ b/gcc/ChangeLog
@@ -1,3 +1,9 @@
+2015-03-27 Trevor Saunders tbsau...@tbsaunde.org
+
+ * config/alpha/alpha.c (alpha_use_linkage):
On Wed, Apr 8, 2015 at 7:10 AM, Rainer Orth r...@cebitec.uni-bielefeld.de
wrote:
Ian Lance Taylor i...@golang.org writes:
On Wed, Apr 8, 2015 at 6:48 AM, Rainer Orth r...@cebitec.uni-bielefeld.de
wrote:
Ian Lance Taylor i...@golang.org writes:
On Mon, Nov 3, 2014 at 8:59 AM, Rainer Orth
On 04/08/2015 06:02 AM, Jakub Jelinek wrote:
(cp_build_qualified_type_real): Use check_base_type. Build a
variant and copy over even TYPE_CONTEXT and
TYPE_ALIGN/TYPE_USER_ALIGN if any of those are different.
This seems wrong. If there is an array with the same name,
On Wed, Apr 08, 2015 at 10:47:15AM -0400, Jason Merrill wrote:
On 04/08/2015 06:02 AM, Jakub Jelinek wrote:
(cp_build_qualified_type_real): Use check_base_type. Build a
variant and copy over even TYPE_CONTEXT and
TYPE_ALIGN/TYPE_USER_ALIGN if any of those are different.
This
On Wed, Apr 08, 2015 at 03:31:42PM +0100, Julian Brown wrote:
It passes libgomp testing on NVPTX. OK?
Please write a proper ChangeLog entry for it.
Ok with that.
Jakub
Thank you for the feedback. Comments inline.
On Wed, Apr 8, 2015 at 3:05 PM, Andi Kleen a...@firstfloor.org wrote:
Nuno Diegues n...@ist.utl.pt writes:
What workloads did you test this on?
On the STAMP suite of benchmarks for transactional memory (described here [1]).
I have ran an
On 08/04/15 19:40 +0200, Jakub Jelinek wrote:
Hi!
Attached patch updates baseline_symbols.txt for a couple of architectures.
Don't have 32-bit powerpc-linux around though this time.
Ok for trunk?
OK, thanks very much for doing this.
I ran into this while trying to help a user. Committed.
Gerald
2015-04-08 Gerald Pfeifer ger...@pfeifer.com
* doc/extend.texi (__sync Builtins): Fix grammar.
Index: doc/extend.texi
===
--- doc/extend.texi (revision
Two typos and remove two unneeded forward declarations for static
functions in tree-tailcall.c.
These patches were applied when doing a config-list.mk build and showed
no negative effect. I'd usually consider them obvious but given current
stage..
Ok for trunk now?
Bernhard Reutner-Fischer (2):
gcc/ChangeLog:
2015-04-01 Bernhard Reutner-Fischer al...@gcc.gnu.org
* tree.h (CONVERT_EXPR_P): Commentary typo fix.
Signed-off-by: Bernhard Reutner-Fischer rep.dot@gmail.com
---
gcc/tree.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/gcc/tree.h
gcc/ChangeLog:
2015-04-01 Bernhard Reutner-Fischer al...@gcc.gnu.org
* tree-tailcall.c (suitable_for_tail_opt_p, find_tail_calls):
Remove unneeded forward declarations.
(suitable_for_tail_call_opt_p): Commentary typo fix.
Signed-off-by: Bernhard Reutner-Fischer
I'm retiring, and my last day at google is this Friday, April 10. I
plan to continue to contribute to GCC and binutils in my retirement.
I've updated the MAINTAINERS file to use my personal address,
ccout...@gmail.com.
-cary
2015-04-08 Cary Coutant ccout...@gmail.com
* MAINTAINERS:
On Wed, 8 Apr 2015, Bernhard Reutner-Fischer wrote:
2015-04-01 Bernhard Reutner-Fischer al...@gcc.gnu.org
* tree.h (CONVERT_EXPR_P): Commentary typo fix.
Go ahead. Pure comment fixes still should be fine at this point
in time.
Gerald
building all-gcc for bfin-linux-uclibc results in
build/genchecksum cp/cp-lang.o c-family/stub-objc.o ... glibc-c.o \
libbackend.a .. cc1plus-checksum.c.tmp
opening glibc-c.o: No such file or directory
make[2]: *** [cc1-checksum.c] Error 1
Fix this by prepending tmake_file which nowadays
On 8 April 2015 at 20:34, Gerald Pfeifer ger...@pfeifer.com wrote:
On Wed, 8 Apr 2015, Bernhard Reutner-Fischer wrote:
2015-04-01 Bernhard Reutner-Fischer al...@gcc.gnu.org
* tree.h (CONVERT_EXPR_P): Commentary typo fix.
Go ahead. Pure comment fixes still should be fine at this
On Mon, 2 Feb 2015, Janis Johnson wrote:
I retired from Mentor Graphics 3 weeks ago and have no immediate plans
to be active in GCC, so I'm resigning as a testsuite maintainer. I'm
leaving myself under Write After Approval with my personal email address
so people can find me.
Thanks for your
Hi Michael,
and big apologies for this falling through a lot of cracks
apparently. I just committed your patch with the ChangeLog
below.
If there are any other patches that have not been committed
(nor NACKed yet, I know there were some as well), please let
us know and I will look into getting
On 08/04/15 18:59 +0200, Torvald Riegel wrote:
+// set or the maximum number of reader locks is held, then
increment the
+// reader lock count.
+// To release decrement the count, then if the write-entered flag
is set
+// and the count is zero then signal gate2 to wake a queued
On Wed, 8 Apr 2015 17:58:56 +0300
Ilya Verbin iver...@gmail.com wrote:
Have you tested it with disabled offloading?
I see several regressions:
FAIL: libgomp.oacc-c/../libgomp.oacc-c-c++-common/acc_on_device-1.c
-DACC_DEVICE_TYPE_host_nonshm=1 -DACC_MEM_SHARED=0 execution test
FAIL:
On 08/04/15 15:41 +0200, Rainer Orth wrote:
With the GCC 5 release approaching, it's time to update the Solaris
baselines again.
This patch does just that and is pretty much straightforward:
* With one exception, all new symbols are in the GLIBCXX_3.4.21 and
CXXABI_1.3.9 versions.
* On
On Sun, 16 Nov 2014, Andi Kleen wrote:
This patch describes some user visible changes that were
added to gcc 5.
Thanks, Andi!
I added some code environments, broke up a long one, changed
a LTO build to an LTO build,
Index: changes.html
There is an correctness issue related to mutex destruction. The added
documentation is a good start, but I'd still add some more for the
complicated pieces of reasoning. Details inline below.
On Tue, 2015-04-07 at 15:28 +0100, Jonathan Wakely wrote:
diff --git
On Wed, Apr 08, 2015 at 06:32:29PM +0200, Jan Hubicka wrote:
On Wed, Apr 08, 2015 at 06:22:10PM +0200, Jan Hubicka wrote:
On 04/08/2015 06:02 AM, Jakub Jelinek wrote:
(cp_build_qualified_type_real): Use check_base_type. Build a
variant and copy over even TYPE_CONTEXT
On the STAMP suite of benchmarks for transactional memory (described here
[1]).
I have ran an unmodified GCC 5.0.0 against the patched GCC with these
modifications and obtain the following speedups in STAMP with 4
threads (on a Haswell with 4 cores, average 10 runs):
I expect you'll need
On 2 April 2015 at 01:20, Jan Hubicka hubi...@ucw.cz wrote:
Your follow-up patch 88ada5e935d58223ae2d9ce6d0c1c71c372680a8 a.k.a r221269
added this to emit_local():
static bool
-emit_local (tree decl ATTRIBUTE_UNUSED,
+emit_local (tree decl,
const char *name ATTRIBUTE_UNUSED,
Hi,
Committed as r221926.
2015-04-08 Bernhard Reutner-Fischer al...@gcc.gnu.org
* doc/cfg.texi (GIMPLE statement iterators): Fix typo.
Index: gcc/doc/cfg.texi
===
--- gcc/doc/cfg.texi (revision 221925)
+++ gcc/doc/cfg.texi
gcc/ChangeLog:
2015-04-01 Bernhard Reutner-Fischer al...@gcc.gnu.org
* varasm.c (emit_bss): Remove redundant guard.
Signed-off-by: Bernhard Reutner-Fischer rep.dot@gmail.com
---
gcc/varasm.c | 2 --
1 file changed, 2 deletions(-)
diff --git a/gcc/varasm.c b/gcc/varasm.c
index
contrib/ChangeLog:
2015-04-08 Bernhard Reutner-Fischer al...@gcc.gnu.org
* mklog ($name, $addr): Fallback to env author settings.
---
contrib/mklog | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/contrib/mklog b/contrib/mklog
index f7974a7..4cad351 100755
---
On 08/04/15 20:11 +0100, Jonathan Wakely wrote:
We can get rid of the _Mutex type then, and just use std::mutex, and
that also means we can provide the timed locking functions even when
!defined(_GTHREAD_USE_MUTEX_TIMEDLOCK).
And so maybe we should use this fallback implementation instead of
On 07 Apr 22:43, Jan Hubicka wrote:
2015-04-07 Ilya Enkovich ilya.enkov...@intel.com
* tree-chkp.c (chkp_find_const_bounds_var): Remove.
(chkp_make_static_const_bounds): Search existing
symbol by assembler name. Use make_decl_one_only.
gcc/testsuite/
On Wed, Apr 8, 2015 at 3:36 PM, Bernhard Reutner-Fischer
rep.dot@gmail.com wrote:
contrib/ChangeLog:
2015-04-08 Bernhard Reutner-Fischer al...@gcc.gnu.org
* mklog ($name, $addr): Fallback to env author settings.
This looks fine, but note that I no longer have approval rights
On 8 April 2015 at 21:45, Diego Novillo dnovi...@google.com wrote:
On Wed, Apr 8, 2015 at 3:36 PM, Bernhard Reutner-Fischer
rep.dot@gmail.com wrote:
contrib/ChangeLog:
2015-04-08 Bernhard Reutner-Fischer al...@gcc.gnu.org
* mklog ($name, $addr): Fallback to env author
ping
On 2 April 2015 at 22:49, Bernhard Reutner-Fischer
rep.dot@gmail.com wrote:
On Wed, Apr 01, 2015 at 11:37:41PM +0200, Bernhard Reutner-Fischer wrote:
Bernd,
same for c6x for unk_isa, fwiw.
Attached.
Ok for trunk for the c6x bits?
Ok for trunk for the bfin bits?
thanks,
On 04/08/2015 03:33 PM, Segher Boessenkool wrote:
PR65693 exposes a case where combine does a worse job after my patches
to split parallels before combining. We start with a parallel of an
udiv and an umod, and a clobber; the umod is dead. The instruction
is combined with one setting the
PR65693 exposes a case where combine does a worse job after my patches
to split parallels before combining. We start with a parallel of an
udiv and an umod, and a clobber; the umod is dead. The instruction
is combined with one setting the divisor pseudo to a power-of-two
constant, so we end up
Hi!
This patch extends the PR65220 patch:
1) there is really no reason to limit the divisor to 32 or 64,
we can divide/modulo by pow2 constants up to 2G (0x7fff is
then still representable in 32-bit signed immediate)
2) on the testcase RTL cprop unfortunately isn't performed, because
Hi!
Right now, stmt.c on constraints not hardcoded in it, and not
define_{register,address,memory}_constraint just assumes the
constraint might allow both reg and mem. Unfortunately, on some
constraints which clearly can't allow either of those leads to errors
at -O0, because the expander
On Wed, Apr 08, 2015 at 05:16:08PM -0500, Segher Boessenkool wrote:
On Wed, Apr 08, 2015 at 05:12:07PM -0500, Segher Boessenkool wrote:
On Wed, Apr 08, 2015 at 11:00:59PM +0200, Jakub Jelinek wrote:
+case MATCH_CODE:
+ if (*XSTR (exp, 1) == '\0')
+ {
+ const char *code,
On Wed, 8 Apr 2015, Segher Boessenkool wrote:
2015-04-08 Segher Boessenkool seg...@kernel.crashing.org
* combine.c (is_parallel_of_n_reg_sets): Change first argument
from an rtx_insn * to an rtx.
(try_combine): Adjust both callers. Use it once more.
That once more is
On Wed, Apr 08, 2015 at 11:00:59PM +0200, Jakub Jelinek wrote:
+case MATCH_CODE:
+ if (*XSTR (exp, 1) == '\0')
+ {
+ const char *code, *codes = XSTR (exp, 0);
+ int ret = 0;
+ while ((code = scan_comma_elt (codes)) != 0)
+ if (strncmp (code, reg, 3) ==
On Wed, Apr 08, 2015 at 05:12:07PM -0500, Segher Boessenkool wrote:
On Wed, Apr 08, 2015 at 11:00:59PM +0200, Jakub Jelinek wrote:
+case MATCH_CODE:
+ if (*XSTR (exp, 1) == '\0')
+ {
+ const char *code, *codes = XSTR (exp, 0);
+ int ret = 0;
+ while ((code =
ping
On 1 April 2015 at 23:34, Bernhard Reutner-Fischer
rep.dot@gmail.com wrote:
Hi,
gcc/c-family/c-cppbuiltin.c
In file included from ./tm.h:21:0,
from
../../../../../../home/me/src/gcc-5.0.mine/gcc/c-family/c-cppbuiltin.c:23:
75 matches
Mail list logo