[Bug translation/80189] gimplify.c: check whether parallel/task/teams should be translated
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80189 Eric Gallager changed: What|Removed |Added Status|ASSIGNED|RESOLVED CC||egallager at gcc dot gnu.org Resolution|--- |FIXED --- Comment #6 from Eric Gallager --- (In reply to Jakub Jelinek from comment #5) > Author: jakub > Date: Thu Mar 30 20:31:40 2017 > New Revision: 246599 > > URL: https://gcc.gnu.org/viewcvs?rev=246599=gcc=rev > Log: > PR translation/80189 > * gimplify.c (omp_default_clause): Use %qs instead of %s in > diagnostic messages. > testsuite/ > * g++.dg/gomp/predetermined-1.C: Adjust expected diagnostics. > * g++.dg/gomp/sharing-1.C: Likewise. > * gfortran.dg/gomp/pr44536.f90: Likewise. > * gfortran.dg/gomp/pr44036-1.f90: Likewise. > * gfortran.dg/gomp/sharing-3.f90: Likewise. > * gfortran.dg/gomp/crayptr3.f90: Likewise. > * gfortran.dg/gomp/pr33439.f90: Likewise. > * gfortran.dg/gomp/appendix-a/a.24.1.f90: Likewise. > * gfortran.dg/gomp/sharing-1.f90: Likewise. > * gfortran.dg/gomp/sharing-2.f90: Likewise. > * gcc.dg/gomp/appendix-a/a.24.1.c: Likewise. > * gcc.dg/gomp/sharing-1.c: Likewise. > > Modified: > trunk/gcc/ChangeLog > trunk/gcc/gimplify.c > trunk/gcc/testsuite/ChangeLog > trunk/gcc/testsuite/g++.dg/gomp/predetermined-1.C > trunk/gcc/testsuite/g++.dg/gomp/sharing-1.C > trunk/gcc/testsuite/gcc.dg/gomp/appendix-a/a.24.1.c > trunk/gcc/testsuite/gcc.dg/gomp/sharing-1.c > trunk/gcc/testsuite/gfortran.dg/gomp/appendix-a/a.24.1.f90 > trunk/gcc/testsuite/gfortran.dg/gomp/crayptr3.f90 > trunk/gcc/testsuite/gfortran.dg/gomp/pr33439.f90 > trunk/gcc/testsuite/gfortran.dg/gomp/pr44036-1.f90 > trunk/gcc/testsuite/gfortran.dg/gomp/pr44536.f90 > trunk/gcc/testsuite/gfortran.dg/gomp/sharing-1.f90 > trunk/gcc/testsuite/gfortran.dg/gomp/sharing-2.f90 > trunk/gcc/testsuite/gfortran.dg/gomp/sharing-3.f90 Assuming this fixed it.
[Bug translation/80189] gimplify.c: check whether parallel/task/teams should be translated
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80189 --- Comment #5 from Jakub Jelinek --- Author: jakub Date: Thu Mar 30 20:31:40 2017 New Revision: 246599 URL: https://gcc.gnu.org/viewcvs?rev=246599=gcc=rev Log: PR translation/80189 * gimplify.c (omp_default_clause): Use %qs instead of %s in diagnostic messages. testsuite/ * g++.dg/gomp/predetermined-1.C: Adjust expected diagnostics. * g++.dg/gomp/sharing-1.C: Likewise. * gfortran.dg/gomp/pr44536.f90: Likewise. * gfortran.dg/gomp/pr44036-1.f90: Likewise. * gfortran.dg/gomp/sharing-3.f90: Likewise. * gfortran.dg/gomp/crayptr3.f90: Likewise. * gfortran.dg/gomp/pr33439.f90: Likewise. * gfortran.dg/gomp/appendix-a/a.24.1.f90: Likewise. * gfortran.dg/gomp/sharing-1.f90: Likewise. * gfortran.dg/gomp/sharing-2.f90: Likewise. * gcc.dg/gomp/appendix-a/a.24.1.c: Likewise. * gcc.dg/gomp/sharing-1.c: Likewise. Modified: trunk/gcc/ChangeLog trunk/gcc/gimplify.c trunk/gcc/testsuite/ChangeLog trunk/gcc/testsuite/g++.dg/gomp/predetermined-1.C trunk/gcc/testsuite/g++.dg/gomp/sharing-1.C trunk/gcc/testsuite/gcc.dg/gomp/appendix-a/a.24.1.c trunk/gcc/testsuite/gcc.dg/gomp/sharing-1.c trunk/gcc/testsuite/gfortran.dg/gomp/appendix-a/a.24.1.f90 trunk/gcc/testsuite/gfortran.dg/gomp/crayptr3.f90 trunk/gcc/testsuite/gfortran.dg/gomp/pr33439.f90 trunk/gcc/testsuite/gfortran.dg/gomp/pr44036-1.f90 trunk/gcc/testsuite/gfortran.dg/gomp/pr44536.f90 trunk/gcc/testsuite/gfortran.dg/gomp/sharing-1.f90 trunk/gcc/testsuite/gfortran.dg/gomp/sharing-2.f90 trunk/gcc/testsuite/gfortran.dg/gomp/sharing-3.f90
[Bug translation/80189] gimplify.c: check whether parallel/task/teams should be translated
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80189 Jakub Jelinek changed: What|Removed |Added Status|UNCONFIRMED |ASSIGNED Last reconfirmed||2017-03-30 Assignee|unassigned at gcc dot gnu.org |jakub at gcc dot gnu.org Ever confirmed|0 |1 --- Comment #4 from Jakub Jelinek --- Created attachment 41089 --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=41089=edit gcc7-pr80189.patch Untested fix.
[Bug translation/80189] gimplify.c: check whether parallel/task/teams should be translated
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80189 --- Comment #3 from Roland Illig --- To me that would be fine since then I had not even looked at what the %qs might be. But like the code is now, I suspected the %s to be part of the grammar of the diagnostic.
[Bug translation/80189] gimplify.c: check whether parallel/task/teams should be translated
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80189 Jakub Jelinek changed: What|Removed |Added CC||jakub at gcc dot gnu.org --- Comment #2 from Jakub Jelinek --- Yeah, these are all keywords. Perhaps we should use %qs instead of %s in both strings though.
[Bug translation/80189] gimplify.c: check whether parallel/task/teams should be translated
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80189 --- Comment #1 from Andrew Pinski --- IIRC these are keywords for OpenMP.
