On Wed, Dec 11, 2013 at 3:37 PM, Ryan Mansfield wrote:
>>> 2013-12-10 Ryan Mansfield
>>>
>>> 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: Likewise.
>>> * gcc.targe
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 t
Ian Lance Taylor 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.
Cheers,
mwh
>
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" 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
> @@ -22,6 +22,14 @@ if
Hi!
On Thu, 5 Sep 2013 18:11:05 +0200, Jakub Jelinek 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 properly align i
On Thu, Dec 12, 2013 at 10:53:02AM +0100, Thomas Schwinge wrote:
> On Thu, 5 Sep 2013 18:11:05 +0200, Jakub Jelinek 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 glo
Hi!
On Thu, 12 Dec 2013 11:02:30 +0100, Jakub Jelinek 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 wrote:
> > > 3) I figured out we need to tell the runtime library not just
> > > address, size and kind, but also a
> 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 s/
> 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 viola
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?
>>
>>
> 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 = x86_6
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
>
>> 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
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
> -gna
> 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 pa
On 12/11/13 10:44, Iyer, Balaji V wrote:
Just out of curiosity, why can't I keep it as-is? It is giving the
correct output/behavior and doesn't seem to interfere with anything
else. The only extra thing I am doing is to add an extra if-statement
while recursing through all the functions to check
> 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
"Iyer, Balaji V" 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 += -D_Cilk_spawn=
> -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
>
>
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
PR midd
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 include
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"
> 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
> +++ b/gcc/ada/gcc-interface/Mak
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
>> --- a/gcc/ada/gcc-i
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
--- a/libgo/go/r
On Thu, Dec 12, 2013 at 10:59:01AM -0500, Vladimir Makarov wrote:
> 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 a
On Thu, Dec 12, 2013 at 12:21 AM, Michael Hudson-Doyle
wrote:
> Ian Lance Taylor 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 cas
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 i
> -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 Ru
On 12/11/2013, 1:59 PM, Yvan Roux wrote:
On 11 December 2013 19:25, Vladimir Makarov 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:
g++.dg/cilk-plus/AN/array_test_
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". B
> .. 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)
> warn_implicit_
On Wed, Dec 11, 2013 at 10:05 PM, Teresa Johnson wrote:
> On Fri, Dec 6, 2013 at 6:23 AM, Jan Hubicka 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 in an earlier e
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 do
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 testsui
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. Bo
> On Wed, Dec 11, 2013 at 1:21 AM, Martin Liška 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 -fno-reorder-blocks-and-partitions: 638
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.
Ia
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 tr
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
diff
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 failing.
Ian Lance Taylor writes:
> On Thu, Dec 12, 2013 at 12:21 AM, Michael Hudson-Doyle
> wrote:
>> Ian Lance Taylor 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
> On Wed, Dec 11, 2013 at 1:21 AM, Martin Liška 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 -fno-reorder-blocks-and-partitions: 638
On Thu, Dec 12, 2013 at 1:55 PM, bin.cheng wrote:
>
>
>> -Original Message-
>> From: Richard Biener [mailto:richard.guent...@gmail.com]
>> Sent: Wednesday, December 11, 2013 6:04 PM
>> To: Jakub Jelinek
>> Cc: Bin.Cheng; Jeff Law; Bin Cheng; gcc-patches List
>> Subject: Re: [PATCH PR41488]
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 x86_6
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
Date: Wed Dec 11 11:29:37 2013 -0500
* gdbinit.in (input_li
On 12/12/2013 03:08 PM, Jan Hubicka wrote:
So only reason why this is optimize_size only is the fact that we can't rely on
inliner
to fix up the wrappers?
I was just being conservative. In fact, the inliner seems to handle
small [cd]tors well, inlining them into the wrappers and then into
c
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: pat
On Dec 12, 2013, at 7:51 PM, Jason Merrill 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 trailing-array-suppo
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 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 stmt_may_cl
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 pr
On Fri, Dec 13, 2013 at 07:30:12AM +0100, Richard Biener wrote:
> Jakub Jelinek 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 walk_stmt_l
2013/12/11 Monk Chiang :
> 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: common/config/nds32/nds32-common.c
> ==
61 matches
Mail list logo
