https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66852
--- Comment #5 from Markus Trippelsdorf trippels at gcc dot gnu.org ---
See PR65709 for a similar issue.
Hi Bernhard,
On 11/07/15 00:00, Bernhard Reutner-Fischer wrote:
On 10 July 2015 at 14:31, Kyrill Tkachov kyrylo.tkac...@arm.com wrote:
Hi all,
This patch makes if-conversion more aggressive when handling code of the
form:
if (test)
x := a //THEN
else
x := b //ELSE
The current code
[ was: Re: [gomp4, committed] Handle nested loops in kernels regions ]
On 13/07/15 10:36, Jakub Jelinek wrote:
On Mon, Jul 13, 2015 at 10:19:56AM +0200, Thomas Schwinge wrote:
We rely on pass_lim to move the *.omp_data_i loads out of the loop nest.
For the test-case, pass_lim was managing to
Hi Jim!
On Mon, 8 Jun 2015 10:06:21 -0500, James Norris jnor...@codesourcery.com
wrote:
--- /dev/null
+++ b/libgomp/testsuite/libgomp.oacc-c++/declare-1.C
@@ -0,0 +1,24 @@
+
+templateclass T
+T foo()
+{
+ T a;
+ #pragma acc declare create (a)
+
+ #pragma acc parallel
+ {
+a
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66858
Bug ID: 66858
Summary: [6 Regression] FAIL: g++.dg/pch/system-2.C -O2 -g
assembly comparison on aarch64-none-elf, arm-none-eabi
Product: gcc
Version: 6.0
Status:
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66852
--- Comment #7 from Markus Trippelsdorf trippels at gcc dot gnu.org ---
(In reply to Richard Biener from comment #6)
So I suppose the IsAligned template is wrong.
Yes.
390 template class T
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66852
--- Comment #8 from Jonathan Wakely redi at gcc dot gnu.org ---
The code in algparam.h is just disgusting. AssignFromHelperClass binds a
reference to NULL just to default-construct a temporary of some type, then
binds a const-reference to that
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66852
--- Comment #9 from Jeffrey Walton noloader at gmail dot com ---
(In reply to Jonathan Wakely from comment #8)
The code in algparam.h is just disgusting. AssignFromHelperClass binds a
reference to NULL just to default-construct a temporary of
On July 13, 2015 11:45:55 AM GMT+02:00, Kyrill Tkachov kyrylo.tkac...@arm.com
wrote:
Hi Bernhard,
Did you include go in your testing?
I see:
Unexpected results in this build (new failures)
FAIL: encoding/json
FAIL: go/printer
FAIL: go/scanner
FAIL: html/template
FAIL: log
FAIL:
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65709
Jeffrey Walton noloader at gmail dot com changed:
What|Removed |Added
CC||noloader at
Hi,
this patch fixes PR46193.
It handles min and max reductions of pointer type in parloops.
Bootstrapped and reg-tested on x86_64.
OK for trunk?
Thanks,
- Tom
Handle mix/max pointer reductions in parloops
2015-07-13 Tom de Vries t...@codesourcery.com
PR tree-optimization/46193
*
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=46193
vries at gcc dot gnu.org changed:
What|Removed |Added
Keywords||patch
--- Comment #4 from
Hi,
On Mon, Jul 13, 2015 at 02:13:59PM +0200, Paolo Bonzini wrote:
On 13/07/2015 13:55, Martin Jambor wrote:
I can't approve it, but FWIW, I'm generally fine with the patch.
Although the original idea was to share one func_body_info in between
ipa-cp and ipa-inline analyses, this is
On Mon, Jul 13, 2015 at 10:19:56AM +0200, Thomas Schwinge wrote:
We rely on pass_lim to move the *.omp_data_i loads out of the loop nest.
For the test-case, pass_lim was managing to move the load out of the
inner loop, but not the outer loop, because the load was classified as
On Thu, Jul 9, 2015 at 8:02 PM, Armin Rigo ar...@tunes.org wrote:
Hi all,
Here is an updated patch (attached) for __seg_fs and __seg_gs:
* added a target hook default_pointer_address_modes to avoid
disabling a few gcc optimizations which, according to my reading of
the documentation, should
On Fri, 10 Jul 2015, Richard Biener wrote:
On Fri, 10 Jul 2015, Richard Biener wrote:
I was just testing the patch below which runs into latent issues when
building libjava (at least)...
/space/rguenther/src/svn/trunk/libjava/java/lang/natClassLoader.cc: In
function
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66842
Jonathan Wakely redi at gcc dot gnu.org changed:
What|Removed |Added
Component|libstdc++ |c
--- Comment
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65709
--- Comment #15 from Jakub Jelinek jakub at gcc dot gnu.org ---
I'm saying that if the program does not trigger undefined behavior (e.g. by
accessing misaligned integers without telling the compiler about it (by using
memcpy, or packed attribute
On 13/07/2015 13:55, Martin Jambor wrote:
I can't approve it, but FWIW, I'm generally fine with the patch.
Although the original idea was to share one func_body_info in between
ipa-cp and ipa-inline analyses, this is certainly better than what we
have now and perhaps even good enough
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66857
Jonathan Wakely redi at gcc dot gnu.org changed:
What|Removed |Added
Keywords||wrong-code
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66852
Markus Trippelsdorf trippels at gcc dot gnu.org changed:
What|Removed |Added
Status|WAITING
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66852
--- Comment #11 from Jeffrey Walton noloader at gmail dot com ---
(In reply to Jonathan Wakely from comment #8)
The code in algparam.h is just disgusting. AssignFromHelperClass binds a
reference to NULL just to default-construct a temporary of
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66809
Eric Botcazou ebotcazou at gcc dot gnu.org changed:
What|Removed |Added
Status|UNCONFIRMED |RESOLVED
Try to recycle the last non-nested team to avoid the use of malloc() and
free() in the normal case where the number of threads is the same.
Avoid superfluous destruction and initialization of team synchronization
objects.
Using the microbenchmark posted here
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65709
--- Comment #16 from Jeffrey Walton noloader at gmail dot com ---
(In reply to Jakub Jelinek from comment #15)
I'm saying that if the program does not trigger undefined behavior (e.g. by
accessing misaligned integers without telling the
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66856
Bug ID: 66856
Summary: [6 Regression] ICE in compute_live_loop_exits, at
tree-ssa-loop-manip.c:234
Product: gcc
Version: 6.0
Status: UNCONFIRMED
Severity:
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66555
Marek Polacek mpolacek at gcc dot gnu.org changed:
What|Removed |Added
Status|NEW |ASSIGNED
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66857
--- Comment #1 from Sebastian Lauwers sebastian.lauwers at gmail dot com ---
Command: g++ -v -save-temps test.cpp
Output:
Using built-in specs.
COLLECT_GCC=g++
COLLECT_LTO_WRAPPER=/usr/libexec/gcc/x86_64-redhat-linux/5.1.1/lto-wrapper
Target:
Ping
2015-06-18 12:54 GMT+03:00 Ilya Enkovich enkovich@gmail.com:
Hi,
In early_inliner we do recompute inline summaries for edges after
optimize_inline_calls, but check this summary exists in case new edges
appear. But then it calls inline_update_overall_summary which also going
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66852
Richard Biener rguenth at gcc dot gnu.org changed:
What|Removed |Added
Target||x86_64-*-*
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66854
Bug ID: 66854
Summary: libgcc2.c:1846:9: internal compiler error:
Segmentation fault
Product: gcc
Version: 6.0
Status: UNCONFIRMED
Severity: normal
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66852
--- Comment #10 from Jeffrey Walton noloader at gmail dot com ---
(In reply to Richard Biener from comment #6)
So I suppose the IsAligned template is wrong.
So I'm clear (please forgive my ignorance)...
According to
Hi Tom!
On Mon, 13 Jul 2015 12:32:20 +0200, Tom de Vries tom_devr...@mentor.com wrote:
On 13/07/15 10:31, Thomas Schwinge wrote:
On Mon, 13 Jul 2015 09:20:16 +0200, Tom de Vriestom_devr...@mentor.com
wrote:
On 12/07/15 11:39, Thomas Schwinge wrote:
I've looked at the merge commit,
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=58066
--- Comment #14 from Uroš Bizjak ubizjak at gmail dot com ---
(In reply to Uroš Bizjak from comment #13)
Patch in testing.
This patch fixes the testcase, now we get:
inet_ntoa:
0: 41 56 push %r14
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66760
--- Comment #4 from Martin Jambor jamborm at gcc dot gnu.org ---
Paolo has submitted a patch for this issue to the mailing list:
https://gcc.gnu.org/ml/gcc-patches/2015-07/msg00984.html
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65709
--- Comment #17 from Jakub Jelinek jakub at gcc dot gnu.org ---
(In reply to Jeffrey Walton from comment #16)
OK, so you'll have to forgive my ignorance again.
So you are saying that it may be a bug to use vmovdqa if the source and/or
Hi Mikael, hi everyone,
thanks for the review, Mikael. Commited as r225730.
Regards,
Andre
On Fri, 10 Jul 2015 18:12:53 +0200
Mikael Morin mikael.mo...@sfr.fr wrote:
Le 10/07/2015 16:51, Andre Vehreschild a écrit :
Hi everyone,
attached is a rather trivial patch to fix a
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64589
--- Comment #5 from vehre at gcc dot gnu.org ---
Author: vehre
Date: Mon Jul 13 09:01:54 2015
New Revision: 225730
URL: https://gcc.gnu.org/viewcvs?rev=225730root=gccview=rev
Log:
gcc/testsuite/ChangeLog:
2015-07-13 Andre Vehreschild
On 10/07/15 21:21, James Greenhalgh wrote:
On Fri, Jul 10, 2015 at 01:21:05PM +0100, Kyrill Tkachov wrote:
Hi all,
Some of the testcases in aarch64.exp can fail their scan-assembler patterns
if if-conversion becomes more aggressive.
This patch adjusts the testcases in case the branches are
On 13 July 2015 at 01:25, Ville Voutilainen ville.voutilai...@gmail.com wrote:
On 12 July 2015 at 21:45, Ville Voutilainen ville.voutilai...@gmail.com
wrote:
Tested on Linux-PPC64.
2015-07-12 Ville Voutilainen ville.voutilai...@gmail.com
Implement
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64968
--- Comment #47 from Markus Trippelsdorf trippels at gcc dot gnu.org ---
Another cosmetic issue that I've noticed is that an extra newline is added
after every quoted comment.
For example:
test
reply
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=58066
--- Comment #13 from Uroš Bizjak ubizjak at gmail dot com ---
Created attachment 35964
-- https://gcc.gnu.org/bugzilla/attachment.cgi?id=35964action=edit
Combined middle/end/target patch
Patch in testing.
Hi!
On Fri, 10 Jul 2015 11:04:14 +0200, I wrote:
On Thu, 09 Jul 2015 20:25:22 -0400, Nathan Sidwell nat...@acm.org wrote:
This is the patch I committed.
2. Don't be shy to remove a bunch of XFAILs, in fact all :-) of those
remaining from the test cases that Julian had added in
On Thu, Jun 25, 2015 at 6:08 PM, Michael Collison
michael.colli...@linaro.org wrote:
This patch is designed to optimize constructs such as:
#define min(x, y) ((x) = (y)) ? (x) : (y)
unsignedint foo (unsignedint i, unsignedint x ,unsignedint y)
{
return i (min (x, y));
}
int bar
Hi,
On Sun, Jul 12, 2015 at 11:39:55PM +0200, Paolo Bonzini wrote:
From: bonz...@gnu.org
In this PR, a lot of time is spent doing the same ipa_load_from_parm_agg
query over and over. Luckily a memoization scheme is already there, it's
just not used by ipa-inline-analysis.c. The patch
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=58066
Uroš Bizjak ubizjak at gmail dot com changed:
What|Removed |Added
Component|target |rtl-optimization
On Mon, 13 Jul 2015, Richard Biener wrote:
On Sun, 12 Jul 2015, Jeff Law wrote:
On 06/29/2015 01:58 AM, Richard Biener wrote:
In principle the following works for the testcase (even w/o fixing
the VRP part).
Index: gcc/tree-ssa-dom.c
On 13/07/15 10:31, Thomas Schwinge wrote:
Hi Tom!
On Mon, 13 Jul 2015 09:20:16 +0200, Tom de Vriestom_devr...@mentor.com wrote:
On 12/07/15 11:39, Thomas Schwinge wrote:
On Fri, 10 Jul 2015 18:50:20 -0400, Nathan
Sidwellnathan_sidw...@mentor.com wrote:
it looks like the most recent
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66852
--- Comment #12 from Markus Trippelsdorf trippels at gcc dot gnu.org ---
(In reply to Jeffrey Walton from comment #10)
(In reply to Richard Biener from comment #6)
So I suppose the IsAligned template is wrong.
So I'm clear (please forgive
On 07/07/15 09:53, Tom de Vries wrote:
Hi,
currently, we have these spec strings in gcc/gcc.c involving
ftree-parallelize-loops:
...
%{fopenacc|fopenmp|ftree-parallelize-loops=*:%:include(libgomp.spec)%(link_gomp)}
%{fopenacc|fopenmp|ftree-parallelize-loops=*:-pthread}
...
Actually,
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=58066
--- Comment #15 from H.J. Lu hjl.tools at gmail dot com ---
(In reply to Uroš Bizjak from comment #13)
Created attachment 35964 [details]
Combined middle/end/target patch
Patch in testing.
I tried it on GCC 5 and it works on glibc. Thanks.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66857
Bug ID: 66857
Summary: Reference not bound to lvalue
Product: gcc
Version: 5.1.1
Status: UNCONFIRMED
Severity: normal
Priority: P3
Component: c++
On 13/07/2015 14:34, Martin Jambor wrote:
You might want to use Martin's shiny new
function_summary class in symbol-summary.c. That is a mechanism
specifically designed to append to a cgraph_node information specific
to an optimization pass (or two, as ipa-cp and ipa-inline already both
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66852
--- Comment #6 from Richard Biener rguenth at gcc dot gnu.org ---
So I suppose the IsAligned template is wrong.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66726
--- Comment #9 from Andreas Schwab sch...@linux-m68k.org ---
On m68k:
FAIL: gcc.dg/pr46309.c scan-tree-dump-times reassoc1 Optimizing range tests
a_[0-9]*.D. -.1, 1. and -.3, 3.[\n\r]* into 1
FAIL: gcc.dg/pr46309.c scan-tree-dump-times reassoc1
Hi all,
For the testcase in the patch we were generating an extra neg instruction:
cmp w0, wzr
csneg w0, w0, w0, ge
neg w0, w0
ret
instead of the optimal:
cmp w0, wzr
csneg w0, w0, w0, lt
ret
The reason is that combine
On 13/07/15 10:45, Kyrill Tkachov wrote:
+/* Return iff the registers that the insns in BB_A set do not
+ get used in BB_B. */
Return true iff
I tried to be too formal here ;) https://en.wikipedia.org/wiki/If_and_only_if
I'll use a normal if here.
Err, of course you were talking about
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66855
Bug ID: 66855
Summary: codecvt wrong endianness in UTF-16 conversions
Product: gcc
Version: 5.1.1
Status: UNCONFIRMED
Severity: normal
Priority: P3
Component:
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65913
Jonathan Wakely redi at gcc dot gnu.org changed:
What|Removed |Added
Status|UNCONFIRMED |NEW
Last
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66855
Jonathan Wakely redi at gcc dot gnu.org changed:
What|Removed |Added
Status|UNCONFIRMED |ASSIGNED
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66848
--- Comment #8 from Jack Howarth howarth.at.gcc at gmail dot com ---
Note that radr://21372179 has been closed by Apple as behaves as expected' so
that they believe the bug lies in the FSF gcc boehm-gc code.
[CC Jason since the patch also touches the C++ front end]
The updated patch is here:
https://gcc.gnu.org/ml/gcc-patches/2015-07/msg00258.html
Thanks
Martin
On 07/04/2015 04:32 PM, Martin Sebor wrote:
I don't think c_validate_addressable is a good name - given that it's
called for lots of
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66834
--- Comment #10 from Jason Merrill jason at gcc dot gnu.org ---
(In reply to Jason Merrill from comment #9)
So it seems that applying the DR 1430 tentative resolution to concepts severely
limits the usability of variadic concepts, and we should
On 07/13/2015 03:32 AM, Richard Biener wrote:
On Mon, 13 Jul 2015, Richard Biener wrote:
On Sun, 12 Jul 2015, Jeff Law wrote:
On 06/29/2015 01:58 AM, Richard Biener wrote:
In principle the following works for the testcase (even w/o fixing
the VRP part).
Index: gcc/tree-ssa-dom.c
On 07/13/2015 11:03 AM, Uros Bizjak wrote:
This is rtl-optimization part of a two-part patch series.
As discussed in the PR, we have to prcompute register parameters
before stack alignment is performed, otherwise eventual call to
__tls_get_addr can be called with unaligned stack. When compiling
On 07/13/2015 04:58 AM, Tom de Vries wrote:
On 07/07/15 09:53, Tom de Vries wrote:
Hi,
currently, we have these spec strings in gcc/gcc.c involving
ftree-parallelize-loops:
...
%{fopenacc|fopenmp|ftree-parallelize-loops=*:%:include(libgomp.spec)%(link_gomp)}
Still looking for a review of this small patch to help detect
potentially unsafe calls to __builtin_{frame,return}_address
(with an argument 2) that tend to either return bogus values
or lead to crashes at runtime.
The problem the function can cause was originally reported in
pr8743 and
Hi Richard and Gerald,
This patch adds the documentation in changes.html for the GCC trunk (gcc-6) .
Please let me know if it is good to commit.
Index: htdocs/gcc-6/changes.html
===
RCS file:
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66864
Bug ID: 66864
Summary: floor function error
Product: gcc
Version: 5.1.0
Status: UNCONFIRMED
Severity: normal
Priority: P3
Component: fortran
Assignee:
Thread model: posix
gcc version 6.0.0 20150713 (experimental) [trunk revision 225727] (GCC)
$
$ gcc-trunk -O1 small.c; ./a.out
$ gcc-5.1 -Os small.c; ./a.out
$
$ gcc-trunk -Os small.c
$ ./a.out
Floating point exception (core dumped)
$
-
int a, b;
int
fn1 (int
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=23664
hs.naveen naveen_gcc at indiatimes dot com changed:
What|Removed |Added
CC||naveen_gcc
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66852
--- Comment #13 from Jeffrey Walton noloader at gmail dot com ---
A quick update
We did out best to take the advice of Jakub, Janathan, Markus and others:
https://github.com/weidai11/cryptopp/commit/9bf0eed0f6ff6d0b0a2d277e5401d69dc8c0e394.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66092
--- Comment #5 from Jason Merrill jason at gcc dot gnu.org ---
Author: jason
Date: Tue Jul 14 04:11:11 2015
New Revision: 225758
URL: https://gcc.gnu.org/viewcvs?rev=225758root=gccview=rev
Log:
PR c++/66092
PR c++/66834
*
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66834
--- Comment #11 from Jason Merrill jason at gcc dot gnu.org ---
Author: jason
Date: Tue Jul 14 04:11:11 2015
New Revision: 225758
URL: https://gcc.gnu.org/viewcvs?rev=225758root=gccview=rev
Log:
PR c++/66092
PR c++/66834
On 07/12/2015 07:56 AM, Segher Boessenkool wrote:
Currently combine tries to make assignments to bitfields (of a register)
whenever it can. If the target has no insv pattern, the result will not
ever match (if the MD is sane at all). Doing insv on registers generates
worse code than what you
On 07/12/2015 12:35 PM, Uros Bizjak wrote:
Another missing case of CALL_INSN_FUNCTION_USAGE, where clobbered
registers are also marked, this time in postreload/
reload_cse_move2add.
Fixed compiler now generates following code
callsysv_abi_func
movl$global, %esi
Hi Tom,
this patch provides a 'bool independent' field in struct loop, which
will be switched on by an independent clause in a #pragma acc loop directive.
I assume you'll be wiring it to the kernels parloops pass in a followup patch.
Note: there are already a few other similar fields in struct
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=19987
Bug 19987 depends on bug 23664, which changed state.
Bug 23664 Summary: fold does not change (aC1)+(bC2) to (aC1)|(bC2) iff (C1
C2) == 0
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=23664
What|Removed |Added
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=23664
Andrew Pinski pinskia at gcc dot gnu.org changed:
What|Removed |Added
Status|ASSIGNED|RESOLVED
On Thu, Jul 09, 2015 at 04:06:57PM +0200, Jakub Jelinek wrote:
The latest spec adds a bunch of new functions, this patch attempts to
implement them, except I gave up partly in omp_target_associate_ptr
and completely in omp_target_disassociate_ptr for now.
As for the plugins, I think we'll
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66857
Markus Trippelsdorf trippels at gcc dot gnu.org changed:
What|Removed |Added
CC||jason
Hi,
On Mon, 13 Jul 2015, Tom de Vries wrote:
Implementing multi-step maps or making the hashmaps non-caching
doesn't solve any of the above problems
I'm not saying that making those hashmaps non-caching solves any of
these problems.
Ah, I didn't mean to imply this, I meant to imply
On Thu, Jul 09, 2015 at 12:00:29 +0200, Thomas Schwinge wrote:
I noticed that -- at least with current versions of GCC -- there are
several compiler diagnostics displayed during the build. It would be
nice to get these addressed -- as applicable, presumably in the Intel
upstream version, and
On 07/13/15 07:26, Thomas Schwinge wrote:
Hi!
On Fri, 10 Jul 2015 11:04:14 +0200, I wrote:
On Thu, 09 Jul 2015 20:25:22 -0400, Nathan Sidwell nat...@acm.org wrote:
This is the patch I committed.
2. Don't be shy to remove a bunch of XFAILs, in fact all :-) of those
remaining from the test
On Mon, Jul 13, 2015 at 3:30 PM, Martin Jambor mjam...@suse.cz wrote:
On Mon, Jul 13, 2015 at 02:47:23PM +0200, Paolo Bonzini wrote:
On 13/07/2015 14:34, Martin Jambor wrote:
You might want to use Martin's shiny new
function_summary class in symbol-summary.c. That is a mechanism
On 13/07/2015 15:45, Richard Biener wrote:
It would be nice to have a patch that can be backported to the GCC 5 branch
as well. We can improve this on trunk as followup,no?
The patch I've already posted can be backported. O:-)
Paolo
On Mon, Jul 13, 2015 at 01:15:44PM +0200, Sebastian Huber wrote:
diff --git a/libgomp/team.c b/libgomp/team.c
index b98b233..0bcbaf8 100644
--- a/libgomp/team.c
+++ b/libgomp/team.c
@@ -134,6 +134,25 @@ gomp_thread_start (void *xdata)
return NULL;
}
+static struct gomp_team *
On Mon, Jul 13, 2015 at 15:17:29 +0200, Jakub Jelinek wrote:
Here is a new version that I've committed. I've finished up
associate/disassociate, wrote a test and tested also with intelmicemul
offloading.
Great!
+ k-refcount = INT_MAX;
Shouldn't it be UINTPTR_MAX?
+ /* FIXME:
Hi,
while going thru the pending template template issues, I noticed that
check_template_shadow still emits two errors. I believe we also want to
simply use D instead of #D for template parameters, that avoids the nit
about class T vs template T which could be slightly confusing to
novices,
On Mon, Jul 13, 2015 at 3:46 PM, Paolo Bonzini pbonz...@redhat.com wrote:
On 13/07/2015 15:45, Richard Biener wrote:
It would be nice to have a patch that can be backported to the GCC 5 branch
as well. We can improve this on trunk as followup,no?
The patch I've already posted can be
On Mon, Jul 13, 2015 at 09:42:50 -0400, Nathan Sidwell wrote:
GOMP_offload_register's target data argument is 'void *'. Is there
any reason it shouldn't be 'const void *'? It would seem to me that
that would be better?
(a cursory look at i386/intelmic-mkoffload.c suggests a lack of
consts
Bernd,
this patch changes the offload target data type from an array of void *, to a
struct, which is somewhat easier to deal with than remembering numeric indices
and type casts.
This is step 1 in reworking the launch API.
ok?
nathan
2015-07-13 Nathan Sidwell nat...@acm.org
gcc/
*
Hi Bernhard,
On 13/07/15 10:45, Kyrill Tkachov wrote:
PS: no -mbranch-cost and, a tad more seriously, no --param branch-cost either ;)
PPS: attached meant to illustrate comments above. Untested.
Thanks a lot! This is all very helpful.
I'll respin the patch.
Here it is. I've expanded the
On 13/07/15 15:43, Michael Matz wrote:
Hi,
On Sun, 12 Jul 2015, Tom de Vries wrote:
I'm trying to get to a defined policy for what is allowed for caches.
Either forbidding or allowing multi-step dependencies, I don't really
mind.
I think forbidding is the way to go, because ...
I managed
On 13/07/15 16:17, Jakub Jelinek wrote:
On Mon, Jul 13, 2015 at 01:15:44PM +0200, Sebastian Huber wrote:
diff --git a/libgomp/team.c b/libgomp/team.c
index b98b233..0bcbaf8 100644
--- a/libgomp/team.c
+++ b/libgomp/team.c
@@ -134,6 +134,25 @@ gomp_thread_start (void *xdata)
return NULL;
On 07/13/2015 03:57 PM, Nathan Sidwell wrote:
this patch changes the offload target data type from an array of void *,
to a struct, which is somewhat easier to deal with than remembering
numeric indices and type casts.
This is step 1 in reworking the launch API.
Looks fine.
Bernd
Hi,
this patch fixes PR66851.
In parloops, we manage to parallelize outer loops, but not if the inner
loop contains a reduction. There is an xfail in autopar/outer-4.c for this:
...
/* { dg-final { scan-tree-dump-times parallelizing outer loop 1
parloops { xfail *-*-* } } } */
...
This
Ilya,
GOMP_offload_register's target data argument is 'void *'. Is there any reason
it shouldn't be 'const void *'? It would seem to me that that would be better?
(a cursory look at i386/intelmic-mkoffload.c suggests a lack of consts in the
variable decls there. ptx suffers the same
Nice clean-up :-)
tbsaunde+...@tbsaunde.org writes:
@@ -2248,11 +2249,14 @@ process_define_subst (void)
static void
rtx_handle_directive (int lineno, const char *rtx_name)
{
- rtx queue, x;
+ auto_vecrtx, 10 subrtxs;
+ if (!read_rtx (rtx_name, subrtxs))
+return;
Very minor, but
Hi,
On Sun, 12 Jul 2015, Tom de Vries wrote:
I'm trying to get to a defined policy for what is allowed for caches.
Either forbidding or allowing multi-step dependencies, I don't really
mind.
I think forbidding is the way to go, because ...
I managed to write a patch series that
1 - 100 of 197 matches
Mail list logo
