On 07/16/2015 04:05 PM, Martin Liška wrote:
On 07/10/2015 03:31 PM, Martin Jambor wrote:
Hi,
thanks for working on this and sorry for a tad late review:
On Thu, Jul 09, 2015 at 11:13:52AM +0200, Martin Liska wrote:
gcc/ChangeLog:
2015-07-03 Martin Liska mli...@suse.cz
* cgraph.c
Le 03/08/2015 14:36, Paul Richard Thomas a écrit :
Dear Mikael,
Thanks for your green light!
I have been mulling over the trans-decl part of the patch and having
been wondering if it is necessary.
You mean marking entities as public? Or setting the hidden visibility
attribute? Or both?
I
On 03/08/15 16:45, Uros Bizjak wrote:
On Mon, Aug 3, 2015 at 5:37 PM, Kyrill Tkachov kyrylo.tkac...@arm.com wrote:
Looking at the x86 movcc expansion code (ix86_expand_int_movcc) I
don't think this is a good idea. In the expander, there is already
quite some target-dependent code that goes
On August 3, 2015 5:03:05 PM GMT+02:00, Segher Boessenkool
seg...@kernel.crashing.org wrote:
The two files *-match.o files always finish building last, so if we
start building them as soon as possible (instead of pretty late) the
total build time will be less on a parallel build.
Bootstrapped
On 08/03/2015 08:48 AM, Martin Sebor wrote:
On 08/03/2015 05:55 AM, Jan-Benedict Glaw wrote:
On Sun, 2015-08-02 17:15:27 -0600, Martin Sebor mse...@gmail.com wrote:
OK for the trunk. Sorry for the delay.
Thank you. Committed in revision 226480.
...und breaks native builds. When doing
On 03/08/15 15:15, Uros Bizjak wrote:
On Mon, Aug 3, 2015 at 3:43 PM, Kyrill Tkachov kyrylo.tkac...@arm.com wrote:
On 03/08/15 14:37, Uros Bizjak wrote:
On Mon, Aug 3, 2015 at 3:02 PM, Kyrill Tkachov kyrylo.tkac...@arm.com
wrote:
On 03/08/15 13:33, Uros Bizjak wrote:
Hello!
2015-07-30
On 08/03/2015 05:42 AM, Kai Tietz wrote:
2015-08-03 5:49 GMT+02:00 Jason Merrill ja...@redhat.com:
On 07/31/2015 05:54 PM, Kai Tietz wrote:
The STRIP_NOPS-requirement in 'reduced_constant_expression_p' I could
remove, but for one case in constexpr. Without folding we don't do
On Mon, Aug 3, 2015 at 5:37 PM, Kyrill Tkachov kyrylo.tkac...@arm.com wrote:
Looking at the x86 movcc expansion code (ix86_expand_int_movcc) I
don't think this is a good idea. In the expander, there is already
quite some target-dependent code that goes great length to utilize sbb
insn as much
2015-08-03 17:04 GMT+03:00 Uros Bizjak ubiz...@gmail.com:
On Mon, Aug 3, 2015 at 3:02 PM, Kyrill Tkachov kyrylo.tkac...@arm.com wrote:
On 03/08/15 13:33, Uros Bizjak wrote:
Hello!
2015-07-30 Kyrylo Tkachov kyrylo.tkac...@arm.com
* ifcvt.c (noce_try_store_flag_constants): Make logic
We can't use _N as an identifier, as per
https://gcc.gnu.org/onlinedocs/libstdc++/manual/source_code_style.html
Tested ppc64le-linux, committed to trunk.
commit ae65b96cfb69c64d636f86d9f9154da2364e0440
Author: Jonathan Wakely jwak...@redhat.com
Date: Mon Aug 3 16:29:44 2015 +0100
PR
On 08/03/2015 05:55 AM, Jan-Benedict Glaw wrote:
On Sun, 2015-08-02 17:15:27 -0600, Martin Sebor mse...@gmail.com wrote:
OK for the trunk. Sorry for the delay.
Thank you. Committed in revision 226480.
...und breaks native builds. When doing builds using config-list.mk, I
first build a GCC
I've committed this patch to gomp4. The existing implementation of firstprivate
presumes the existence of memory at the CTA level. This patch does away with
that, treating firstprivate as thread-private variables initialized from the
host.
During development there was some fallout from
I've committed this to gomp4. I was reusing a size_t variable in a loop where
unsigned would do, and fprintf got upset.
nathan
2015-08-03 Nathan Sidwell nat...@codesourcery.com
* config/nvptx/mkoffload.c (process): Avoid printf formatting
warning.
Index: config/nvptx/mkoffload.c
Hi,
Ping for the patch submitted here
https://gcc.gnu.org/ml/gcc-patches/2015-07/msg02251.html
Thanks,
Pawel Kupidura
On 07/20/2015 08:19 AM, James Greenhalgh wrote:
I think we either want to defer the unordered_remove until we're done
processing all the vector elements, or make sure to look at element 'i'
again after we've moved something new in to it.
Correct. Two loops had this mistake -- while others got
On Mon, Jul 27, 2015 at 02:22:41PM +0100, Pawel Kupidura wrote:
diff --git a/gcc/ChangeLog b/gcc/ChangeLog
index 10df325..ffafc3f 100644
--- a/gcc/ChangeLog
+++ b/gcc/ChangeLog
@@ -1,3 +1,7 @@
+2015-07-27 Pawel Kupidura pawel.kupid...@arm.com
Two spaces between your name and your email
On 08/03/2015 09:50 AM, Ilya Enkovich wrote:
The original code looks better, tree height is just 2 and therefore it
can be executed in 2 cycles. New code has more dependencies and tree
height becomes 5. It is always hard to say for all x86 targets but as
a generic code the original version is
I've committed this to gomp4 branch. It tidies up the error message processing
and removes some code that shows a remarkable confusion about how shared
libraries work.
nathan
--
Nathan Sidwell - Director, Sourcery Services - Mentor Embedded
2015-08-03 Nathan Sidwell nat...@codesourcery.com
I suspect the back end or even the middle end route isn't going to
work even if there was enough context to diagnose the problem
expressions because some of them will have been optimized away by then
(e.g., 'if ( __builtin_foo != 0)' is optimized into if (1) by gimple).
I was thinking that if
In preparing the next IEEE 128-bit floating point patch, I needed a quick way
to load -0.0q into a vector registers (i.e. just the MSB set). I originally had
a special purpose insn to load this value, but I decided to widen it to allow
the easy_altivec_constant support to generate constants where
On Mon, Aug 3, 2015 at 11:14 PM, FX fxcoud...@gmail.com wrote:
The attached patch extends the IEEE modules to all floating-point kinds. Last
time, when I added IEEE support, I restricted it to the float and double
types (real kinds 4 and 8), to be extra safe. After discussion with Uros
On Mon, Aug 3, 2015 at 11:14 PM, FX fxcoud...@gmail.com wrote:
The attached patch extends the IEEE modules to all floating-point kinds. Last
time, when I added IEEE support, I restricted it to the float and double
types (real kinds 4 and 8), to be extra safe. After discussion with Uros
On Mon, Aug 03, 2015 at 11:14:41PM +0200, FX wrote:
So, this enables full IEEE support for all real kinds. Nothing
changes to the underlying architecture, it???s almost exclusively
mechanical changes (adding the necessary variants to the interfaces, etc.).
Bootstrapped and regtested on
On Jul 30, 2015, H.J. Lu hjl.to...@gmail.com wrote:
aoliva/pr64164 is fine on x32.
Thanks. I have made a large number of changes since you tested it,
fixing all the reported issues and then some. Now, x86_64-linux-gnu
(-m64 and -m32), i686-pc-linux-gnu, powerpc64-linux-gnu and
On 08/03/15 19:52, Nathan Sidwell wrote:
I've committed this to gomp4 branch. It tidies up the error message processing
and removes some code that shows a remarkable confusion about how shared
libraries work.
I've also committed it to trunk as obvious.
nathan
Now I get
../../../gcc/libgo/runtime/mprof.goc: In function ‘runtime_Stack’:
../../../gcc/libgo/runtime/mprof.goc:437:19: error: ‘enablegc’ may be
used uninitialized in this function [-Werror=maybe-uninitialized]
mstats.enablegc = enablegc;
^
On Mon, 2015-08-03 at 16:27 -0400, David Edelsohn wrote:
On Mon, Aug 3, 2015 at 3:39 PM, Peter Bergner berg...@vnet.ibm.com wrote:
This has passed bootstrapping and regtesting on trunk. Ok for mainline?
I'd like to also backport this to the release branches. Is this ok for
them once
On Mon, Aug 3, 2015 at 7:24 PM, Michael Hudson-Doyle
michael.hud...@canonical.com wrote:
Now I get
../../../gcc/libgo/runtime/mprof.goc: In function ‘runtime_Stack’:
../../../gcc/libgo/runtime/mprof.goc:437:19: error: ‘enablegc’ may be
used uninitialized in this function
I was starting to look at Abe's changes to the gimple if-converter and
realized a handful of the changes were just fixing comments. No reason
those shouldn't go in immediately.
So I pulled them out and applied those changes to the trunk.
Abe -- if you find more of those kind of changes,
On 07/17/2015 01:57 PM, Abe wrote:
Dear all,
Relative to the previous submission of this same patch, the below
corrects some minor spacing and/or indentation issues,
misc. other formatting fixes, and makes the disabled vectorization tests
be disabled via xfail rather than by adding spaces to
On Mon, 3 Aug 2015, Michael Meissner wrote:
The intention of theese changes (currently unwritten) is to change the
existing
problematical names that use TF in their name to be something else, and
provide
via a weak reference an alias for the old name. So if for example, we change
the
The gc compiler determines the type of the Data field in the
syscall.RawSockaddr type by looking at the system header files. The
gccgo library just has a fixed definition for it. Unfortunately the
fixed definition does not match the header files on PPC64 systems.
This is
On Sun, Aug 2, 2015 at 9:40 PM, Patrick Palka patr...@parcs.ath.cx wrote:
On Sun, Aug 2, 2015 at 2:44 PM, Bernhard Reutner-Fischer
rep.dot@gmail.com wrote:
On August 2, 2015 8:18:51 PM GMT+02:00, Patrick Palka patr...@parcs.ath.cx
wrote:
No comment on the patch itself, but please
On 24/07/15 09:36, Kyrill Tkachov wrote:
On 21/07/15 17:52, James Greenhalgh wrote:
On Thu, Jul 16, 2015 at 04:21:05PM +0100, Kyrill Tkachov wrote:
Hi all,
This patch implements target pragmas for aarch64.
The pragmas accepted are the same as for target attributes (as required).
In addition
On Fri, Jul 24, 2015 at 09:21:46AM +0100, Kyrill Tkachov wrote:
Thanks for the review, here's an updated version.
In the end, I chose to retain the use alloca (other patches in the series
are reworked to use it too).
How's this?
A nit or two from code you were moving or that got caught up
Since we allow arbitrary IDs in captures the capture array is dense
and thus we don't need to clear it before initializing. We can
also emit more compact source by simply using an initializer.
- tree captures[3] ATTRIBUTE_UNUSED = {};
- captures[0] = o20;
-
Could you probably review the patch, please?
2015-07-28 18:42 GMT+03:00 Maxim Blumental bvm...@gmail.com:
Applied the idea with python script alternative. Review, please.
2015-07-24 17:18 GMT+03:00 David Malcolm dmalc...@redhat.com:
On Fri, 2015-07-24 at 10:01 +0200, Jakub Jelinek wrote:
Hi!
On Tue, 28 Jul 2015 10:20:39 +0200, Tom de Vries tom_devr...@mentor.com wrote:
this patch adds a test-case with a double reduction in an oacc kernels
region.
In order to get it in the proper shape for parloops to deal with, I
needed to repeat the pass_lim/pass_copy_prop sequence.
The following patch fixes PR66917 - the vectorizer assumes element
alignment even for unaligned accesses, which in case of packed
ones is not a valid assumption. This breaks testcases on arm
at least which can do unaligned (but not completely unaligned)
loads and stores.
Bootstrapped and tested
On Fri, Jul 31, 2015 at 1:17 PM, Yuri Rumyantsev ysrum...@gmail.com wrote:
Hi Richard,
I learned your updated patch for 23825 and it is more general in
comparison with my.
I'd like to propose you a compromise - let's consider my patch only
for force-vectorize outer loop only to allow
Hi Michael,
It looks like this patch introduces regressions on armeb in:
gcc.dg/vect/vect-reduc-7.c
gcc.dg/vect/vect-reduc-8.c
gcc.dg/vect/vect-reduc-9.c
See
http://people.linaro.org/~christophe.lyon/cross-validation/gcc/trunk/226476/report-build-info.html
for a bit more details.
Can you have
On 07/20/2015 02:45 PM, Pierre-Marie de Rodat wrote:
On PowerPC targets with -mlongcall, most subprogram calls are turned
into indirect calls: the call target is read from a register even though
it is compile-time known. This makes it difficult for machine code
static analysis engines to recover
2015-08-03 5:49 GMT+02:00 Jason Merrill ja...@redhat.com:
On 07/31/2015 05:54 PM, Kai Tietz wrote:
The STRIP_NOPS-requirement in 'reduced_constant_expression_p' I could
remove, but for one case in constexpr. Without folding we don't do
type-sinking/raising.
Right.
So binary/unary
On Fri, Jul 24, 2015 at 11:43:32AM +0100, Kyrill Tkachov wrote:
On 21/07/15 16:37, James Greenhalgh wrote:
On Thu, Jul 16, 2015 at 04:20:59PM +0100, Kyrill Tkachov wrote:
Hi all,
This patch implements target attribute support via the
TARGET_OPTION_VALID_ATTRIBUTE_P hook.
The
On 28/07/15 11:21, Szabolcs Nagy wrote:
On 24/07/15 12:27, Kyrill Tkachov wrote:
On 24/07/15 12:10, Szabolcs Nagy wrote:
(-a)*b should not be compiled to vnmul a,b with -frounding-math.
Added a new -(a*b) pattern for vnmul and the old one is only
used if !flag_rounding_math. Updated the
Le 29/07/2015 17:08, Paul Richard Thomas a écrit :
Dear All,
On 24 July 2015 at 10:08, Damian Rouson dam...@sourceryinstitute.org wrote:
I love this idea and had similar thoughts as well.
:D
Sent from my iPhone
On Jul 24, 2015, at 1:06 AM, Paul Richard Thomas
paul.richard.tho...@gmail.com
On Thu, Jul 23, 2015 at 11:17:20AM +0100, Kyrill Tkachov wrote:
Thanks, I've implemented the suggestions.
Re-bootstrapped and tested on aarch64.
How's this?
This is good. OK for trunk.
Thanks,
James
Hi!
On Wed, 29 Jul 2015 19:13:01 -0400, Nathan Sidwell nat...@acm.org wrote:
gcc/
* gimplify.c (oacc_default_clause): Outer scope searching moved to
omp_notice_variable.
(omp_notice_variable): For OpenACC search enclosing scopes before
applying default.
On 07/20/2015 09:39 AM, Pierre-Marie de Rodat wrote:
This patch fixes the static link description in DWARF to comply with the
specification. In order to do so, it appends a field to all FRAME
objects to hold the frame base address (DW_AT_frame_base) so that the
nested subrograms can directly
On Jul 26, 2015, at 11:50 AM, Andi Kleen a...@firstfloor.org wrote:
I've been compiling gcc with tcmalloc to do a similar speedup. It would be
interesting to compare that to your patch.
I repeated the test with TCMalloc and jemalloc. TCMalloc shows nice results,
though it required some tweaks:
On Fri, Jul 31, 2015 at 8:27 PM, Jeff Law l...@redhat.com wrote:
On 07/31/2015 12:18 PM, Michael Collison wrote:
Hi Jeff,
Yes I will create a test case. I'm not quite sure what to check for even
in the machine dependent test case. It's quite possible for the
instructions that are generated
Ian Lance Taylor i...@golang.org writes:
This patch from Chris Manghane fixes the Go frontend error reporting
for invalid builtin calls, by not losing track of whether the call is
erroneous. This fixes https://golang.org/issue/11561. Bootstrapped
and ran Go testsuite on
On Thu, Jul 30, 2015 at 8:41 PM, H.J. Lu hongjiu...@intel.com wrote:
On Tue, Jul 21, 2015 at 02:45:39PM -0700, H.J. Lu wrote:
When __builtin_frame_address is used to retrieve the address of the
function stack frame, the frame pointer is always kept, which wastes one
register and 2
On 07/25/2015 09:44 PM, Pierre-Marie de Rodat wrote:
This change makes GCC materialize subprogram renamings in Ada as
imported declarations (DW_TAG_imported_declarations). For instance,
procedure Foo renames Bar;
will output:
DW_TAG_imported_declaration:
DW_AT_name: foo
On Mon, Aug 3, 2015 at 7:04 AM, Uros Bizjak ubiz...@gmail.com wrote:
On Mon, Aug 3, 2015 at 3:02 PM, Kyrill Tkachov kyrylo.tkac...@arm.com wrote:
On 03/08/15 13:33, Uros Bizjak wrote:
Hello!
2015-07-30 Kyrylo Tkachov kyrylo.tkac...@arm.com
* ifcvt.c (noce_try_store_flag_constants):
On Fri, Oct 17, 2014 at 7:48 AM, Richard Sandiford
richard.sandif...@arm.com wrote:
This is the main patch, to add new preferred_for_size and
preferred_for_speed attributes that can be used to selectively disable
alternatives when optimising for size or speed. As explained in the
docs, the
The two files *-match.o files always finish building last, so if we
start building them as soon as possible (instead of pretty late) the
total build time will be less on a parallel build.
Bootstrapped and tested on powerpc64-linux. Is this okay for trunk?
Segher
2014-080-3 Segher
Hello.
Following patch has been installed to HSA branch.
Thanks,
Martin
From 64f7f75a8b16dee2071fef7f546fbf6b06f82c4f Mon Sep 17 00:00:00 2001
From: mliska mli...@suse.cz
Date: Fri, 31 Jul 2015 17:53:57 +0200
Subject: [PATCH 1/3] HSA: introduce more pure kernel information emission.
Hello.
Following patch has been installed to HSA branch.
Thanks,
Martin
From 125ee1ed4123527d4ce841631d4930ac96bec281 Mon Sep 17 00:00:00 2001
From: mliska mli...@suse.cz
Date: Mon, 3 Aug 2015 13:00:42 +0200
Subject: [PATCH 2/3] HSA: fix kernel initialization for dependent kernels
Hello.
Following patch has been installed to HSA branch.
Thanks,
Martin
From 163c6a4bb917ae76dad265dca5762a3bd87c947a Mon Sep 17 00:00:00 2001
From: mliska mli...@suse.cz
Date: Mon, 3 Aug 2015 14:35:26 +0200
Subject: [PATCH 3/3] HSA: calculate properly maximum OMP data size which is
necessary.
On Sun, 2015-08-02 at 12:18 +0200, Andreas Schwab wrote:
Bill Schmidt wschm...@linux.vnet.ibm.com writes:
* gcc.target/powerpc/vec-cmp-sel.c: New test.
FAIL: gcc.target/powerpc/vec-cmp-sel.c (test for excess errors)
Excess errors:
PING.
Thanks,
Martin
On 07/20/2015 09:51 AM, Martin Liška wrote:
On 07/16/2015 01:03 PM, Martin Liška wrote:
On 07/09/2015 06:24 PM, Jeff Law wrote:
On 07/09/2015 07:56 AM, mliska wrote:
gcc/ChangeLog:
2015-07-09 Martin Liska mli...@suse.cz
* dbgcnt.def: Add new debug counter.
On 03/08/15 11:52, James Greenhalgh wrote:
On Fri, Jul 24, 2015 at 11:43:32AM +0100, Kyrill Tkachov wrote:
On 21/07/15 16:37, James Greenhalgh wrote:
On Thu, Jul 16, 2015 at 04:20:59PM +0100, Kyrill Tkachov wrote:
Hi all,
This patch implements target attribute support via the
On Thu, Jul 30, 2015 at 04:08:47PM +0200, Mikael Morin wrote:
Le 29/07/2015 18:45, Steve Kargl a écrit :
This builds and passes regression testing on x86_64-*-freebsd.
OP found the problem by running the sanatizers. I don't
know how to build gcc with this as a build option. I'll
The 64-bit register indirect call does not use register %r1. So, %r1 shouldn't
be clobbered in
the call pattern. Normally, this doesn't matter as register %r1 is call
clobbered. However,
reload can fail when the register used for the indirect call needs to be
reloaded from a static
location.
This patch from Andrew Wilkins kills off extra sleep processes when
running the libgo testsuite. Otherwise, the sleep process would
continue after the test completes. This doesn't affect the testsuite
but the extra sleep processes are not desirable. Committed to
mainline.
Ian
Index:
On 07/30/2015 04:19 PM, Evgeny Stupachenko wrote:
Hi All,
The patch enables new attribute 'ctarget',
The attribute force compiler to create clones of a function with the attribute.
For example:
__attribute__((ctarget(avx,arch=slm,arch=core-avx2,default)))
So presumably we're allowing both
On Fri, Jul 31, 2015 at 12:43:20AM +, Joseph Myers wrote:
On Wed, 29 Jul 2015, Michael Meissner wrote:
#6Add support for using different names for the 64/128-bit integer
conversion to IBM extended double, to allow a future version to
switch the default for what long
PING: https://gcc.gnu.org/ml/gcc-patches/2015-07/msg02581.html
Thanks,
Manuel.
On 30 July 2015 at 17:35, Manuel López-Ibáñez lopeziba...@gmail.com wrote:
When I fixed PR59304, I forgot that a command-line warning can be also
an error if -Werror was enabled. This introduced a regression since
On 07/28/2015 06:00 AM, Kugan wrote:
On 27/07/15 05:38, Andreas Schwab wrote:
Kugan kugan.vivekanandara...@linaro.org writes:
* cse.c (cse_insn): Fix missing check for STRICT_LOW_PART and minor
clean up.
This breaks
gcc.target/m68k/tls-ie-xgot.c scan-assembler jsr
PING: https://gcc.gnu.org/ml/gcc-patches/2015-07/msg01860.html
Actually, the xfailed test was because the function folded to nothing
and the offending code was removed without warning. Fixed in the
attached version. Same changelog.
On 22 July 2015 at 17:52, Manuel López-Ibáñez
This patch by Chris Manghane changes the Go frontend to not make type
descriptor for named types if we have seen errors during the
compilation. This avoids a compiler crash, and fixes
https://golang.org/issue/11560 . Bootstrapped and ran Go testsuite on
x86_64-unknown-linux-gnu. Committed to
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67077
is fixed by
https://gcc.gnu.org/viewcvs/gcc?view=revisionrevision=226089
I checked in this patch to add a testcase for PR tree-optimization/67077.
Index: ChangeLog
===
---
On 08/02/2015 05:03 AM, Kumar, Venkataramanan wrote:
Hi Jakub,
Thank you for reviewing the patch.
I have incorporated your comments in the attached patch.
Note Jakub is on PTO for the next 3 weeks.
vectorize_mults_via_shift.diff.txt
diff --git
On 03/08/15 16:53, Kyrill Tkachov wrote:
On 03/08/15 16:45, Uros Bizjak wrote:
On Mon, Aug 3, 2015 at 5:37 PM, Kyrill Tkachov kyrylo.tkac...@arm.com wrote:
Looking at the x86 movcc expansion code (ix86_expand_int_movcc) I
don't think this is a good idea. In the expander, there is already
On Mon, Aug 3, 2015 at 7:20 PM, Kyrill Tkachov kyrylo.tkac...@arm.com wrote:
Looking at the x86 movcc expansion code (ix86_expand_int_movcc) I
don't think this is a good idea. In the expander, there is already
quite some target-dependent code that goes great length to utilize sbb
insn as much
On 07/16/2015 05:03 AM, Martin Liška wrote:
So a general question. We're passing in STRICT to several routines, which is
fine. But then we're also checking M_TAIL_MERGE_MODE. What's the difference
between the two? Can they be unified?
Hello.
I would say that STRICT is a bit generic
On 07/26/2015 07:05 PM, Kugan wrote:
I thought that when !gimple_assign_cast_p (stmt), RHS will always
boolean. I have now added this check in the attached patch.
Thanks.
I also noticed that in maybe_optimize_range_tests, GIMPLE_COND can
have non compatible types when new_op is updated
Bill Schmidt wschm...@linux.vnet.ibm.com writes:
Index: gcc/testsuite/gcc.target/powerpc/vec-cmp-sel.c
===
--- gcc/testsuite/gcc.target/powerpc/vec-cmp-sel.c(revision 226505)
+++ gcc/testsuite/gcc.target/powerpc/vec-cmp-sel.c
On 07/20/2015 01:52 AM, Martin Liška wrote:
Due to changes dump file, it will be necessary to amend scanning of dump files.
Those changes are fine and can be installed once the prerequisites have
been approved/installed.
Thanks,
Jeff
On 07/31/2015 03:45 PM, David Malcolm wrote:
In r223092 (aka dd4d567f4b6b498242097c41d63666bdae320ac1) I moved the
state of timevar.c from being global data into a class timer with
a global instance g_timer.
This followup patch generalizes the timing within toplev so that an
external timer
On Mon, Aug 3, 2015 at 2:10 AM, Andreas Schwab sch...@suse.de wrote:
../../../libgo/runtime/mprof.goc: In function 'runtime_Stack':
../../../libgo/runtime/mprof.goc:408:5: error: calling
'__builtin_frame_address' with a nonzero argument is unsafe
[-Werror=frame-address]
sp =
On Mon, Aug 3, 2015 at 11:10 AM, Andreas Schwab sch...@suse.de wrote:
Ian Lance Taylor i...@golang.org writes:
This patch from Chris Manghane fixes the Go frontend error reporting
for invalid builtin calls, by not losing track of whether the call is
erroneous. This fixes
Hi!
On Mon, 3 Aug 2015 08:00:36 -0400, Nathan Sidwell nat...@codesourcery.com
wrote:
On 08/03/15 07:37, Thomas Schwinge wrote:
On Sun, 2 Aug 2015 21:23:30 -0400, Nathan Sidwell nat...@acm.org wrote:
I've committed this to gomp4 branch. It expands the acc_on_device builtin
earlier in the
On 03/08/15 13:33, Uros Bizjak wrote:
Hello!
2015-07-30 Kyrylo Tkachov kyrylo.tkac...@arm.com
* ifcvt.c (noce_try_store_flag_constants): Make logic of the case
when diff == STORE_FLAG_VALUE or diff == -STORE_FLAG_VALUE more
explicit. Prefer to add the flag whenever
On Mon, Aug 03, 2015 at 10:36:17AM +0100, Kyrill Tkachov wrote:
And here is a rebased version to resolve a conflict after Alan's patches went
in.
OK with the nits below fixed.
2015-08-03 Kyrylo Tkachov kyrylo.tkac...@arm.com
* config.gcc (aarch64*-*-*): Specify c_target_objs and
Hi!
On Sat, 1 Aug 2015 20:20:49 -0400, Nathan Sidwell nat...@acm.org wrote:
this is the gomp4 committed version.
--- liboffloadmic/plugin/libgomp-plugin-intelmic.cpp (revision 226462)
+++ liboffloadmic/plugin/libgomp-plugin-intelmic.cpp (working copy)
@@ -327,12 +327,26 @@ offload_image
On Fri, Jul 24, 2015 at 09:38:34AM +0100, Kyrill Tkachov wrote:
Thanks, here's an updated version.
2015-07-24 Kyrylo Tkachov kyrylo.tkac...@arm.com
* config/aarch64/aarch64.c (aarch64_option_valid_attribute_p):
Initialize simd builtins if TARGET_SIMD.
*
On Fri, Jul 24, 2015 at 09:40:28AM +0100, Kyrill Tkachov wrote:
On 21/07/15 18:14, James Greenhalgh wrote:
On Thu, Jul 16, 2015 at 04:21:15PM +0100, Kyrill Tkachov wrote:
Hi all,
These are the tests for target attributes and pragmas.
I've tried to test for the inlining rules, some of
Richard Biener wrote:
On Wed, Jul 29, 2015 at 3:57 PM, Ulrich Weigand uweig...@de.ibm.com wrote:
Hello,
this patch is a workaround for the problem discussed here:
https://gcc.gnu.org/ml/gcc-patches/2015-07/msg01597.html
The problem is that the new pool allocator code relies on C++
Hi!
On Sun, 2 Aug 2015 21:23:30 -0400, Nathan Sidwell nat...@acm.org wrote:
I've committed this to gomp4 branch. It expands the acc_on_device builtin
earlier in the new oacc_xform pass. This will allow more optimization
earlier on.
As far as I remember, the Fortran front end doesn't know
Hello!
2015-07-30 Kyrylo Tkachov kyrylo.tkac...@arm.com
* ifcvt.c (noce_try_store_flag_constants): Make logic of the case
when diff == STORE_FLAG_VALUE or diff == -STORE_FLAG_VALUE more
explicit. Prefer to add the flag whenever possible.
(noce_process_if_block): Try
Dear Mikael,
Thanks for your green light!
I have been mulling over the trans-decl part of the patch and having
been wondering if it is necessary. Without optimization, private
entities can be linked to. Given the discussion concerning the
combination of submodules and private entities, I wonder
On 08/03/15 07:37, Thomas Schwinge wrote:
Hi!
On Sun, 2 Aug 2015 21:23:30 -0400, Nathan Sidwell nat...@acm.org wrote:
I've committed this to gomp4 branch. It expands the acc_on_device builtin
earlier in the new oacc_xform pass. This will allow more optimization earlier
on.
Thanks!
The
Catherine,
Inline-memcpy-2.c updated to not run with -Os.
Patch rebased off current gcc sources.
Thanks,
Simon
diff --git a/gcc/config/mips/mips.c b/gcc/config/mips/mips.c
index 1733457..627e078 100644
--- a/gcc/config/mips/mips.c
+++ b/gcc/config/mips/mips.c
@@ -7520,12 +7520,22 @@
On Fri, Jul 24, 2015 at 11:55:33AM +0100, Kyrill Tkachov wrote:
Hi all,
This patch implements an aarch64-specific expansion of the signed modulo by a
power of 2.
The proposed sequence makes use of the conditional negate instruction CSNEG.
For a power of N, x % N can be calculated with:
Hi!
On Sun, 2 Aug 2015 21:23:30 -0400, Nathan Sidwell nat...@acm.org wrote:
I've committed this to gomp4 branch. It expands the acc_on_device builtin
earlier in the new oacc_xform pass. This will allow more optimization
earlier on.
Thanks!
The existing expansion point now only needs to
I've committed this to gomp4. The ptx backend can now examine the openacc
attribute to determine launch dimensions and figure out whether vector or worker
single neutering is needed.
nathan
2015-08-03 Nathan Sidwell nat...@codesourcery.com
* config/nvptx/nvptx.c (nvptx_reorg): Check
This changes for lowering for most cases that can end up being tail-merged
to not substitute the operator into the result expression but refer to it
using a local variable of the name of the for iterator.
Boostrapped and tested on x86_64-unknown-linux-gnu, applied to trunk.
This drops the
On 08/02/15 23:44, Jason Merrill wrote:
It seems to me that DR 1658 ignores vbases of abstract classes for determining
whether a destructor is deleted, but says nothing about exception
specifications.
DR 1351 specifically ignores vbases of abstract classes for determining the
exception
1 - 100 of 117 matches
Mail list logo