On Wed, Dec 11, 2013 at 3:37 PM, Ryan Mansfield rmansfi...@qnx.com wrote:
2013-12-10 Ryan Mansfield rmansfi...@qnx.com
PR testsuite/59442
* gcc.target/i386/sse2-movapd-1.c: Fix alignment attributes.
* gcc.target/i386/sse2-movapd-2.c: Likewise.
* gcc.target/i386/avx-vmovapd-256-1.c:
On Wed, Dec 11, 2013 at 06:47:37PM +0100, Oleg Endo wrote:
On Thu, 2013-11-21 at 00:04 +0100, Steven Bosscher wrote:
Declaring the edge_iterator inside the for() is not a good argument
against FOR_EACH_EDGE. Of course, brownie points are up for grabs for
the brave soul daring enough to make
Ian Lance Taylor i...@google.com writes:
This patch to the Go frontend and libgo implements method values in the
reflect package. Working with method values and reflect now works
correctly, at least on x86.
Can you give me a test case? I can try it on a few other architectures
tomorrow.
Hi!
Copyprivate clause has following restriction (in 2.5, 3.0, 3.1 and 4.0):
All list items that appear in the copyprivate clause must be either
threadprivate or private in the enclosing context.
but we weren't diagnosing it and even in crayptr2.f90 testcase violated it.
Fixed thusly, regtested
Iyer, Balaji V balaji.v.i...@intel.com writes:
diff --git a/gcc/testsuite/g++.dg/cilk-plus/cilk-plus.exp
b/gcc/testsuite/g++.dg/cilk-plus/cilk-plus.exp
index 707d17e..36c8111 100644
--- a/gcc/testsuite/g++.dg/cilk-plus/cilk-plus.exp
+++ b/gcc/testsuite/g++.dg/cilk-plus/cilk-plus.exp
@@
Hi!
On Thu, 5 Sep 2013 18:11:05 +0200, Jakub Jelinek ja...@redhat.com wrote:
3) I figured out we need to tell the runtime library not just
address, size and kind, but also alignment (we won't need that for
the #pragma omp declare target global vars though), so that the
runtime library can
On Thu, Dec 12, 2013 at 10:53:02AM +0100, Thomas Schwinge wrote:
On Thu, 5 Sep 2013 18:11:05 +0200, Jakub Jelinek ja...@redhat.com wrote:
3) I figured out we need to tell the runtime library not just
address, size and kind, but also alignment (we won't need that for
the #pragma omp declare
Hi!
On Thu, 12 Dec 2013 11:02:30 +0100, Jakub Jelinek ja...@redhat.com wrote:
On Thu, Dec 12, 2013 at 10:53:02AM +0100, Thomas Schwinge wrote:
On Thu, 5 Sep 2013 18:11:05 +0200, Jakub Jelinek ja...@redhat.com wrote:
3) I figured out we need to tell the runtime library not just
address,
Anyway, let's keep the current tests as is, the patch is ok for trunk.
Commited in 205925.
I have some more fixes for Ada cross-builds that Eric commented on but need
a little more work - will try to re-test this evening and re-post tomorrow.
It's also PR ada/55946. Would mind trying the attached patch?
--
Eric Botcazou
Hi Eric,
your patch looks quite nice, (maybe
your patch looks quite nice, (maybe s/host_alias= @host_alias@/host_alias =
@host_alias@/)
Thanks for spotting it, now fixed.
but I have to make a few more patches, to get it working:
error: system.ads has restriction No_Implicit_Dynamic_Code
error: but the following files violate this
Hi Eric,
On 12 Dec 2013, at 12:11, Bernd Edlinger wrote:
I have some more fixes for Ada cross-builds that Eric commented on but need
a little more work - will try to re-test this evening and re-post tomorrow.
It's also PR ada/55946. Would mind trying the attached patch?
--
Eric
using your patch + the mod I made for LDFLAGS.
In gcc-interface/Makefile.in? I wasn't sure if it was really needed.
Out of curiosity, what do you set LDFLAGS to exactly?
I built x86_64-darwin12 X powerpc-darwin9 [build = x86_64-darwin12]
and then a native X powerpc-darwin9 [build =
Hi Eric,
On 12 Dec 2013, at 12:34, Eric Botcazou wrote:
using your patch + the mod I made for LDFLAGS.
In gcc-interface/Makefile.in? I wasn't sure if it was really needed.
Out of curiosity, what do you set LDFLAGS to exactly?
Darwin doesn't have gettext in libSystem, I build it as a
your patch looks quite nice, (maybe s/host_alias= @host_alias@/host_alias =
@host_alias@/)
Thanks for spotting it, now fixed.
but I have to make a few more patches, to get it working:
error: system.ads has restriction No_Implicit_Dynamic_Code
error: but the following files violate this
On Wed, Dec 11, 2013 at 11:51:55AM -0500, Jason Merrill wrote:
It's only safe to free the targs if they weren't used to instantiate
any templates, so I lean toward option #1. Did you test this with
strict gc?
Ok, after IRC discussion and another bootstrap/regtest I've installed
this variant
I wouldn't expect it to cause problems.
Jason
arm-linux-gnueabihf-gcc -c -I./
-I/home/ed/gnu/x/arm-linux-gnueabihf-linux64/lib/gcc/arm-linux-gnueabihf/4.
9.0/adalib/../adainclude
-I/home/ed/gnu/x/arm-linux-gnueabihf-linux64/lib/gcc/arm-linux-gnueabihf/4.
9.0/adalib/ -I. -I/home/ed/gnu/x/gcc-4.9-20131208/gcc/ada -g -O2 -W -Wall
-gnatpg
Darwin doesn't have gettext in libSystem, I build it as a convenience
library, but it still needs to refer to a system framework. For this to
link the gnattools I need:
LDFLAGS=-L/path/to/my/convenience/lib -framework CoreFoundation
OK, I'll add $(LDFLAGS). It was actually already passed
OK, so we want the attached patch? FWIW it passed
make -k check-c check-c++ RUNTESTFLAGS=compat.exp struct-layout-1.exp
on x86/Linux, x86-64/Linux, PowerPC/Linux [*], IA-64/Linux, SPARC/Solaris
and SPARC64/Solaris with ALT_CC_UNDER_TEST set to the unpatched compiler.
As well as on
Iyer, Balaji V balaji.v.i...@intel.com writes:
# Compiler and linker flags.
GENERAL_FLAGS = -I$(top_srcdir)/include -I$(top_srcdir)/runtime
-I$(top_srcdir)/runtime/config/$(config_dir) -DIN_CILK_RUNTIME=1
-GENERAL_FLAGS += -D_Cilk_spawn= -D_Cilk_sync= -D_Cilk_for=for
+# GENERAL_FLAGS +=
-Original Message-
From: Aldy Hernandez [mailto:al...@redhat.com]
Sent: Thursday, December 12, 2013 10:26 AM
To: Iyer, Balaji V
Cc: Jakub Jelinek; 'gcc-patches@gcc.gnu.org'
Subject: Re: [PING]: [GOMP4] [PATCH] SIMD-Enabled Functions (formerly
Elemental functions) for C
On
The following patch fixes
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59470
Committed as rev. 205930 for trunk and 205929 for gcc-4.8 branch.
Jakub is doing reg testing but I am sure the results will be not worse
as the patch is quite safe IMO.
2013-12-12 Vladimir Makarov
On 12/11/13 17:35, Kenneth Zadeck wrote:
This patch is for the trunk, but it solves a problem that comes up for
wide-int. For wide-int we
need to have the BITS_PER_UNIT available earlier.So this patch sets the
default value (8) in
genmodes.c so that it is available by anyone who
On 12 Dec 2013, at 15:23, Eric Botcazou wrote:
Darwin doesn't have gettext in libSystem, I build it as a convenience
library, but it still needs to refer to a system framework. For this to
link the gnattools I need:
LDFLAGS=-L/path/to/my/convenience/lib -framework CoreFoundation
OK,
With your blanket change to gnattools/Makefile, isn't it also reasonable to
apply the following?
diff --git a/gcc/ada/gcc-interface/Make-lang.in
b/gcc/ada/gcc-interface/Make-lang.in index cd3676f..241571d 100644
--- a/gcc/ada/gcc-interface/Make-lang.in
+++
On 12 Dec 2013, at 17:21, Eric Botcazou wrote:
With your blanket change to gnattools/Makefile, isn't it also reasonable to
apply the following?
diff --git a/gcc/ada/gcc-interface/Make-lang.in
b/gcc/ada/gcc-interface/Make-lang.in index cd3676f..241571d 100644
---
This patch to libgo fixes using reflect.MakeFunc with functions that
return a single floating point value on 32-bit x86. Bootstrapped and
ran Go testsuite on x86_64-unknown-linux-gnu. Committed to mainline and
4.8 branch.
Ian
diff -r 454895d0147d libgo/go/reflect/makefunc_386.S
---
On Thu, Dec 12, 2013 at 12:21 AM, Michael Hudson-Doyle
michael.hud...@linaro.org wrote:
Ian Lance Taylor i...@google.com writes:
This patch to the Go frontend and libgo implements method values in the
reflect package. Working with method values and reflect now works
correctly, at least on
The Go language spec was clarified to say that a struct with a blank
field of non-comparable type may not be compared. This patch implements
that restriction in the Go frontend, by removing the code that permitted
it. This change requires a couple of test cases to be updated; I've
simply copied
-Original Message-
From: gcc-patches-ow...@gcc.gnu.org [mailto:gcc-patches-
ow...@gcc.gnu.org] On Behalf Of Aldy Hernandez
Sent: Thursday, December 12, 2013 10:47 AM
To: Iyer, Balaji V
Cc: gcc-patches@gcc.gnu.org; Jeff Law
Subject: Re: [PATCH] Enable Cilk keywords in Cilk Runtime
On 12/11/2013, 1:59 PM, Yvan Roux wrote:
On 11 December 2013 19:25, Vladimir Makarov vmaka...@redhat.com wrote:
On 12/11/2013, 5:35 AM, Yvan Roux wrote:
Hi Vladimir,
I've some regressions on ARM after this SP elimination patch, and they
are execution failures. Here is the list:
In Go the top-level name init is special, as it names a function that
is run when the program starts. The language therefore does not permit
other uses of init at top level. This change to the Go frontend
detects and bans one of those cases: importing a package under the local
name init.
.. then does the second block need hoisting to bracket the two cases with
host!=build?
This code works fine so I don't think that we really need to do anything.
--
Eric Botcazou
Hi,
diff --git a/gcc/c-family/c-opts.c b/gcc/c-family/c-opts.c
index f368cab..3576f7d 100644
--- a/gcc/c-family/c-opts.c
+++ b/gcc/c-family/c-opts.c
@@ -899,6 +899,10 @@ c_common_post_options (const char **pfilename)
if (warn_implicit_function_declaration == -1)
On Wed, Dec 11, 2013 at 10:05 PM, Teresa Johnson tejohn...@google.com wrote:
On Fri, Dec 6, 2013 at 6:23 AM, Jan Hubicka hubi...@ucw.cz wrote:
Hi, all
This is the new patch for gcov-tool (previously profile-tool).
Honza: can you comment on the new merge interface? David posted some
comments
On 12/12/13 07:56, Iyer, Balaji V wrote:
Will it be Ok if I don’t mark them as cilk simd function but just
keep it as omp declare simd from the start? That should get around
this issue.
No, because then we won't be able to distinguish between OMP and Cilk
Plus clones. This is something we
This patch to libgo fixes the handling of the deferring of the unlock
thread at program startup. The code was incorrectly freeing a stack
object. This patch also cleans up some cases of freeing a defer block
to avoid doing it when there is no Go context available. Bootstrapped
and ran Go
This patch to the Go frontend tweaks the existing error messages about
use of untyped nil to do a better job when nil appears as the first
argument to the builtin append function. This avoids an error about nil
not being a slice type--it's more useful to say that the problem is an
untyped nil.
On Wed, Dec 11, 2013 at 1:21 AM, Martin Liška marxin.li...@gmail.com wrote:
Hello,
I prepared a collection of systemtap graphs for GIMP.
1) just my profile-based function reordering: 550 pages
2) just -freorder-blocks-and-partitions: 646 pages
3) just
When Go code takes a slice of a pointer to an array, the compiler was
failing to check whether that point is nil, as is required by the Go 1.2
addition of reliable nil checks. This patch fixes that oversight.
Bootstrapped and ran Go testsuite on x86_64-unknown-linux-gnu.
Committed to mainline.
I stumbled over this while trying to fix one of the regressions for 4.9.
This peep2 pattern in the x86 backend is obviously broken as it does not
have a mode on the zero_extend in the resulting insn. As a result,
if/when this peep2 matches we get an unrecognized insn.
(define_peephole2
Hi!
Before the PR58956 fix find_replaceable_in_bb only gave up TER if
stmt was gimple_assign_single_p that was storing something that
could alias with TERed load, but now it also punts on calls (a lot of them
apparently) and inline asm (all of them, because stmt_may_clobber_ref_p
always returns
The Go language requires that the { starting a block for an if, for, or
switch statement be on the same line as the if/for/switch. This patch
gives better error messages when a program does it wrong. Bootstrapped
and ran Go testsuite on x86_64-unknown-linux-gnu. Committed to
mainline.
Ian
flags_from_decl_or_type() only handles a TYPE or DECL. Make this
explicit instead.
I also added a check in the use in trans-mem.c, just in case. The
subsequent conditionals should take care of the TM case.
It would be nice if Marc Glisse could provide the testcase he mentioned
was
Ian Lance Taylor i...@google.com writes:
On Thu, Dec 12, 2013 at 12:21 AM, Michael Hudson-Doyle
michael.hud...@linaro.org wrote:
Ian Lance Taylor i...@google.com writes:
This patch to the Go frontend and libgo implements method values in the
reflect package. Working with method values and
Recently I've improved fn_type_unification's handling of access checks,
but resolve_overloaded_unification didn't get the same attention.
Fortunately, it's a simple matter of switching it over to using
instantiate_template so that we have a function to check the accesses
against.
Tested
I often use input_line in breakpoint conditions when debugging the
compiler, and losing the macro complicates that. Any objection to
adding it to gdbinit.in?
commit 5bee3eed904bb31ffcca5330f2c36e5aa1c35cb0
Author: Jason Merrill ja...@redhat.com
Date: Wed Dec 11 11:29:37 2013 -0500
*
This patch updates the Go testsuite to a copy of the current master
testsuite. This brings the testsuite, and thus the compiler, up to the
final Go 1.2 release. Tested by, of course, running the testsuite, on
x86_64-unknown-linux-gnu. Committed to mainline.
Ian
foo.patch.bz2
Description:
On Dec 12, 2013, at 7:51 PM, Jason Merrill ja...@redhat.com wrote:
I often use input_line in breakpoint conditions when debugging the compiler,
and losing the macro complicates that. Any objection to adding it to
gdbinit.in?
That'd be wonderful.
On 12/11/13 12:19, Eric Botcazou wrote:
Yes we do, even for struct { struct { int a; char a[1] } }; (note the not
really trailing as there is padding after the trailing array). We do
take size limitations from a DECL (if we see one) into account to limit the
effect of this
On 12/12/13 16:09, Aldy Hernandez wrote:
flags_from_decl_or_type() only handles a TYPE or DECL. Make this
explicit instead.
I also added a check in the use in trans-mem.c, just in case. The
subsequent conditionals should take care of the TM case.
It would be nice if Marc Glisse could provide
Jakub Jelinek ja...@redhat.com wrote:
Hi!
Before the PR58956 fix find_replaceable_in_bb only gave up TER if
stmt was gimple_assign_single_p that was storing something that
could alias with TERed load, but now it also punts on calls (a lot of
them
apparently) and inline asm (all of them, because
On Thu, 12 Dec 2013, Aldy Hernandez wrote:
flags_from_decl_or_type() only handles a TYPE or DECL. Make this explicit
instead.
I also added a check in the use in trans-mem.c, just in case. The subsequent
conditionals should take care of the TM case.
It would be nice if Marc Glisse could
On Fri, Dec 13, 2013 at 07:30:12AM +0100, Richard Biener wrote:
Jakub Jelinek ja...@redhat.com wrote:
lhs of a call for calls or somewhere in output arguments of inline asm.
Can you please simply use walk_stmt_load_store_ops to get at the stmt outputs?
No, unfortunately. The problem is that
2013/12/11 Monk Chiang sh.chian...@gmail.com:
Hi,
Recently I used --target=nds32be-elf to configure nds32 gcc,
it seems that the big endian is not set as default.
[...]
The following is the patch to fix this issue. Tested on nds32be-elf.
OK to apply?
Index:
56 matches
Mail list logo