Tobias Burnus wrote:
Except for the ChangeLog comments above: LGTM.

Actually, it seems as if 3/4 fixes one indentation issue,
introduced by this 1/4 patch. I think it should be folded
into this patch. Namely, 3/4 has:

@@ -3996,9 +4027,9 @@ gfc_match_omp_clauses (gfc_omp_clauses **cp, const 
omp_mask mask,
              else if (gfc_match ("master )") == MATCH_YES)
              {
-                 gfc_warning (OPT_Wdeprecated_openmp, "%<master%> affinity "
-                     "policy at %C deprecated since OpenMP 5.1, use "
-                     "%<primary%>");
+               gfc_warning (OPT_Wdeprecated_openmp, "%<master%> affinity "
+                 "policy at %L deprecated since OpenMP 5.1, use "
+                 "%<primary%>", &old_loc);
                c->proc_bind = OMP_PROC_BIND_MASTER;
              }

Tobias

Reply via email to