https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85422
Tom de Vries changed:
What|Removed |Added
Keywords||openacc
--- Comment #1 from Tom de Vries
Priority: P3
Component: lto
Assignee: unassigned at gcc dot gnu.org
Reporter: vries at gcc dot gnu.org
CC: marxin at gcc dot gnu.org
Target Milestone: ---
When compiling this program:
...
int
main (void)
{
#pragma acc parallel
#pragma acc loop
for (int
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85129
--- Comment #5 from Tom de Vries ---
(In reply to Tom de Vries from comment #0)
> I. trunk
>
> The GOMP_OPENACC_DIM environment variable is not documented (it should have
> an entry in libgomp.texi at 'OpenACC Environment Variables').
Atm, in
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85411
Tom de Vries changed:
What|Removed |Added
Keywords||patch
--- Comment #1 from Tom de Vries
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85129
--- Comment #4 from Tom de Vries ---
(In reply to Thomas Schwinge from comment #3)
> (In reply to Tom de Vries from comment #2)
> > GOMP_OPENACC_DIM seems to be the odd one here, and we should document
> > the non-standard behaviour that it
Priority: P3
Component: libgomp
Assignee: unassigned at gcc dot gnu.org
Reporter: vries at gcc dot gnu.org
CC: jakub at gcc dot gnu.org
Target Milestone: ---
As mentioned here ( https://gcc.gnu.org/ml/gcc-patches/2017-02/msg01020.html
: normal
Priority: P3
Component: target
Assignee: unassigned at gcc dot gnu.org
Reporter: vries at gcc dot gnu.org
Target Milestone: ---
When running parallel-loop-1.c with this patch, it hangs:
...
diff --git a/libgomp/testsuite/libgomp.oacc-c-c++-common
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85246
Tom de Vries changed:
What|Removed |Added
Keywords||wrong-code
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85246
--- Comment #5 from Tom de Vries ---
Created attachment 43921
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=43921=edit
0002-nvptx-Fix-propagation-of-branch-cond-in-vw-neutered-code.patch
Tentative fix.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85246
--- Comment #4 from Tom de Vries ---
Created attachment 43920
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=43920=edit
0001-nvptx-Simplifly-logic-in-nvptx_single.patch
NFC patch to make fix easier
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85246
--- Comment #3 from Tom de Vries ---
(In reply to Tom de Vries from comment #1)
> I went through a couple of cycles of minimizing the failure, seeing
> something suspicious, modifying by hand or writing a tentative patch, but
> every time I went
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85246
--- Comment #2 from Tom de Vries ---
(In reply to Tom de Vries from comment #1)
> I went through a couple of cycles of minimizing the failure, seeing
> something suspicious, modifying by hand or writing a tentative patch, but
> every time I went
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85296
Tom de Vries changed:
What|Removed |Added
Target Milestone|--- |8.0
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85296
Tom de Vries changed:
What|Removed |Added
Status|UNCONFIRMED |RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85296
--- Comment #2 from Tom de Vries ---
Author: vries
Date: Thu Apr 12 07:17:29 2018
New Revision: 259337
URL: https://gcc.gnu.org/viewcvs?rev=259337=gcc=rev
Log:
[nvptx] Fix handling of extern var with flexible array member
2018-04-12 Tom de
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85204
Tom de Vries changed:
What|Removed |Added
Status|ASSIGNED|RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84041
Tom de Vries changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84041
--- Comment #6 from Tom de Vries ---
(In reply to Tom de Vries from comment #4)
> A conservative fix is to define the memory_barrier insn as membar.sys.
Filed PR85341 - "[nvptx] Implement atomic load" to fix this more optimally.
Assignee: unassigned at gcc dot gnu.org
Reporter: vries at gcc dot gnu.org
Target Milestone: ---
[ Follow-up PR of PR84041 - "[nvptx] Hang in for-3.c" ]
Atm the nvptx port does not define an atomic load insn. Consequently, it goes
through the fallback scenario in expand_a
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85303
Tom de Vries changed:
What|Removed |Added
CC||jakub at gcc dot gnu.org
: testsuite
Assignee: unassigned at gcc dot gnu.org
Reporter: vries at gcc dot gnu.org
Target Milestone: ---
In libgomp test-cases, dg-error, dg-warning and dg-bogus can be used, but
dg-message not.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84041
--- Comment #5 from Tom de Vries ---
Author: vries
Date: Mon Apr 9 13:22:00 2018
New Revision: 259233
URL: https://gcc.gnu.org/viewcvs?rev=259233=gcc=rev
Log:
[nvptx] Add memory_barrier insn
2018-04-09 Tom de Vries
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84041
Tom de Vries changed:
What|Removed |Added
Keywords||wrong-code
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85296
--- Comment #1 from Tom de Vries ---
Created attachment 43882
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=43882=edit
Tentative patch
Assignee: unassigned at gcc dot gnu.org
Reporter: vries at gcc dot gnu.org
Target Milestone: ---
We see failures for this testcase:
...
PASS: gcc.dg/torture/pr85244-1.c -O0 (test for excess errors)
FAIL: gcc.dg/torture/pr85244-1.c -O0 execution test
PASS: gcc.dg/torture
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85246
--- Comment #1 from Tom de Vries ---
I went through a couple of cycles of minimizing the failure, seeing something
suspicious, modifying by hand or writing a tentative patch, but every time I
went back to the original non-minimized example I got
Priority: P3
Component: target
Assignee: unassigned at gcc dot gnu.org
Reporter: vries at gcc dot gnu.org
Target Milestone: ---
When adding:
...
/* { dg-additional-options "-foffload=-mlong-vector-in-workers" } */
...
to gemm.f90 it fails.
: normal
Priority: P3
Component: target
Assignee: unassigned at gcc dot gnu.org
Reporter: vries at gcc dot gnu.org
Target Milestone: ---
On the og7 branch, with the current state of the vector-length patches applied,
for vector-length-128-1.c we generate
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85204
--- Comment #3 from Tom de Vries ---
Fixed on trunk.
Todo: revert r241652 on og7 and backport this patch.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85204
--- Comment #2 from Tom de Vries ---
Author: vries
Date: Thu Apr 5 08:36:37 2018
New Revision: 259125
URL: https://gcc.gnu.org/viewcvs?rev=259125=gcc=rev
Log:
[nvptx] Fix neutering of bb with only cond jump
2018-04-05 Tom de Vries
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85223
--- Comment #1 from Tom de Vries ---
> that minimized uses of head and tail,
Also, the use of the long-lived non-descriptive variable "before" needs to be
cleaned up.
Assignee: unassigned at gcc dot gnu.org
Reporter: vries at gcc dot gnu.org
Target Milestone: ---
In nvptx_single, we locate the head and tail of the code that needs neutering
and/or jump condition state propagation, and then insert the insns before and
after head and tail.
PR85204
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85204
--- Comment #1 from Tom de Vries ---
Created attachment 43848
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=43848=edit
Tentative patch
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80547
--- Comment #6 from Tom de Vries ---
The og7 test-case par-reduction-3.c fails with this ICE on trunk.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85221
--- Comment #1 from Tom de Vries ---
Same for declare-allocatable-3.f90 and declare-allocatable-4.f90:
...
$ grep 'internal compiler error:.*install_var_field'
libgomp.testsuite/libgomp.log | sed 's/.*\///' | sort -u
Component: middle-end
Assignee: unassigned at gcc dot gnu.org
Reporter: vries at gcc dot gnu.org
Target Milestone: ---
When compiling the og7 testcase declare-allocatable-1.f with trunk, we get:
...
libgomp/testsuite/libgomp.oacc-fortran/declare-allocatable-1.f90:8:0: Error:
'b
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77371
Tom de Vries changed:
What|Removed |Added
CC||vries at gcc dot gnu.org
--- Comment #8
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85220
--- Comment #2 from Tom de Vries ---
And for libgomp.oacc-fortran/fortran.exp:
...
$ cat libgomp.testsuite/libgomp.sum | grep ^FAIL:.*execution | grep -v /lib- |
sed 's/libgomp.oacc-fortran\///'
FAIL: avoid-offloading-1.f
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85220
--- Comment #1 from Tom de Vries ---
I've copied the test-cases, and build r259123.
For libgomp.oacc-c/c.exp, we get:
...
$ cat libgomp.testsuite/libgomp.sum | grep ^FAIL:.*execution | grep -v /lib- |
sed
Severity: normal
Priority: P3
Component: target
Assignee: unassigned at gcc dot gnu.org
Reporter: vries at gcc dot gnu.org
Target Milestone: ---
OpenACC for nvptx development has been going on in the development branch
openacc-gcc-7-branch (og7 for short
Assignee: unassigned at gcc dot gnu.org
Reporter: vries at gcc dot gnu.org
Target Milestone: ---
Consider the following testcase (broadcast-1.c in the og7 branch):
...
#include
#include
#define N 1024
int A[N][N] ;
void test(int x)
{
#pragma acc parallel num_gangs(16) num_workers(4
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85166
--- Comment #3 from Tom de Vries ---
(In reply to Thomas Koenig from comment #2)
> Does this fix the problem?
>
> Index: minimal.c
> ===
> --- minimal.c (Revision 259055)
> +++
Severity: normal
Priority: P3
Component: target
Assignee: unassigned at gcc dot gnu.org
Reporter: vries at gcc dot gnu.org
Target Milestone: ---
Atm for nvptx offloading we have lots of fortran tests (both openmp and
openacc) fail like this:
...
unresolved
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83126
--- Comment #13 from Tom de Vries ---
(In reply to Arseny Solokha from comment #12)
> I suppose it is fixed.
Filed PR85157 - "[parloops] Prevent canonicalize_loop_ivs failure in
gen_parallel_loop" to fix this in an optimal way.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85157
Tom de Vries changed:
What|Removed |Added
Priority|P3 |P4
Summary|[parloops] Prevent
: tree-optimization
Assignee: unassigned at gcc dot gnu.org
Reporter: vries at gcc dot gnu.org
Target Milestone: ---
In the commit r258713 "[parloops] Handle canonicalize_loop_ivs failure" (a
stage4 fix for PR83126) we bail out of gen_parallel_loop if
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85129
--- Comment #2 from Tom de Vries ---
> There are two test-cases excercising GOMP_OPENACC_DIM (using setenv rather
> than dg-set-target-env-var, to allow remote testing setups to work where
> dg-set-target-env-var is broken)
Actually, using
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85129
Tom de Vries changed:
What|Removed |Added
Keywords||documentation
CC|
Assignee: unassigned at gcc dot gnu.org
Reporter: vries at gcc dot gnu.org
CC: jakub at gcc dot gnu.org
Target Milestone: ---
I. trunk
The GOMP_OPENACC_DIM environment variable is not documented (it should have an
entry in libgomp.texi at 'OpenACC Environment
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85106
Tom de Vries changed:
What|Removed |Added
Keywords||patch
--- Comment #9 from Tom de Vries
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85106
--- Comment #8 from Tom de Vries ---
(In reply to Richard Biener from comment #7)
> Thanks for working on this! Note that there may be multiple ltrans units
> (in theory, not likely in the testsuite), so forcing -flto-partition=one
> when we
>
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85106
--- Comment #6 from Tom de Vries ---
Created attachment 43789
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=43789=edit
Tentative followup patch that adds scan-offload-rtl-dump
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85106
Tom de Vries changed:
What|Removed |Added
Attachment #43786|0 |1
is obsolete|
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85106
Tom de Vries changed:
What|Removed |Added
Attachment #43785|0 |1
is obsolete|
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85106
--- Comment #3 from Tom de Vries ---
Created attachment 43786
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=43786=edit
Tentative followup patch that adds scan-offload-tree-dump
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85106
Tom de Vries changed:
What|Removed |Added
Attachment #43782|0 |1
is obsolete|
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85106
--- Comment #1 from Tom de Vries ---
Created attachment 43782
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=43782=edit
Tentative patch that adds scan-wpa-ipa-dump
Test results look like this (note the scan-wpa-ipa-dump):
...
Test Run By
Component: testsuite
Assignee: unassigned at gcc dot gnu.org
Reporter: vries at gcc dot gnu.org
Target Milestone: ---
https://gcc.gnu.org/ml/gcc-patches/2017-06/msg00321.html :
...
Maybe add scan-ltrans-tree-dump and scan-wpa-ipa-dump that look at appropriate
files plus passing
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84592
Tom de Vries changed:
What|Removed |Added
Summary|[openacc,openmp] lto1: ICE |[openacc,openmp] lto1: ICE
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85063
Tom de Vries changed:
What|Removed |Added
Assignee|unassigned at gcc dot gnu.org |vries at gcc dot gnu.org
Target
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85063
Tom de Vries changed:
What|Removed |Added
Status|UNCONFIRMED |RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85063
--- Comment #10 from Tom de Vries ---
Author: vries
Date: Mon Mar 26 09:45:49 2018
New Revision: 258852
URL: https://gcc.gnu.org/viewcvs?rev=258852=gcc=rev
Log:
Fix switch conversion in offloading functions
2018-03-26 Tom de Vries
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85063
Tom de Vries changed:
What|Removed |Added
Keywords||ice-on-valid-code, patch
--- Comment #9
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85063
Tom de Vries changed:
What|Removed |Added
Keywords||openmp
Summary|Switch
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85063
--- Comment #6 from Tom de Vries ---
(In reply to Jakub Jelinek from comment #3)
> What about switch inside of OpenACC constructs?
The standard forbids function static vars in routines. It doesn't forbid
function static vars in OpenACC
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85063
--- Comment #5 from Tom de Vries ---
> Switch conversion triggers for this example, which introduces
> a function static var,
Hmm, that was my impression from PR84592 comment 5, and the comments in
tree-switch-conversion.c.
When I compile
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85063
--- Comment #4 from Tom de Vries ---
Created attachment 43751
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=43751=edit
Tentative patch that doesn't disable the switch conversion optimization
(In reply to Jakub Jelinek from comment #3)
>
Assignee: unassigned at gcc dot gnu.org
Reporter: vries at gcc dot gnu.org
Target Milestone: ---
Recently I've run into the following issues:
1. https://gcc.gnu.org/bugzilla/show_bug.cgi?id=82415 "Bug 82415 - FAIL:
gcc.target/i386/naked-1.c scan-assembler-not ret"
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85063
--- Comment #2 from Tom de Vries ---
Created attachment 43749
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=43749=edit
Updated tentative patch
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85063
--- Comment #1 from Tom de Vries ---
Created attachment 43748
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=43748=edit
Tentative patch
Currently doing:
- bootstrap & reg-test on x86_64, and
- build and libgomp reg-test on x86_64 with
Priority: P3
Component: tree-optimization
Assignee: unassigned at gcc dot gnu.org
Reporter: vries at gcc dot gnu.org
Target Milestone: ---
Consider this test-case (filed earlier at lto PR84592):
...
/* { dg-additional-options "-ftree-switch-conve
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85056
Tom de Vries changed:
What|Removed |Added
Status|UNCONFIRMED |NEW
Last reconfirmed|
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85056
--- Comment #2 from Tom de Vries ---
Created attachment 43745
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=43745=edit
Tentative patch with testcase
(In reply to cesar from comment #1)
> Created attachment 43744 [details]
> WIP patch
>
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84956
Tom de Vries changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84956
--- Comment #7 from Tom de Vries ---
Author: vries
Date: Thu Mar 22 11:01:15 2018
New Revision: 258762
URL: https://gcc.gnu.org/viewcvs?rev=258762=gcc=rev
Log:
backport "[tail-merge] Don't merge bbs with bb_has_abnormal_pred"
2018-03-22 Tom
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80092
--- Comment #12 from Tom de Vries ---
Author: vries
Date: Thu Mar 22 10:44:51 2018
New Revision: 258760
URL: https://gcc.gnu.org/viewcvs?rev=258760=gcc=rev
Log:
Backport "Require effective target global_constructor for two testcases"
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84956
--- Comment #6 from Tom de Vries ---
Author: vries
Date: Thu Mar 22 10:39:43 2018
New Revision: 258759
URL: https://gcc.gnu.org/viewcvs?rev=258759=gcc=rev
Log:
backport "[tail-merge] Don't merge bbs with bb_has_abnormal_pred"
2018-03-22 Tom
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84956
--- Comment #5 from Tom de Vries ---
Author: vries
Date: Thu Mar 22 10:21:12 2018
New Revision: 258758
URL: https://gcc.gnu.org/viewcvs?rev=258758=gcc=rev
Log:
[tail-merge] Don't merge bbs with bb_has_abnormal_pred
2018-03-22 Tom de Vries
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84956
Tom de Vries changed:
What|Removed |Added
Keywords||patch
--- Comment #4 from Tom de Vries
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84956
--- Comment #3 from Tom de Vries ---
(In reply to Tom de Vries from comment #2)
> Created attachment 43722 [details]
> Tentative patch
Bootstrapped and reg-tested on x86_64, no issues found.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84992
--- Comment #1 from Tom de Vries ---
(In reply to Tom de Vries from comment #0)
>
> But the first question here is: is this a valid openacc program?
>
> Given that function static variables are not supported in functions to which
> a routine
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83126
--- Comment #11 from Tom de Vries ---
Author: vries
Date: Wed Mar 21 12:25:03 2018
New Revision: 258713
URL: https://gcc.gnu.org/viewcvs?rev=258713=gcc=rev
Log:
[parloops] Handle canonicalize_loop_ivs failure
2018-03-21 Tom de Vries
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84956
--- Comment #2 from Tom de Vries ---
Created attachment 43722
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=43722=edit
Tentative patch
-end
Assignee: unassigned at gcc dot gnu.org
Reporter: vries at gcc dot gnu.org
Target Milestone: ---
When compiling this openacc testcase:
#include
int
main (void)
{
int n[1];
n[0] = 3;
#pragma acc parallel copy(n)
{
static const int test[] = {1,2,3,4};
n[0
Priority: P3
Component: middle-end
Assignee: unassigned at gcc dot gnu.org
Reporter: vries at gcc dot gnu.org
Target Milestone: ---
The openacc spec tells us at 2.15.1. Routine Directive:
...
In C and C++, function static variables are not supported
|--- |FIXED
Assignee|unassigned at gcc dot gnu.org |vries at gcc dot gnu.org
Target Milestone|--- |8.0
--- Comment #10 from Tom de Vries ---
(In reply to Tom de Vries from comment #7)
> Created attachment 43708 [details]
> 0002-Verify-ba
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84952
--- Comment #9 from Tom de Vries ---
Author: vries
Date: Tue Mar 20 10:31:23 2018
New Revision: 258676
URL: https://gcc.gnu.org/viewcvs?rev=258676=gcc=rev
Log:
[nvptx] Fix bar.sync position
2018-03-20 Tom de Vries
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84954
Tom de Vries changed:
What|Removed |Added
Status|UNCONFIRMED |RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84954
Tom de Vries changed:
What|Removed |Added
Assignee|unassigned at gcc dot gnu.org |vries at gcc dot gnu.org
Target
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84954
--- Comment #2 from Tom de Vries ---
Author: vries
Date: Tue Mar 20 09:14:07 2018
New Revision: 258674
URL: https://gcc.gnu.org/viewcvs?rev=258674=gcc=rev
Log:
[nvptx] Fix prevent_branch_around_nothing
2018-03-20 Tom de Vries
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84958
--- Comment #1 from Tom de Vries ---
https://gcc.gnu.org/ml/gcc-cvs/2018-03/msg00566.html :
Author: vries
Date: Mon Mar 19 19:12:53 2018
New Revision: 258656
URL: https://gcc.gnu.org/viewcvs?rev=258656=gcc=rev
Log:
[testsuite] Add nvptx xfail
: tree-optimization
Assignee: unassigned at gcc dot gnu.org
Reporter: vries at gcc dot gnu.org
Target Milestone: ---
[ As discussed here: https://gcc.gnu.org/ml/gcc-patches/2018-03/msg00800.html ]
This test-case:
...
int foo()
{
int a[10];
for(int i = 0; i < 10; ++i)
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84952
--- Comment #8 from Tom de Vries ---
(In reply to Tom de Vries from comment #5)
> For stage4, however, we want a fix without fixing optimization issue
> PR84025
> ...
> @ %r34 bra.uni $L8;
> @ %r33 bra $L9;
> // join 2;
> $L9:
> $L8:
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84952
--- Comment #7 from Tom de Vries ---
Created attachment 43708
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=43708=edit
0002-Verify-bar.sync-position.patch (stage4)
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84952
--- Comment #6 from Tom de Vries ---
Created attachment 43707
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=43707=edit
0001-Fix-bar.sync-position.patch (stage4)
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84952
--- Comment #5 from Tom de Vries ---
For stage4, however, we want a fix without fixing optimization issue PR84025,
so we have:
...
$ git log --pretty=%s --reverse HEAD^^..HEAD | cat -n
1 Fix bar.sync position
2 Verify bar.sync
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84954
--- Comment #1 from Tom de Vries ---
Created attachment 43706
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=43706=edit
Tentative patch
Priority: P3
Component: target
Assignee: unassigned at gcc dot gnu.org
Reporter: vries at gcc dot gnu.org
Target Milestone: ---
[ Encountered at PR84952 - [nvptx] bar.sync generated in divergent code ]
Consider testcase workers.c:
...
int
main (void)
{
int a[10
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84952
--- Comment #4 from Tom de Vries ---
Created attachment 43705
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=43705=edit
0003-Verify-bar.sync-position.patch
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84952
--- Comment #3 from Tom de Vries ---
Created attachment 43704
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=43704=edit
0002-Fix-bar.sync-position.patch
901 - 1000 of 3233 matches
Mail list logo