]> git.ipfire.org Git - thirdparty/gcc.git/commitdiff
Kernels loops annotation: C and C++.
authorSandra Loosemore <sandra@codesourcery.com>
Sun, 15 Mar 2020 22:13:46 +0000 (15:13 -0700)
committerKwok Cheung Yeung <kcy@codesourcery.com>
Thu, 22 Apr 2021 17:14:23 +0000 (10:14 -0700)
This patch detects loops in kernels regions that are candidates for
parallelization, and adds "#pragma acc loop auto" annotations to them.
This annotation is controlled by the -fopenacc-kernels-annotate-loops
option, which is enabled by default.  -Wopenacc-kernels-annotate-loops
can be used to produce diagnostics about loops that cannot be annotated.

2020-03-27  Sandra Loosemore  <sandra@codesourcery.com>

Kernels loops annotation: C and C++.

gcc/c-family/
* c-common.h (c_oacc_annotate_loops_in_kernels_regions): Declare.
* c-omp.c: Include tree-iterator.h
(enum annotation_state): New.
(struct annotation_info): New.
(do_not_annotate_loop): New.
(do_not_annotate_loop_nest): New.
(annotation_error): New.
(c_finish_omp_for_internal): Split from c_finish_omp_for.  Use
annotation_error function.  Code refactoring to avoid destructive
changes that cannot be undone in case of error.
(is_local_var): New.
(lang_specific_unwrap_initializer): New.
(annotate_for_loop): New.
(check_and_annotate_for_loop): New.
(annotate_loops_in_kernels_regions): New.
(c_oacc_annotate_loops_in_kernels_regions): New.
* c.opt (Wopenacc-kernels-annotate-loops): New.
(fopenacc-kernels-annotate-loops): New.

gcc/c/
* c-decl.c (c_unwrap_for_init): New.
(finish_function): Call c_oacc_annotate_loops_in_kernels_regions.

gcc/cp/
* decl.c (cp_unwrap_for_init): New.
(finish_function): Call c_oacc_annotate_loops_in_kernels_regions.

gcc/
* doc/invoke.texi (Option Summary): Add entries for
-Wopenacc-kernels-annotate-loops and
-fno-openacc-kernels-annotate-loops.
(Warning Options): Document -Wopenacc-kernels-annotate-loops.
(Optimization Options): Document -fno-openacc-kernels-annotate-loops.

gcc/testsuite/
* c-c++-common/goacc/classify-kernels-unparallelized.c: Add
-fno-openacc-kernels-annotate-loops option.
* c-c++-common/goacc/classify-kernels.c: Likewise.
* c-c++-common/goacc/kernels-counter-var-redundant-load.c: Likewise.
* c-c++-common/goacc/kernels-counter-vars-function-scope.c: Likewise.
* c-c++-common/goacc/kernels-double-reduction-n.c: Likewise.
* c-c++-common/goacc/kernels-double-reduction.c: Likewise.
* c-c++-common/goacc/kernels-loop-2.c: Likewise.
* c-c++-common/goacc/kernels-loop-3.c: Likewise.
* c-c++-common/goacc/kernels-loop-data-2.c: Likewise.
* c-c++-common/goacc/kernels-loop-data-enter-exit-2.c: Likewise.
* c-c++-common/goacc/kernels-loop-data-enter-exit.c: Likewise.
* c-c++-common/goacc/kernels-loop-data-update.c: Likewise.
* c-c++-common/goacc/kernels-loop-data.c: Likewise.
* c-c++-common/goacc/kernels-loop-g.c: Likewise.
* c-c++-common/goacc/kernels-loop-mod-not-zero.c: Likewise.
* c-c++-common/goacc/kernels-loop-n.c: Likewise.
* c-c++-common/goacc/kernels-loop-nest.c: Likewise.
* c-c++-common/goacc/kernels-loop.c: Likewise.
* c-c++-common/goacc/kernels-one-counter-var.c: Likewise.
* c-c++-common/goacc/kernels-parallel-loop-data-enter-exit.c:
Likewise.
* c-c++-common/goacc/kernels-reduction.c: Likewise.
* c-c++-common/goacc/kernels-loop-annotation-1.c: New.
* c-c++-common/goacc/kernels-loop-annotation-2.c: New.
* c-c++-common/goacc/kernels-loop-annotation-3.c: New.
* c-c++-common/goacc/kernels-loop-annotation-4.c: New.
* c-c++-common/goacc/kernels-loop-annotation-5.c: New.
* c-c++-common/goacc/kernels-loop-annotation-6.c: New.
* c-c++-common/goacc/kernels-loop-annotation-7.c: New.
* c-c++-common/goacc/kernels-loop-annotation-8.c: New.
* c-c++-common/goacc/kernels-loop-annotation-9.c: New.
* c-c++-common/goacc/kernels-loop-annotation-10.c: New.
* c-c++-common/goacc/kernels-loop-annotation-11.c: New.
* c-c++-common/goacc/kernels-loop-annotation-12.c: New.
* c-c++-common/goacc/kernels-loop-annotation-13.c: New.
* c-c++-common/goacc/kernels-loop-annotation-14.c: New.
* c-c++-common/goacc/kernels-loop-annotation-15.c: New.
* c-c++-common/goacc/kernels-loop-annotation-16.c: New.
* c-c++-common/goacc/kernels-loop-annotation-17.c: New.

49 files changed:
gcc/ChangeLog.omp
gcc/c-family/ChangeLog.omp
gcc/c-family/c-common.h
gcc/c-family/c-omp.c
gcc/c-family/c.opt
gcc/c/ChangeLog.omp
gcc/c/c-decl.c
gcc/cp/ChangeLog.omp
gcc/cp/decl.c
gcc/doc/invoke.texi
gcc/testsuite/ChangeLog.omp
gcc/testsuite/c-c++-common/goacc/classify-kernels-unparallelized.c
gcc/testsuite/c-c++-common/goacc/classify-kernels.c
gcc/testsuite/c-c++-common/goacc/kernels-counter-var-redundant-load.c
gcc/testsuite/c-c++-common/goacc/kernels-counter-vars-function-scope.c
gcc/testsuite/c-c++-common/goacc/kernels-double-reduction-n.c
gcc/testsuite/c-c++-common/goacc/kernels-double-reduction.c
gcc/testsuite/c-c++-common/goacc/kernels-loop-2.c
gcc/testsuite/c-c++-common/goacc/kernels-loop-3.c
gcc/testsuite/c-c++-common/goacc/kernels-loop-annotation-1.c [new file with mode: 0644]
gcc/testsuite/c-c++-common/goacc/kernels-loop-annotation-10.c [new file with mode: 0644]
gcc/testsuite/c-c++-common/goacc/kernels-loop-annotation-11.c [new file with mode: 0644]
gcc/testsuite/c-c++-common/goacc/kernels-loop-annotation-12.c [new file with mode: 0644]
gcc/testsuite/c-c++-common/goacc/kernels-loop-annotation-13.c [new file with mode: 0644]
gcc/testsuite/c-c++-common/goacc/kernels-loop-annotation-14.c [new file with mode: 0644]
gcc/testsuite/c-c++-common/goacc/kernels-loop-annotation-15.c [new file with mode: 0644]
gcc/testsuite/c-c++-common/goacc/kernels-loop-annotation-16.c [new file with mode: 0644]
gcc/testsuite/c-c++-common/goacc/kernels-loop-annotation-17.c [new file with mode: 0644]
gcc/testsuite/c-c++-common/goacc/kernels-loop-annotation-2.c [new file with mode: 0644]
gcc/testsuite/c-c++-common/goacc/kernels-loop-annotation-3.c [new file with mode: 0644]
gcc/testsuite/c-c++-common/goacc/kernels-loop-annotation-4.c [new file with mode: 0644]
gcc/testsuite/c-c++-common/goacc/kernels-loop-annotation-5.c [new file with mode: 0644]
gcc/testsuite/c-c++-common/goacc/kernels-loop-annotation-6.c [new file with mode: 0644]
gcc/testsuite/c-c++-common/goacc/kernels-loop-annotation-7.c [new file with mode: 0644]
gcc/testsuite/c-c++-common/goacc/kernels-loop-annotation-8.c [new file with mode: 0644]
gcc/testsuite/c-c++-common/goacc/kernels-loop-annotation-9.c [new file with mode: 0644]
gcc/testsuite/c-c++-common/goacc/kernels-loop-data-2.c
gcc/testsuite/c-c++-common/goacc/kernels-loop-data-enter-exit-2.c
gcc/testsuite/c-c++-common/goacc/kernels-loop-data-enter-exit.c
gcc/testsuite/c-c++-common/goacc/kernels-loop-data-update.c
gcc/testsuite/c-c++-common/goacc/kernels-loop-data.c
gcc/testsuite/c-c++-common/goacc/kernels-loop-g.c
gcc/testsuite/c-c++-common/goacc/kernels-loop-mod-not-zero.c
gcc/testsuite/c-c++-common/goacc/kernels-loop-n.c
gcc/testsuite/c-c++-common/goacc/kernels-loop-nest.c
gcc/testsuite/c-c++-common/goacc/kernels-loop.c
gcc/testsuite/c-c++-common/goacc/kernels-one-counter-var.c
gcc/testsuite/c-c++-common/goacc/kernels-parallel-loop-data-enter-exit.c
gcc/testsuite/c-c++-common/goacc/kernels-reduction.c

index 301a9ece91b6953de203cf42f515495df8bca598..632e8000b10a85afccd954f3b3f5235849ac0ea9 100644 (file)
@@ -1,3 +1,11 @@
+2020-03-27  Sandra Loosemore  <sandra@codesourcery.com>
+
+       * doc/invoke.texi (Option Summary): Add entries for
+       -Wopenacc-kernels-annotate-loops and
+       -fno-openacc-kernels-annotate-loops.
+       (Warning Options): Document -Wopenacc-kernels-annotate-loops.
+       (Optimization Options): Document -fno-openacc-kernels-annotate-loops.
+
 2020-03-19  Kwok Cheung Yeung  <kcy@codesourcery.com>
 
        * omp-offload.c (oacc_loop_auto_partitions): Check for 'omp declare
index 3aa2991f6f321df36615ad2a201ee1644aef5c3d..17247b205f829cf0a775dbcc1a35b69af450a822 100644 (file)
@@ -1,3 +1,24 @@
+2020-03-27  Sandra Loosemore  <sandra@codesourcery.com>
+
+       * c-common.h (c_oacc_annotate_loops_in_kernels_regions): Declare.
+       * c-omp.c: Include tree-iterator.h
+       (enum annotation_state): New.
+       (struct annotation_info): New.
+       (do_not_annotate_loop): New.
+       (do_not_annotate_loop_nest): New.
+       (annotation_error): New.
+       (c_finish_omp_for_internal): Split from c_finish_omp_for.  Use
+       annotation_error function.  Code refactoring to avoid destructive
+       changes that cannot be undone in case of error.
+       (is_local_var): New.
+       (lang_specific_unwrap_initializer): New.
+       (annotate_for_loop): New.
+       (check_and_annotate_for_loop): New.
+       (annotate_loops_in_kernels_regions): New.
+       (c_oacc_annotate_loops_in_kernels_regions): New.
+       * c.opt (Wopenacc-kernels-annotate-loops): New.
+       (fopenacc-kernels-annotate-loops): New.
+
 2019-01-09  Julian Brown  <julian@codesourcery.com>
 
        * c-cppbuiltin.c (c_cpp_builtins): Update _OPENACC define to 201711.
index f30b6c6ac33474d27939372c4529e9513804b67e..4b5a2bcf3c78c3519ba2c51eb7cf70be1f1bb4e8 100644 (file)
@@ -1239,6 +1239,7 @@ extern enum omp_clause_default_kind c_omp_predetermined_sharing (tree);
 extern enum omp_clause_defaultmap_kind c_omp_predetermined_mapping (tree);
 extern tree c_omp_check_context_selector (location_t, tree);
 extern void c_omp_mark_declare_variant (location_t, tree, tree);
+extern void c_oacc_annotate_loops_in_kernels_regions (tree, tree (*) (tree));
 extern const char *c_omp_map_clause_name (tree, bool);
 extern void c_omp_adjust_map_clauses (tree, bool);
 
index fb1a0d6041ec7cd215b83ac6f48bdfdae332f314..ef01cdc83468cfc722d254353747c946be5e1a00 100644 (file)
@@ -36,7 +36,7 @@ along with GCC; see the file COPYING3.  If not see
 #include "gimplify.h"
 #include "langhooks.h"
 #include "bitmap.h"
-
+#include "tree-iterator.h"
 
 /* Complete a #pragma oacc wait construct.  LOC is the location of
    the #pragma.  */
@@ -695,6 +695,110 @@ c_omp_for_incr_canonicalize_ptr (location_t loc, tree decl, tree incr)
   return incr;
 }
 
+/* State of annotation traversal for FOR loops in kernels regions,
+   used to control processing and diagnostic messages that are deferred until
+   the entire loop has been scanned.  */
+enum annotation_state {
+  as_outer,
+  as_in_kernels_region,
+  as_in_kernels_loop,
+  /* The remaining state values represent conversion failures caught
+     while in as_in_kernels_loop state.  To test whether the traversal is
+     in the body of a kernels loop, use (state >= as_in_kernels_loop).  */
+  as_invalid_variable_type,
+  as_missing_initializer,
+  as_invalid_initializer,
+  as_missing_predicate,
+  as_invalid_predicate,
+  as_missing_increment,
+  as_invalid_increment,
+  as_explicit_annotation,
+  as_invalid_control_flow,
+  as_invalid_break,
+  as_invalid_return,
+  as_invalid_call,
+  as_invalid_modification
+};
+
+/* Structure used to hold state for automatic annotation of FOR loops
+   in kernels regions.  LOOP is the nearest enclosing loop, or
+   NULL_TREE if outside of a loop context.  VARS is a tree_list
+   containing the variables controlling LOOP's termination (the
+   induction variable and a possible limit variable).  STATE keeps
+   track of whether loop satisfies all criteria making it legal to
+   parallelize.  Otherwise, REASON is a statement that blocks
+   automatic parallelization, such as an unstructured jump or an
+   assignment to a variable in VARS, used for printing diagnostics.
+
+   These structures are chained through NEXT, which points to the
+   next-closest enclosing loop's or the kernels region's annotation info, if
+   any.  */
+
+struct annotation_info
+{
+  tree loop;
+  tree vars;
+  bool break_ok;
+  enum annotation_state state;
+  tree reason;
+  struct annotation_info *next;
+};
+
+/* Mark the current loop's INFO as not OK to annotate, recording STATE
+   and REASON for producing diagnostics later.  */
+
+static void
+do_not_annotate_loop (struct annotation_info *info,
+                     enum annotation_state state, tree reason)
+{
+  if (info->state == as_in_kernels_loop)
+    {
+      info->state = state;
+      info->reason = reason;
+    }
+}
+
+/* Mark the current loop identified by INFO and all of its ancestors (i.e.,
+   enclosing loops) as not OK to annotate.  Arguments are the same as
+   for do_not_annotate_loop.  */
+
+static void
+do_not_annotate_loop_nest (struct annotation_info *info,
+                          enum annotation_state state, tree reason)
+{
+  while (info != NULL)
+    {
+      do_not_annotate_loop (info, state, reason);
+      info = info->next;
+    }
+}
+
+/* If INFO is non-null, call do_not_annotate_loop with STATE and REASON
+   to record info for diagnosing an error later.  Otherwise emit an error now
+   at ELOCUS with message MSG and the optional arguments.  */
+
+static void annotation_error (struct annotation_info *,
+                             enum annotation_state, tree, location_t,
+                             const char *, ...) ATTRIBUTE_GCC_DIAG(5,6);
+static
+void annotation_error (struct annotation_info *info,
+                             enum annotation_state state,
+                             tree reason,
+                             location_t elocus,
+                             const char *msg, ...)
+{
+  if (info)
+    do_not_annotate_loop (info, state, reason);
+  else
+    {
+      auto_diagnostic_group d;
+      va_list ap;
+      va_start (ap, msg);
+      emit_diagnostic_valist (DK_ERROR, elocus, -1, msg, &ap);
+      va_end (ap);
+    }
+}
+
 /* Validate and generate OMP_FOR.
    DECLV is a vector of iteration variables, for each collapsed loop.
 
@@ -704,12 +808,19 @@ c_omp_for_incr_canonicalize_ptr (location_t loc, tree decl, tree incr)
    INITV, CONDV and INCRV are vectors containing initialization
    expressions, controlling predicates and increment expressions.
    BODY is the body of the loop and PRE_BODY statements that go before
-   the loop.  */
+   the loop.  FINAL_P is true if not inside a C++ template.
 
-tree
-c_finish_omp_for (location_t locus, enum tree_code code, tree declv,
-                 tree orig_declv, tree initv, tree condv, tree incrv,
-                 tree body, tree pre_body, bool final_p)
+   INFO is null if called to parse an explicitly-annotated OMP for
+   loop, otherwise it holds state information for automatically
+   annotating a regular FOR loop in a kernels region.  In the former case,
+   malformed loops are hard errors; otherwise we just record the annotation
+   failure in INFO.  */
+
+static tree
+c_finish_omp_for_internal (location_t locus, enum tree_code code, tree declv,
+                          tree orig_declv, tree initv, tree condv, tree incrv,
+                          tree body, tree pre_body, bool final_p,
+                          struct annotation_info *info)
 {
   location_t elocus;
   bool fail = false;
@@ -733,12 +844,14 @@ c_finish_omp_for (location_t locus, enum tree_code code, tree declv,
       if (!INTEGRAL_TYPE_P (TREE_TYPE (decl))
          && TREE_CODE (TREE_TYPE (decl)) != POINTER_TYPE)
        {
-         error_at (elocus, "invalid type for iteration variable %qE", decl);
+         annotation_error (info, as_invalid_variable_type, decl, elocus,
+                           "invalid type for iteration variable %qE", decl);
          fail = true;
        }
       else if (TYPE_ATOMIC (TREE_TYPE (decl)))
        {
-         error_at (elocus, "%<_Atomic%> iteration variable %qE", decl);
+         annotation_error (info, as_invalid_variable_type, decl, elocus,
+                           "%<_Atomic%> iteration variable %qE", decl);
          fail = true;
          /* _Atomic iterator confuses stuff too much, so we risk ICE
             trying to diagnose it further.  */
@@ -754,7 +867,8 @@ c_finish_omp_for (location_t locus, enum tree_code code, tree declv,
          init = DECL_INITIAL (decl);
          if (init == NULL)
            {
-             error_at (elocus, "%qE is not initialized", decl);
+             annotation_error (info, as_missing_initializer, decl, elocus,
+                               "%qE is not initialized", decl);
              init = integer_zero_node;
              fail = true;
            }
@@ -775,7 +889,8 @@ c_finish_omp_for (location_t locus, enum tree_code code, tree declv,
 
       if (cond == NULL_TREE)
        {
-         error_at (elocus, "missing controlling predicate");
+         annotation_error (info, as_missing_predicate, NULL_TREE, elocus,
+                           "missing controlling predicate");
          fail = true;
        }
       else
@@ -791,12 +906,14 @@ c_finish_omp_for (location_t locus, enum tree_code code, tree declv,
          if (EXPR_HAS_LOCATION (cond))
            elocus = EXPR_LOCATION (cond);
 
-         if (TREE_CODE (cond) == LT_EXPR
-             || TREE_CODE (cond) == LE_EXPR
-             || TREE_CODE (cond) == GT_EXPR
-             || TREE_CODE (cond) == GE_EXPR
-             || TREE_CODE (cond) == NE_EXPR
-             || TREE_CODE (cond) == EQ_EXPR)
+         enum tree_code condcode = TREE_CODE (cond);
+
+         if (condcode == LT_EXPR
+             || condcode == LE_EXPR
+             || condcode == GT_EXPR
+             || condcode == GE_EXPR
+             || condcode == NE_EXPR
+             || condcode == EQ_EXPR)
            {
              tree op0 = TREE_OPERAND (cond, 0);
              tree op1 = TREE_OPERAND (cond, 1);
@@ -816,79 +933,88 @@ c_finish_omp_for (location_t locus, enum tree_code code, tree declv,
              if (TREE_CODE (op0) == NOP_EXPR
                  && decl == TREE_OPERAND (op0, 0))
                {
-                 TREE_OPERAND (cond, 0) = TREE_OPERAND (op0, 0);
-                 TREE_OPERAND (cond, 1)
-                   = fold_build1_loc (elocus, NOP_EXPR, TREE_TYPE (decl),
-                                  TREE_OPERAND (cond, 1));
+                 op0 = TREE_OPERAND (op0, 0);
+                 op1 = fold_build1_loc (elocus, NOP_EXPR, TREE_TYPE (decl),
+                                        op1);
                }
              else if (TREE_CODE (op1) == NOP_EXPR
                       && decl == TREE_OPERAND (op1, 0))
                {
-                 TREE_OPERAND (cond, 1) = TREE_OPERAND (op1, 0);
-                 TREE_OPERAND (cond, 0)
-                   = fold_build1_loc (elocus, NOP_EXPR, TREE_TYPE (decl),
-                                  TREE_OPERAND (cond, 0));
+                 op1 = TREE_OPERAND (op1, 0);
+                 op0 = fold_build1_loc (elocus, NOP_EXPR, TREE_TYPE (decl),
+                                        op0);
                }
 
-             if (decl == TREE_OPERAND (cond, 0))
+             if (decl == op0)
                cond_ok = true;
-             else if (decl == TREE_OPERAND (cond, 1))
+             else if (decl == op1)
                {
-                 TREE_SET_CODE (cond,
-                                swap_tree_comparison (TREE_CODE (cond)));
-                 TREE_OPERAND (cond, 1) = TREE_OPERAND (cond, 0);
-                 TREE_OPERAND (cond, 0) = decl;
+                 condcode = swap_tree_comparison (condcode);
+                 op1 = op0;
+                 op0 = decl;
                  cond_ok = true;
                }
 
-             if (TREE_CODE (cond) == NE_EXPR
-                 || TREE_CODE (cond) == EQ_EXPR)
+             if (condcode == NE_EXPR || condcode == EQ_EXPR)
                {
                  if (!INTEGRAL_TYPE_P (TREE_TYPE (decl)))
                    {
-                     if (code == OACC_LOOP || TREE_CODE (cond) == EQ_EXPR)
+                     if (code == OACC_LOOP || condcode == EQ_EXPR)
                        cond_ok = false;
                    }
-                 else if (operand_equal_p (TREE_OPERAND (cond, 1),
+                 else if (operand_equal_p (op1,
                                            TYPE_MIN_VALUE (TREE_TYPE (decl)),
                                            0))
-                   TREE_SET_CODE (cond, TREE_CODE (cond) == NE_EXPR
-                                        ? GT_EXPR : LE_EXPR);
-                 else if (operand_equal_p (TREE_OPERAND (cond, 1),
+                   condcode = (condcode == NE_EXPR ? GT_EXPR : LE_EXPR);
+                 else if (operand_equal_p (op1,
                                            TYPE_MAX_VALUE (TREE_TYPE (decl)),
                                            0))
-                   TREE_SET_CODE (cond, TREE_CODE (cond) == NE_EXPR
-                                        ? LT_EXPR : GE_EXPR);
-                 else if (code == OACC_LOOP || TREE_CODE (cond) == EQ_EXPR)
+                   condcode = (condcode == NE_EXPR ? LT_EXPR : GE_EXPR);
+                 else if (code == OACC_LOOP || condcode == EQ_EXPR)
                    cond_ok = false;
                }
 
-             if (cond_ok && TREE_VEC_ELT (condv, i) != cond)
+             if (cond_ok)
                {
-                 tree ce = NULL_TREE, *pce = &ce;
-                 tree type = TREE_TYPE (TREE_OPERAND (cond, 1));
-                 for (tree c = TREE_VEC_ELT (condv, i); c != cond;
-                      c = TREE_OPERAND (c, 1))
+                 /* We postponed destructive changes to canonicalize
+                    cond until we're sure it is OK.  In the !error_p
+                    case where we are trying to transform a regular FOR_STMT
+                    to OMP_FOR, we don't want to destroy the original
+                    condition if we aren't going to be able to do the
+                    transformation anyway.  */
+                 TREE_SET_CODE (cond, condcode);
+                 TREE_OPERAND (cond, 0) = op0;
+                 TREE_OPERAND (cond, 1) = op1;
+
+                 if (TREE_VEC_ELT (condv, i) != cond)
                    {
-                     *pce = build2 (COMPOUND_EXPR, type, TREE_OPERAND (c, 0),
-                                    TREE_OPERAND (cond, 1));
-                     pce = &TREE_OPERAND (*pce, 1);
+                     tree ce = NULL_TREE, *pce = &ce;
+                     tree type = TREE_TYPE (op1);
+                     for (tree c = TREE_VEC_ELT (condv, i); c != cond;
+                          c = TREE_OPERAND (c, 1))
+                       {
+                         *pce = build2 (COMPOUND_EXPR, type,
+                                        TREE_OPERAND (c, 0), op1);
+                         pce = &TREE_OPERAND (*pce, 1);
+                       }
+                     op1 = ce;
+                     TREE_VEC_ELT (condv, i) = cond;
                    }
-                 TREE_OPERAND (cond, 1) = ce;
-                 TREE_VEC_ELT (condv, i) = cond;
                }
            }
 
          if (!cond_ok)
            {
-             error_at (elocus, "invalid controlling predicate");
+             annotation_error (info, as_invalid_predicate, cond, elocus,
+                               "invalid controlling predicate");
              fail = true;
            }
        }
 
       if (incr == NULL_TREE)
        {
-         error_at (elocus, "missing increment expression");
+         annotation_error (info, as_missing_increment, NULL_TREE, elocus,
+                           "missing increment expression");
          fail = true;
        }
       else
@@ -987,9 +1113,11 @@ c_finish_omp_for (location_t locus, enum tree_code code, tree declv,
                              if (i == NULL_TREE
                                  || !operand_equal_p (unit, i, 0))
                                {
-                                 error_at (elocus,
-                                           "increment is not constant 1 or "
-                                           "-1 for %<!=%> condition");
+                                 annotation_error (info,
+                                                   as_invalid_increment,
+                                                   incr, elocus,
+                                                   "increment is not constant 1 or "
+                                                   "-1 for %<!=%> condition");
                                  fail = true;
                                }
                            }
@@ -1005,9 +1133,10 @@ c_finish_omp_for (location_t locus, enum tree_code code, tree declv,
                    {
                      if (!integer_onep (i) && !integer_minus_onep (i))
                        {
-                         error_at (elocus,
-                                   "increment is not constant 1 or -1 for"
-                                   " %<!=%> condition");
+                         annotation_error (info, as_invalid_increment,
+                                           incr, elocus,
+                                           "increment is not constant 1 or -1 for"
+                                           " %<!=%> condition");
                          fail = true;
                        }
                    }
@@ -1019,7 +1148,8 @@ c_finish_omp_for (location_t locus, enum tree_code code, tree declv,
            }
          if (!incr_ok)
            {
-             error_at (elocus, "invalid increment expression");
+             annotation_error (info, as_invalid_increment, incr,
+                               elocus, "invalid increment expression");
              fail = true;
            }
        }
@@ -1047,6 +1177,20 @@ c_finish_omp_for (location_t locus, enum tree_code code, tree declv,
     }
 }
 
+/* External entry point to c_finish_omp_for_internal, called from the
+   parsers.  See above for description of the arguments.  */
+
+tree
+c_finish_omp_for (location_t locus, enum tree_code code, tree declv,
+                 tree orig_declv, tree initv, tree condv, tree incrv,
+                 tree body, tree pre_body, bool final_p)
+{
+  return c_finish_omp_for_internal (locus, code, declv,
+                                   orig_declv, initv, condv, incrv,
+                                   body, pre_body, final_p, NULL);
+}
+
+
 /* Type for passing data in between c_omp_check_loop_iv and
    c_omp_check_loop_iv_r.  */
 
@@ -2772,6 +2916,543 @@ c_omp_map_clause_name (tree clause, bool oacc)
   return omp_clause_code_name[OMP_CLAUSE_CODE (clause)];
 }
 
+/* The following functions implement automatic recognition and annotation of
+   for loops in OpenACC kernels regions.  Inside a kernels region, a nest of
+   for loops that does not contain any annotated OpenACC loops, nor break
+   or goto statements or assignments to the variables controlling loop
+   termination, is converted to an OMP_FOR node with an "acc loop auto"
+   annotation on each loop.  This feature is controlled by
+   flag_openacc_kernels_annotate_loops.  */
+
+/* Check whether DECL is the declaration of a local variable (or function
+   parameter) of integral type that does not have its address taken.  */
+
+static bool
+is_local_var (tree decl)
+{
+  return ((TREE_CODE (decl) == VAR_DECL || TREE_CODE (decl) == PARM_DECL)
+         && DECL_CONTEXT (decl) != NULL
+         && TREE_CODE (DECL_CONTEXT (decl)) == FUNCTION_DECL
+         && INTEGRAL_TYPE_P (TREE_TYPE (decl))
+         && !TREE_ADDRESSABLE (decl));
+}
+
+/* The initializer for a FOR_STMT is sometimes wrapped in various other
+   language-specific tree structures.  We need a hook to unwrap them.
+   This function takes a tree argument and should return either a
+   MODIFY_EXPR, VAR_DECL, or NULL_TREE.  */
+
+static tree (*lang_specific_unwrap_initializer) (tree);
+
+/* Try to annotate the given NODE, which must be a FOR_STMT, with a
+   "#pragma acc loop auto" annotation.  In practice, this means
+   building an OMP_FOR node for it.  PREV_STMT is the statement
+   immediately before the loop, which may be used as the loop's
+   initialization statement.  Annotating the loop may fail, in which
+   case INFO is used to record the cause of the failure and the
+   original loop remains unchanged.  This function returns the
+   transformed loop if the transformation succeeded, the original node
+   otherwise.  */
+
+static tree
+annotate_for_loop (tree node, tree_stmt_iterator *prev_tsi,
+                  struct annotation_info *info)
+{
+  gcc_checking_assert (TREE_CODE (node) == FOR_STMT);
+
+  location_t loc = EXPR_LOCATION (node);
+  tree cond = FOR_COND (node);
+  gcc_assert (cond);
+  tree decl = TREE_OPERAND (cond, 0);
+  gcc_assert (decl && TREE_CODE (decl) == VAR_DECL);
+  tree init = FOR_INIT_STMT (node);
+  tree prev_stmt = NULL_TREE;
+  bool unlink_prev = false;
+  bool fix_decl = false;
+
+
+  /* Both the C and C++ front ends normally put the initializer in the
+     statement list just before the FOR_STMT instead of in FOR_INIT_STMT.
+     If FOR_INIT_STMT happens to exist but isn't a MODIFY_EXPR, bail out
+     because the code below won't handle it.  */
+  if (init != NULL_TREE && TREE_CODE (init) != MODIFY_EXPR)
+    {
+      do_not_annotate_loop (info, as_invalid_initializer, NULL_TREE);
+      return node;
+    }
+
+  /* Examine the statement before the loop to see if it is a
+     valid initializer.  It must be either a MODIFY_EXPR or VAR_DECL,
+     possibly wrapped in language-specific structure.  */
+  if (init == NULL_TREE && prev_tsi != NULL)
+    {
+      prev_stmt = tsi_stmt (*prev_tsi);
+
+      /* Call the language-specific hook to unwrap prev_stmt.  */
+      if (prev_stmt)
+       prev_stmt = (*lang_specific_unwrap_initializer) (prev_stmt);
+
+      /* See if we have a valid MODIFY_EXPR.  */
+      if (prev_stmt
+         && TREE_CODE (prev_stmt) == MODIFY_EXPR
+         && TREE_OPERAND (prev_stmt, 0) == decl
+         && !TREE_SIDE_EFFECTS (TREE_OPERAND (prev_stmt, 1)))
+       {
+         init = prev_stmt;
+         unlink_prev = true;
+       }
+      else if (prev_stmt == decl
+              && !TREE_SIDE_EFFECTS (DECL_INITIAL (decl)))
+       {
+         /* If the preceding statement is the declaration of the loop
+            variable with its initialization, build an assignment
+            expression for the loop's initializer.  */
+         init = build2 (MODIFY_EXPR, TREE_TYPE (decl), decl,
+                        DECL_INITIAL (decl));
+         /* We need to remove the initializer from the decl if we
+            end up using the init we just built instead.  */
+         fix_decl = true;
+       }
+    }
+
+  if (init == NULL_TREE)
+    /* There is nothing we can do to find the correct init statement for
+       this loop, but c_finish_omp_for insists on having one and would fail
+       otherwise.  In that case, we would just return node.  Do that
+       directly, here.  */
+    {
+      do_not_annotate_loop (info, as_missing_initializer, NULL_TREE);
+      return node;
+    }
+
+  tree incr = FOR_EXPR (node);
+
+  /* The C++ frontend can wrap the increment two levels deep inside a
+     cleanup expression, but c_finish_omp_for does not care about that.  */
+  if (incr != NULL_TREE && TREE_CODE (incr) == CLEANUP_POINT_EXPR)
+    incr = TREE_OPERAND (TREE_OPERAND (incr, 0), 0);
+  tree body = FOR_BODY (node);
+
+  tree declv = make_tree_vec (1);
+  tree initv = make_tree_vec (1);
+  tree condv = make_tree_vec (1);
+  tree incrv = make_tree_vec (1);
+  TREE_VEC_ELT (declv, 0) = decl;
+  TREE_VEC_ELT (initv, 0) = init;
+  TREE_VEC_ELT (condv, 0) = cond;
+  TREE_VEC_ELT (incrv, 0) = incr;
+
+  /* Do the actual transformation.  This can still fail because
+     c_finish_omp_for has some stricter checks than we have performed up to
+     this point.  */
+  tree omp_for = c_finish_omp_for_internal (loc, OACC_LOOP, declv, NULL_TREE,
+                                           initv, condv, incrv, body,
+                                           NULL_TREE, false, info);
+  if (omp_for != NULL_TREE)
+    {
+      if (unlink_prev)
+       /* We don't need the previous statement that we consumed as an
+          initializer in the new OMP_FOR any more.  */
+       tsi_delink (prev_tsi);
+
+      if (fix_decl)
+       /* We no longer need the initializer expression on the decl of
+          the loop variable and don't want to duplicate it.  The
+          kernels conversion pass would interpret it as a stray
+          assignment in a gang-single region.  */
+       DECL_INITIAL (prev_stmt) = NULL_TREE;
+
+      /* Add an auto clause, then return the new loop.  */
+      tree auto_clause = build_omp_clause (loc, OMP_CLAUSE_AUTO);
+      OMP_CLAUSE_CHAIN (auto_clause) = OMP_FOR_CLAUSES (omp_for);
+      OMP_FOR_CLAUSES (omp_for) = auto_clause;
+      return omp_for;
+    }
+
+  return node;
+}
+
+/* Forward declaration.  */
+static tree annotate_loops_in_kernels_regions (tree *, int *, void *);
+
+/* Given a FOR_STMT NODE that is a candidate for parallelization, check its
+   body for validity, then try to annotate it with
+   "#pragma oacc loop auto", possibly modifying the current node in place.
+   The INFO argument contains the traversal state at the point the loop
+   appears.  */
+
+static void
+check_and_annotate_for_loop (tree *nodeptr, tree_stmt_iterator *prev_tsi,
+                            struct annotation_info *info)
+{
+  tree node = *nodeptr;
+  gcc_assert (TREE_CODE (node) == FOR_STMT);
+
+  /* This structure describes the current loop statement.  */
+  struct annotation_info loop_info
+    = { node, NULL_TREE, false, as_in_kernels_loop, NULL_TREE, info };
+  tree cond = FOR_COND (node);
+
+  /* If we are in the body of an explicitly-annotated loop, do not add
+     annotations to this loop or any other nested loops.  */
+  if (info->state == as_explicit_annotation)
+    do_not_annotate_loop (&loop_info, as_explicit_annotation, info->reason);
+
+  /* We need to find the controlling variable for the loop in order
+     to detect whether it is modified in the body of the loop.
+     That is why we are doing some checks on the loop condition
+     that duplicate what c_finish_omp_for is doing.  */
+
+  /* The loop condition must be a comparison.  */
+  else if (cond == NULL_TREE)
+    do_not_annotate_loop (&loop_info, as_missing_predicate, NULL_TREE);
+  else if (TREE_CODE_CLASS (TREE_CODE (cond)) != tcc_comparison)
+    do_not_annotate_loop (&loop_info, as_invalid_predicate, cond);
+  else
+    {
+      /* The condition's LHS must be a local variable that does not
+        have its address taken.  Its RHS must also be such a local
+        variable or a constant.  */
+      tree induction_var = TREE_OPERAND (cond, 0);
+      tree limit_var = TREE_OPERAND (cond, 1);
+      if (!is_local_var (induction_var)
+         || (!is_local_var (limit_var)
+             && (TREE_CODE_CLASS (TREE_CODE (limit_var))
+                 != tcc_constant)))
+       do_not_annotate_loop (&loop_info, as_invalid_predicate, cond);
+      else
+       {
+         /* These variables must not be assigned to in the loop.  */
+         loop_info.vars = tree_cons (NULL_TREE, induction_var,
+                                     loop_info.vars);
+         if (TREE_CODE_CLASS (TREE_CODE (limit_var)) != tcc_constant)
+           loop_info.vars = tree_cons (NULL_TREE, limit_var, loop_info.vars);
+       }
+    }
+
+  /* Walk the body.  This will process any nested loops, so we have to do it
+     even if we have already rejected this loop as a candidate for
+     annotation.  */
+  walk_tree (&FOR_BODY (node), annotate_loops_in_kernels_regions,
+            (void *) &loop_info, NULL);
+
+  if (loop_info.state == as_in_kernels_loop)
+    {
+      /* If the traversal of the loop and all nested loops didn't hit
+        any problems, attempt the actual transformation.  If it
+        succeeds, replace this node with the annotated loop.  */
+      tree result = annotate_for_loop (node, prev_tsi, &loop_info);
+      if (result != node)
+       {
+         /* Success!  */
+         *nodeptr = result;
+         return;
+       }
+    }
+
+  /* If we got here, we have a FOR_STMT we could not convert to an
+     OMP loop.  */
+
+  if (loop_info.state == as_invalid_return)
+    /* This is diagnosed elsewhere as a hard error, so no warning is
+       needed here.  */
+    return;
+
+  /* Issue warnings about other problems.  */
+  auto_diagnostic_group d;
+  if (warning_at (EXPR_LOCATION (node),
+                 OPT_Wopenacc_kernels_annotate_loops,
+                 "loop cannot be annotated for OpenACC parallelization"))
+    {
+      location_t locus;
+      if (loop_info.reason && EXPR_HAS_LOCATION (loop_info.reason))
+       locus = EXPR_LOCATION (loop_info.reason);
+      else
+       locus = EXPR_LOCATION (node);
+      switch (loop_info.state)
+       {
+       case as_invalid_variable_type:
+         inform (locus, "invalid type for iteration variable %qE",
+                 loop_info.reason);
+         break;
+       case as_missing_initializer:
+         inform (locus, "missing iteration variable initializer");
+         break;
+       case as_invalid_initializer:
+         inform (locus, "unrecognized initializer");
+         break;
+       case as_missing_predicate:
+         inform (locus, "missing controlling predicate");
+         break;
+       case as_invalid_predicate:
+         inform (locus, "invalid controlling predicate");
+         break;
+       case as_missing_increment:
+         inform (locus, "missing increment expression");
+         break;
+       case as_invalid_increment:
+         inform (locus, "invalid increment expression");
+         break;
+       case as_explicit_annotation:
+         inform (locus, "explicit OpenACC annotation in loop nest");
+         break;
+       case as_invalid_control_flow:
+         inform (locus, "loop contains unstructured control flow");
+         break;
+       case as_invalid_break:
+         inform (locus, "loop contains %<break%> statement");
+         break;
+       case as_invalid_call:
+         inform (locus, "loop contains call to non-oacc function");
+         break;
+       case as_invalid_modification:
+         inform (locus, "invalid modification of controlling variable");
+         break;
+       default:
+         gcc_unreachable ();
+       }
+    }
+}
+
+/* Traversal function for walk_tree.  Visit the tree, finding OpenACC
+   kernels regions.  DATA is NULL if we are outside of a kernels region,
+   otherwise it is a pointer to the enclosing kernels region's
+   annotation_info struct.  If the traversal encounters a for loop inside a
+   kernels region that is a candidate for parallelization, annotate it
+   with OpenACC loop directives.  */
+
+static tree
+annotate_loops_in_kernels_regions (tree *nodeptr, int *walk_subtrees,
+                                  void *data)
+{
+  tree node = *nodeptr;
+  struct annotation_info *info = (struct annotation_info *) data;
+  gcc_assert (info);
+
+  switch (TREE_CODE (node))
+    {
+    case OACC_KERNELS:
+      /* Recursively process the body of the kernels region in a new info
+        scope.  */
+      if (info->state == as_outer)
+       {
+         struct annotation_info nested_info
+           = { NULL_TREE, NULL_TREE, true,
+               as_in_kernels_region, NULL_TREE, info };
+         walk_tree (&OMP_BODY (node), annotate_loops_in_kernels_regions,
+                    (void *) &nested_info, NULL);
+         *walk_subtrees = 0;
+       }
+      break;
+
+    case OACC_LOOP:
+      /* Do not try to add automatic OpenACC annotations inside manually
+        annotated loops.  Presumably, the user avoided doing it on
+        purpose; for example, all available levels of parallelism may
+        have been used up.  */
+      {
+       struct annotation_info nested_info
+         = { NULL_TREE, NULL_TREE, false, as_explicit_annotation,
+             node, info };
+       if (info->state >= as_in_kernels_region)
+         do_not_annotate_loop_nest (info, as_explicit_annotation,
+                                    node);
+       walk_tree (&OMP_BODY (node), annotate_loops_in_kernels_regions,
+                  (void *) &nested_info, NULL);
+       *walk_subtrees = 0;
+      }
+      break;
+
+    case FOR_STMT:
+      /* Try to annotate the loop if we are in a kernels region.
+        This will do a recursive traversal of the loop body in a new
+        info scope.  */
+      if (info->state >= as_in_kernels_region)
+       {
+         check_and_annotate_for_loop (nodeptr, NULL, info);
+         *walk_subtrees = 0;
+       }
+      break;
+
+    case LABEL_EXPR:
+      /* Possibly unstructured control flow.  Unless we perform further
+        analyses, we must assume that such control flow may enter the
+        current loop.  In this case, we must not parallelize the loop.  */
+      if (info->state >= as_in_kernels_loop
+         && TREE_USED (LABEL_EXPR_LABEL (node)))
+       do_not_annotate_loop_nest (info, as_invalid_control_flow, node);
+      break;
+
+    case GOTO_EXPR:
+      /* Possibly unstructured control flow.  Unless we perform further
+        analyses, we must assume that such control flow may leave the
+        current loop.  In this case, we must not parallelize the loop.  */
+      if (info->state >= as_in_kernels_loop)
+       do_not_annotate_loop_nest (info, as_invalid_control_flow, node);
+      break;
+
+    case BREAK_STMT:
+      /* A break statement.  Whether or not this is valid depends on the
+        enclosing context.  */
+      if (info->state >= as_in_kernels_loop && !info->break_ok)
+       do_not_annotate_loop (info, as_invalid_break, node);
+      break;
+
+    case RETURN_EXPR:
+      /* A return leaves the entire loop nest.  */
+      if (info->state >= as_in_kernels_loop)
+       do_not_annotate_loop_nest (info, as_invalid_return, node);
+      break;
+
+    case CALL_EXPR:
+      /* Direct function calls to functions marked as OpenACC routines are
+        allowed.  Reject indirect calls or calls to non-routines.  */
+      if (info->state >= as_in_kernels_loop)
+       {
+         tree fn = CALL_EXPR_FN (node), fn_decl = NULL_TREE;
+         if (fn != NULL_TREE && TREE_CODE (fn) == FUNCTION_DECL)
+           fn_decl = fn;
+         else if (fn != NULL_TREE && TREE_CODE (fn) == ADDR_EXPR)
+           {
+             tree fn_op = TREE_OPERAND (fn, 0);
+             if (fn_op != NULL_TREE && TREE_CODE (fn_op) == FUNCTION_DECL)
+               fn_decl = fn_op;
+           }
+         if (fn_decl == NULL_TREE)
+           do_not_annotate_loop_nest (info, as_invalid_call, node);
+         else if (!lookup_attribute ("oacc function",
+                                     DECL_ATTRIBUTES (fn_decl)))
+           do_not_annotate_loop_nest (info, as_invalid_call, node);
+       }
+      break;
+
+    case MODIFY_EXPR:
+      /* See if this assignment's LHS is one of the variables that must
+        not be modified in the loop body because they control termination
+        of the loop (or an enclosing loop in the nest).  */
+      if (info->state >= as_in_kernels_loop)
+       {
+         tree lhs = TREE_OPERAND (node, 0);
+         if (!is_local_var (lhs))
+           /* Early exit: This cannot be a variable we care about.  */
+           break;
+         /* Walk up the loop stack.  Invalidate the ones controlled by this
+            variable.  There may be several, if this variable is the common
+            iteration limit for several nested loops.  */
+         for (struct annotation_info *outer_loop = info; outer_loop != NULL;
+              outer_loop = outer_loop->next)
+           for (tree t = outer_loop->vars; t != NULL_TREE; t = TREE_CHAIN (t))
+             if (TREE_VALUE (t) == lhs)
+               {
+                 do_not_annotate_loop (outer_loop,
+                                       as_invalid_modification,
+                                       node);
+                 break;
+               }
+       }
+      break;
+
+    case SWITCH_STMT:
+      /* Needs special handling to allow break in the body.  */
+      if (info->state >= as_in_kernels_loop)
+       {
+         bool save_break_ok = info->break_ok;
+
+         walk_tree (&SWITCH_STMT_COND (node),
+                    annotate_loops_in_kernels_regions,
+                    (void *) info, NULL);
+         info->break_ok = true;
+         walk_tree (&SWITCH_STMT_BODY (node),
+                    annotate_loops_in_kernels_regions,
+                    (void *) info, NULL);
+         info->break_ok = save_break_ok;
+         *walk_subtrees = 0;
+       }
+      break;
+
+    case WHILE_STMT:
+      /* Needs special handling to allow break in the body.  */
+      if (info->state >= as_in_kernels_loop)
+       {
+         bool save_break_ok = info->break_ok;
+
+         walk_tree (&WHILE_COND (node), annotate_loops_in_kernels_regions,
+                    (void *) info, NULL);
+         info->break_ok = true;
+         walk_tree (&WHILE_BODY (node), annotate_loops_in_kernels_regions,
+                    (void *) info, NULL);
+         info->break_ok = save_break_ok;
+         *walk_subtrees = 0;
+       }
+      break;
+
+    case DO_STMT:
+      /* Needs special handling to allow break in the body.  */
+      if (info->state >= as_in_kernels_loop)
+       {
+         bool save_break_ok = info->break_ok;
+
+         walk_tree (&DO_COND (node), annotate_loops_in_kernels_regions,
+                    (void *) info, NULL);
+         info->break_ok = true;
+         walk_tree (&DO_BODY (node), annotate_loops_in_kernels_regions,
+                    (void *) info, NULL);
+         info->break_ok = save_break_ok;
+         *walk_subtrees = 0;
+       }
+      break;
+
+    case STATEMENT_LIST:
+      /* We iterate over these explicitly so that we can track the previous
+        statement in the chain.  It may be the initializer for a following
+        FOR_STMT node.  */
+      if (info->state >= as_in_kernels_region)
+       {
+         tree_stmt_iterator i = tsi_start (node);
+         tree_stmt_iterator prev, *prev_tsi = NULL;
+         while (!tsi_end_p (i))
+           {
+             tree *stmtptr = tsi_stmt_ptr (i);
+             if (TREE_CODE (*stmtptr) == FOR_STMT)
+               {
+                 check_and_annotate_for_loop (stmtptr, prev_tsi, info);
+                 *walk_subtrees = 0;
+               }
+             else
+               walk_tree (stmtptr, annotate_loops_in_kernels_regions,
+                          (void *) info, NULL);
+             prev = i;
+             prev_tsi = &prev;
+             tsi_next (&i);
+           }
+         *walk_subtrees = 0;
+       }
+      break;
+
+    default:
+      break;
+    }
+
+  return NULL_TREE;
+}
+
+/* Find for loops in OpenACC kernels regions that do not have OpenACC
+   annotations but look like they might benefit from automatic
+   parallelization.  Convert them from FOR_STMT to OMP_FOR nodes and
+   add the equivalent of "#pragma acc loop auto" annotations for them.
+   Assumes flag_openacc_kernels_annotate_loops is set.  */
+
+void
+c_oacc_annotate_loops_in_kernels_regions (tree decl,
+                                         tree (*unwrap_fn) (tree))
+{
+  struct annotation_info info
+    = { NULL_TREE, NULL_TREE, true, as_outer, NULL_TREE, NULL };
+  lang_specific_unwrap_initializer = unwrap_fn;
+  walk_tree (&DECL_SAVED_TREE (decl), annotate_loops_in_kernels_regions,
+            (void *) &info, NULL);
+}
+
 /* Used to merge map clause information in c_omp_adjust_map_clauses.  */
 struct map_clause
 {
index 3f8b72cdc0065afd10c75af8ea800919f2f1cab2..c595da46a9272a9490e500bce3ee1421d963934c 100644 (file)
@@ -1037,6 +1037,10 @@ Wold-style-definition
 C ObjC Var(warn_old_style_definition) Init(-1) Warning
 Warn if an old-style parameter definition is used.
 
+Wopenacc-kernels-annotate-loops
+C ObjC C++ ObjC++ Warning Var(warn_openacc_kernels_annotate_loops) Init(0)
+Warn about loops in OpenACC kernels regions that cannot be parallelized.
+
 Wopenmp-simd
 C C++ Var(warn_openmp_simd) Warning LangEnabledBy(C C++,Wall)
 Warn if a simd directive is overridden by the vectorizer cost model.
@@ -1873,6 +1877,10 @@ fopenacc-dim=
 C ObjC C++ ObjC++ LTO Joined Var(flag_openacc_dims)
 Specify default OpenACC compute dimensions.
 
+fopenacc-kernels-annotate-loops
+C ObjC C++ ObjC++ LTO Optimization Var(flag_openacc_kernels_annotate_loops) Init(1)
+Automatically parallelize unannotated loops in OpenACC kernels regions.
+
 fopenmp
 C ObjC C++ ObjC++ LTO Var(flag_openmp)
 Enable OpenMP (implies -frecursive in Fortran).
index f65e2ea9327f2c05c261fc96ac9f777ea907c3b5..e6fadbb1e890922deba3937c708956e84982eb63 100644 (file)
@@ -1,3 +1,8 @@
+2020-03-27  Sandra Loosemore  <sandra@codesourcery.com>
+
+       * c-decl.c (c_unwrap_for_init): New.
+       (finish_function): Call c_oacc_annotate_loops_in_kernels_regions.
+
 2019-09-10  Chung-Lin Tang  <cltang@codesourcery.com>
 
        * config/nvptx/nvptx.c (nvptx_expand_to_rtl_hook): New function
index 3ea4708c5075d9274601a0676f86a6900a9345b0..7d02074ce555e94807e558f2341913f13f0fa8de 100644 (file)
@@ -10149,6 +10149,29 @@ temp_pop_parm_decls (void)
   pop_scope ();
 }
 \f
+/* Function passed to c_oacc_annotate_loop_in_kernels_regions to do
+   language-specific unwrapping of an initializer expression.  */
+static tree
+c_unwrap_for_init (tree x)
+{
+  if (!x)
+    return NULL_TREE;
+
+  while (true)
+    switch (TREE_CODE (x))
+      {
+      case MODIFY_EXPR:
+      case VAR_DECL:
+       return x;
+
+      case DECL_EXPR:
+       x = TREE_OPERAND (x, 0);
+       break;
+
+      default:
+       return NULL_TREE;
+      }
+}
 
 /* Finish up a function declaration and compile that function
    all the way to assembler language output.  Then free the storage
@@ -10251,6 +10274,11 @@ finish_function (location_t end_loc)
   if (warn_unused_parameter)
     do_warn_unused_parameter (fndecl);
 
+  /* If requested, automatically annotate suitable loops in OpenACC kernels
+     regions with OpenACC loop annotations to allow auto-parallelization.  */
+  if (flag_openacc && flag_openacc_kernels_annotate_loops)
+    c_oacc_annotate_loops_in_kernels_regions (fndecl, c_unwrap_for_init);
+
   /* Store the end of the function, so that we get good line number
      info for the epilogue.  */
   cfun->function_end_locus = end_loc;
index 0e08348ed911243da508a4762e6832abab9521bc..566aaa4d2cfda21d830854c166e50deac868e1a5 100644 (file)
@@ -1,3 +1,8 @@
+2020-03-27  Sandra Loosemore  <sandra@codesourcery.com>
+
+       * decl.c (cp_unwrap_for_init): New.
+       (finish_function): Call c_oacc_annotate_loops_in_kernels_regions.
+
 2018-12-13  Cesar Philippidis  <cesar@codesourcery.com>
             Nathan Sidwell  <nathan@acm.org>
             Julian Brown  <julian@codesourcery.com>
index b81de8ef9349dedc5657c89d0b0c78014c298bc9..bc39d50c16c96f619e3aa85061575fb20219cf2d 100644 (file)
@@ -17223,6 +17223,45 @@ emit_coro_helper (tree helper)
   expand_or_defer_fn (helper);
 }
 
+
+/* Function passed to c_oacc_annotate_loop_in_kernels_regions to do
+   language-specific unwrapping of an initializer expression.  */
+static tree
+cp_unwrap_for_init (tree x)
+{
+  if (!x)
+    return NULL_TREE;
+
+  while (true)
+    switch (TREE_CODE (x))
+      {
+      case MODIFY_EXPR:
+      case VAR_DECL:
+       return x;
+
+      case CLEANUP_POINT_EXPR:
+       x = TREE_OPERAND (x, 0);
+       break;
+
+      case EXPR_STMT:
+       x = TREE_OPERAND (x, 0);
+       break;
+
+      case DECL_EXPR:
+       x = TREE_OPERAND (x, 0);
+       break;
+
+      case CONVERT_EXPR:
+       if (TREE_TYPE (x) != void_type_node)
+         return NULL_TREE;
+       x = TREE_OPERAND (x, 0);
+       break;
+
+      default:
+       return NULL_TREE;
+      }
+}
+
 /* Finish up a function declaration and compile that function
    all the way to assembler language output.  The free the storage
    for the function definition. INLINE_P is TRUE if we just
@@ -17527,6 +17566,11 @@ finish_function (bool inline_p)
       && !DECL_CLONED_FUNCTION_P (fndecl))
     do_warn_unused_parameter (fndecl);
 
+  /* If requested, automatically annotate suitable loops in OpenACC kernels
+     regions with OpenACC loop annotations to allow auto-parallelization.  */
+  if (flag_openacc && flag_openacc_kernels_annotate_loops)
+    c_oacc_annotate_loops_in_kernels_regions (fndecl, cp_unwrap_for_init);
+
   /* Genericize before inlining.  */
   if (!processing_template_decl
       && !DECL_IMMEDIATE_FUNCTION_P (fndecl)
index 22f7ef558a8c21d7798cf65b2606a94919fb147b..e89e6bc8abd109e772d26ce9456e8ec8577731a8 100644 (file)
@@ -364,7 +364,8 @@ Objective-C and Objective-C++ Dialects}.
 -Wmissing-include-dirs  -Wmissing-noreturn  -Wno-missing-profile @gol
 -Wno-multichar  -Wmultistatement-macros  -Wnonnull  -Wnonnull-compare @gol
 -Wnormalized=@r{[}none@r{|}id@r{|}nfc@r{|}nfkc@r{]} @gol
--Wnull-dereference  -Wno-odr  -Wopenmp-simd  @gol
+-Wnull-dereference  -Wno-odr @gol
+-Wopenacc-kernels-annotate-loops  -Wopenmp-simd @gol
 -Wno-overflow  -Woverlength-strings  -Wno-override-init-side-effects @gol
 -Wpacked  -Wno-packed-bitfield-compat  -Wpacked-not-aligned  -Wpadded @gol
 -Wparentheses  -Wno-pedantic-ms-format @gol
@@ -525,7 +526,8 @@ Objective-C and Objective-C++ Dialects}.
 -fmerge-constants  -fmodulo-sched  -fmodulo-sched-allow-regmoves @gol
 -fmove-loop-invariants  -fno-branch-count-reg @gol
 -fno-defer-pop  -fno-fp-int-builtin-inexact  -fno-function-cse @gol
--fno-guess-branch-probability  -fno-inline  -fno-math-errno  -fno-peephole @gol
+-fno-guess-branch-probability  -fno-inline  -fno-math-errno @gol
+-fno-openacc-kernels-annotate-loops  -fno-peephole @gol
 -fno-peephole2  -fno-printf-return-value  -fno-sched-interblock @gol
 -fno-sched-spec  -fno-signed-zeros @gol
 -fno-toplevel-reorder  -fno-trapping-math  -fno-zero-initialized-in-bss @gol
@@ -8753,6 +8755,13 @@ Do not warn about compile-time overflow in constant expressions.
 Warn about One Definition Rule violations during link-time optimization.
 Enabled by default.
 
+@item -Wopenacc-kernels-annotate-loops
+@opindex Wopenacc-kernels-annotate-loops
+@opindex Wno-Wopenacc-kernels-annotate-loops
+Warn about @code{for} (C/C++) or @code{DO} (Fortran) loops in OpenACC
+kernels regions that cannot be automatically annotated for
+parallelization with @option{-fopenacc-kernels-annotate-loops}.
+
 @item -Wopenmp-simd
 @opindex Wopenmp-simd
 @opindex Wno-openmp-simd
@@ -14430,6 +14439,27 @@ SIMD iterations.
 
 @end table
 
+@item -fno-openacc-kernels-annotate-loops
+@opindex fno-openacc-kernels-annotate-loops
+@opindex fopenacc-kernels-annotate-loops
+@cindex kernels regions, OpenACC
+Disable automatic parallelization of unannotated loops in OpenACC
+kernels regions.  The default is to attempt to add implicit
+@code{acc loop auto} annotations to loops in kernels regions if
+@option{-fopenacc} is enabled.
+
+Note that you can use @option{-Wopenacc-kernels-annotate-loops} to
+diagnose @code{for} loops that cannot be automatically annotated
+(@pxref{Warning Options}).  Reasons why automatic loop annotations
+cannot be applied include premature exits, calls to functions without
+an @code{openacc routine} annotation, or unstructured control flow in
+the loop body.  In C and C++, the loop variable initialization, end
+test, and increment expressions must additionally conform to
+restrictions similar to those for explicitly-annotated loops, and the
+loop variable must not be otherwise modified in the body of the loop.
+An explicit @code{acc loop} annotation disables automatic annotations
+on any nested or containing loops.
+
 @end table
 
 @node Instrumentation Options
index bd7ef85508a417ebb37ebedf8cd5551214591583..bce8b5b98a88a1d8df4e8fd7b40d393817dae973 100644 (file)
@@ -1,3 +1,46 @@
+2020-03-27  Sandra Loosemore  <sandra@codesourcery.com>
+
+       * c-c++-common/goacc/classify-kernels-unparallelized.c: Add
+       -fno-openacc-kernels-annotate-loops option.
+       * c-c++-common/goacc/classify-kernels.c: Likewise.
+       * c-c++-common/goacc/kernels-counter-var-redundant-load.c: Likewise.
+       * c-c++-common/goacc/kernels-counter-vars-function-scope.c: Likewise.
+       * c-c++-common/goacc/kernels-double-reduction-n.c: Likewise.
+       * c-c++-common/goacc/kernels-double-reduction.c: Likewise.
+       * c-c++-common/goacc/kernels-loop-2.c: Likewise.
+       * c-c++-common/goacc/kernels-loop-3.c: Likewise.
+       * c-c++-common/goacc/kernels-loop-data-2.c: Likewise.
+       * c-c++-common/goacc/kernels-loop-data-enter-exit-2.c: Likewise.
+       * c-c++-common/goacc/kernels-loop-data-enter-exit.c: Likewise.
+       * c-c++-common/goacc/kernels-loop-data-update.c: Likewise.
+       * c-c++-common/goacc/kernels-loop-data.c: Likewise.
+       * c-c++-common/goacc/kernels-loop-g.c: Likewise.
+       * c-c++-common/goacc/kernels-loop-mod-not-zero.c: Likewise.
+       * c-c++-common/goacc/kernels-loop-n.c: Likewise.
+       * c-c++-common/goacc/kernels-loop-nest.c: Likewise.
+       * c-c++-common/goacc/kernels-loop.c: Likewise.
+       * c-c++-common/goacc/kernels-one-counter-var.c: Likewise.
+       * c-c++-common/goacc/kernels-parallel-loop-data-enter-exit.c:
+       Likewise.
+       * c-c++-common/goacc/kernels-reduction.c: Likewise.
+       * c-c++-common/goacc/kernels-loop-annotation-1.c: New.
+       * c-c++-common/goacc/kernels-loop-annotation-2.c: New.
+       * c-c++-common/goacc/kernels-loop-annotation-3.c: New.
+       * c-c++-common/goacc/kernels-loop-annotation-4.c: New.
+       * c-c++-common/goacc/kernels-loop-annotation-5.c: New.
+       * c-c++-common/goacc/kernels-loop-annotation-6.c: New.
+       * c-c++-common/goacc/kernels-loop-annotation-7.c: New.
+       * c-c++-common/goacc/kernels-loop-annotation-8.c: New.
+       * c-c++-common/goacc/kernels-loop-annotation-9.c: New.
+       * c-c++-common/goacc/kernels-loop-annotation-10.c: New.
+       * c-c++-common/goacc/kernels-loop-annotation-11.c: New.
+       * c-c++-common/goacc/kernels-loop-annotation-12.c: New.
+       * c-c++-common/goacc/kernels-loop-annotation-13.c: New.
+       * c-c++-common/goacc/kernels-loop-annotation-14.c: New.
+       * c-c++-common/goacc/kernels-loop-annotation-15.c: New.
+       * c-c++-common/goacc/kernels-loop-annotation-16.c: New.
+       * c-c++-common/goacc/kernels-loop-annotation-17.c: New.
+
 2020-03-19  Kwok Cheung Yeung  <kcy@codesourcery.com>
 
        * c-c++-common/goacc/routine-level-of-parallelism-2.c (g_1): Add
index 32af5330e4488fb7f2db67dd6675ed3b5f0213e0..6fe3c0e635e8419221cf7bf2a230fae9cfb23fe4 100644 (file)
@@ -2,6 +2,7 @@
    OpenACC 'kernels'.  */
 
 /* { dg-additional-options "-O2" }
+   { dg-additional-options "-fno-openacc-kernels-annotate-loops" }
    { dg-additional-options "-fopt-info-optimized-omp" }
    { dg-additional-options "-fdump-tree-ompexp" }
    { dg-additional-options "-fdump-tree-parloops1-all" }
index 93e64066b92b04a443ad95db78834d0afbb1223a..f61f54157c23dc1b67b61a709121b6a59c34bdd9 100644 (file)
@@ -1,7 +1,8 @@
 /* Check offloaded function's attributes and classification for OpenACC
-   'kernels'.  */
+   'kernels' (parloops version).  */
 
 /* { dg-additional-options "-O2" }
+   { dg-additional-options "-fno-openacc-kernels-annotate-loops" }
    { dg-additional-options "-fopt-info-optimized-omp" }
    { dg-additional-options "-fdump-tree-ompexp" }
    { dg-additional-options "-fdump-tree-parloops1-all" }
index 42f67e002d4fb2c0d2e24f178fd54fbaf50ea7da..22d4a30fc805d054beab7b1eddd6db7cc6022189 100644 (file)
@@ -1,6 +1,7 @@
 /* { dg-additional-options "--param=openacc-kernels=parloops" } as this is
    specifically testing "parloops" handling.  */
 /* { dg-additional-options "-O2" } */
+/* { dg-additional-options "-fno-openacc-kernels-annotate-loops" } */
 /* { dg-additional-options "-fdump-tree-dom3" } */
 
 #include <stdlib.h>
index fc4e871d9b79cbb2e0791b43e99c90338086bf35..e14ea2c4c72660aea3d801b7782e3df706276743 100644 (file)
@@ -1,6 +1,7 @@
 /* { dg-additional-options "--param=openacc-kernels=parloops" } as this is
    specifically testing "parloops" handling.  */
 /* { dg-additional-options "-O2" } */
+/* { dg-additional-options "-fno-openacc-kernels-annotate-loops" } */
 /* { dg-additional-options "-fdump-tree-parloops1-all" } */
 /* { dg-additional-options "-fdump-tree-optimized" } */
 
index 4a3cdd764988c247207610014aa55f502d3b7bdf..329de2c4261071f50306243610893d9f70a9d3dc 100644 (file)
@@ -1,6 +1,7 @@
 /* { dg-additional-options "--param=openacc-kernels=parloops" } as this is
    specifically testing "parloops" handling.  */
 /* { dg-additional-options "-O2" } */
+/* { dg-additional-options "-fno-openacc-kernels-annotate-loops" } */
 /* { dg-additional-options "-fopt-info-optimized-omp" } */
 /* { dg-additional-options "-fdump-tree-parloops1-all" } */
 /* { dg-additional-options "-fdump-tree-optimized" } */
index 5b586e734d7f5756dcfb2d2cdfa6edca8e6a1830..278bbc71e2a32272c1d90fcc195468bd4b3852be 100644 (file)
@@ -1,6 +1,7 @@
 /* { dg-additional-options "--param=openacc-kernels=parloops" } as this is
    specifically testing "parloops" handling.  */
 /* { dg-additional-options "-O2" } */
+/* { dg-additional-options "-fno-openacc-kernels-annotate-loops" } */
 /* { dg-additional-options "-fopt-info-optimized-omp" } */
 /* { dg-additional-options "-fdump-tree-parloops1-all" } */
 /* { dg-additional-options "-fdump-tree-optimized" } */
index 9c156f469d18cb53959bef56b14998dea93606b7..eb95dddedd56f7ace4defd47777c85e29e54508b 100644 (file)
@@ -1,6 +1,7 @@
 /* { dg-additional-options "--param=openacc-kernels=parloops" } as this is
    specifically testing "parloops" handling.  */
 /* { dg-additional-options "-O2" } */
+/* { dg-additional-options "-fno-openacc-kernels-annotate-loops" } */
 /* { dg-additional-options "-fdump-tree-parloops1-all" } */
 /* { dg-additional-options "-fdump-tree-optimized" } */
 
index f1be00528b486f5f83e20867573edcc1ce07c65e..c1aae7ffc19bb64d5a3507987ff95fc81bb540eb 100644 (file)
@@ -1,6 +1,7 @@
 /* { dg-additional-options "--param=openacc-kernels=parloops" } as this is
    specifically testing "parloops" handling.  */
 /* { dg-additional-options "-O2" } */
+/* { dg-additional-options "-fno-openacc-kernels-annotate-loops" } */
 /* { dg-additional-options "-fdump-tree-parloops1-all" } */
 /* { dg-additional-options "-fdump-tree-optimized" } */
 
diff --git a/gcc/testsuite/c-c++-common/goacc/kernels-loop-annotation-1.c b/gcc/testsuite/c-c++-common/goacc/kernels-loop-annotation-1.c
new file mode 100644 (file)
index 0000000..c7b5ac8
--- /dev/null
@@ -0,0 +1,26 @@
+/* { dg-additional-options "-fopenacc -fopenacc-kernels-annotate-loops" } */
+/* { dg-additional-options "-Wopenacc-kernels-annotate-loops" } */
+/* { dg-additional-options "-fdump-tree-original" } */
+/* { dg-do compile } */
+
+/* Test that all loops in the nest are annotated.  */
+
+void f (float a[16][16], float b[16][16], float c[16][16])
+{
+  int i, j, k;
+
+#pragma acc kernels copyin(a[0:16][0:16], b[0:16][0:16]) copyout(c[0:16][0:16])
+  {
+    for (i = 0; i < 16; i++) {
+      for (j = 0; j < 16; j++) {
+       float t = 0;
+       for (k = 0; k < 16; k++)
+         t += a[i][k] * b[k][j];
+       c[i][j] = t;
+      }
+    }
+  }
+
+}
+
+/* { dg-final { scan-tree-dump-times "acc loop auto" 3 "original" } } */
diff --git a/gcc/testsuite/c-c++-common/goacc/kernels-loop-annotation-10.c b/gcc/testsuite/c-c++-common/goacc/kernels-loop-annotation-10.c
new file mode 100644 (file)
index 0000000..58b41d2
--- /dev/null
@@ -0,0 +1,32 @@
+/* { dg-additional-options "-fopenacc -fopenacc-kernels-annotate-loops" } */
+/* { dg-additional-options "-Wopenacc-kernels-annotate-loops" } */
+/* { dg-additional-options "-fdump-tree-original" } */
+/* { dg-do compile } */
+
+/* Test that a loop with a random goto in the body can't be annotated.  */
+
+#define n 16
+
+float f (float *a, float *b)
+{
+  float t = 0;
+  int i;
+
+#pragma acc kernels
+  {
+    for (i = 0; i < n; i++)    /* { dg-warning "loop cannot be annotated" } */
+      {
+       if (a[i] < 0)
+         {
+           t = 0;
+           goto bad;
+         }
+       t += a[i] * b[i];
+      }
+  bad:
+    ;
+  }
+  return t;
+}
+
+/* { dg-final { scan-tree-dump-times "acc loop auto" 0 "original" } } */
diff --git a/gcc/testsuite/c-c++-common/goacc/kernels-loop-annotation-11.c b/gcc/testsuite/c-c++-common/goacc/kernels-loop-annotation-11.c
new file mode 100644 (file)
index 0000000..e9d2ef4
--- /dev/null
@@ -0,0 +1,27 @@
+/* { dg-additional-options "-fopenacc -fopenacc-kernels-annotate-loops" } */
+/* { dg-additional-options "-Wopenacc-kernels-annotate-loops" } */
+/* { dg-additional-options "-fdump-tree-original" } */
+/* { dg-do compile } */
+
+/* Test that a loop with a random label in the body triggers a warning.  */
+
+#define n 16
+
+float f (float *a, float *b)
+{
+  float t = 0;
+  int i = n - 1;
+
+#pragma acc kernels
+  {
+    goto spaghetti;
+    for (i = 0; i < n; i++)    /* { dg-warning "loop cannot be annotated" } */
+      {
+      spaghetti:
+       t += a[i] * b[i];
+      }
+  }
+  return t;
+}
+
+/* { dg-final { scan-tree-dump-times "acc loop auto" 0 "original" } } */
diff --git a/gcc/testsuite/c-c++-common/goacc/kernels-loop-annotation-12.c b/gcc/testsuite/c-c++-common/goacc/kernels-loop-annotation-12.c
new file mode 100644 (file)
index 0000000..ba408bc
--- /dev/null
@@ -0,0 +1,28 @@
+/* { dg-additional-options "-fopenacc -fopenacc-kernels-annotate-loops" } */
+/* { dg-additional-options "-Wopenacc-kernels-annotate-loops" } */
+/* { dg-additional-options "-fdump-tree-original" } */
+/* { dg-do compile } */
+
+/* Test that in a situation with nested loops, a problem that prevents
+   annotation of the inner loop only still allows the outer loop to be
+   annotated.  */
+
+float f (float *a, float *b, int n)
+{
+  float t = 0;
+
+#pragma acc kernels
+  {
+    for (int i = 0; i < n; i++)
+      for (int j = 0; j <= i; j++)  /* { dg-warning "loop cannot be annotated" } */
+       {
+         if (a[i] < 0 || b[j] < 0)
+           j = i;
+         else
+           t += a[i] * b[j];
+       }
+  }
+  return t;
+}
+
+/* { dg-final { scan-tree-dump-times "acc loop auto" 1 "original" } } */
diff --git a/gcc/testsuite/c-c++-common/goacc/kernels-loop-annotation-13.c b/gcc/testsuite/c-c++-common/goacc/kernels-loop-annotation-13.c
new file mode 100644 (file)
index 0000000..64433e8
--- /dev/null
@@ -0,0 +1,27 @@
+/* { dg-additional-options "-fopenacc -fopenacc-kernels-annotate-loops" } */
+/* { dg-additional-options "-Wopenacc-kernels-annotate-loops" } */
+/* { dg-additional-options "-fdump-tree-original" } */
+/* { dg-do compile } */
+
+/* Test that in a situation with nested loops, a problem that prevents
+   annotation of the outer loop only still allows the inner loop to be
+   annotated.  */
+
+float f (float *a, float *b, int n)
+{
+  float t = 0;
+
+#pragma acc kernels
+  {
+    for (int i = 0; i < n; i++)          /* { dg-warning "loop cannot be annotated" } */
+      {
+       if (a[i] < 0)
+         n = i;
+       for (int j = 0; j <= i; j++)
+         t += a[i] * b[j];
+      }
+  }
+  return t;
+}
+
+/* { dg-final { scan-tree-dump-times "acc loop auto" 1 "original" } } */
diff --git a/gcc/testsuite/c-c++-common/goacc/kernels-loop-annotation-14.c b/gcc/testsuite/c-c++-common/goacc/kernels-loop-annotation-14.c
new file mode 100644 (file)
index 0000000..379e6ba
--- /dev/null
@@ -0,0 +1,22 @@
+/* { dg-additional-options "-fopenacc -fopenacc-kernels-annotate-loops" } */
+/* { dg-additional-options "-Wopenacc-kernels-annotate-loops" } */
+/* { dg-additional-options "-fdump-tree-original" } */
+/* { dg-do compile } */
+
+/* Test that an explicit annotation on an outer loop suppresses annotation
+   of inner loops, and produces a diagnostic.  */
+
+void f (float *a, float *b)
+{
+  float t = 0;
+
+#pragma acc kernels
+  {
+#pragma acc loop seq
+    for (int l = 0; l < 20; l++)
+      for (int m = 0; m < 20; m++)     /* { dg-warning "loop cannot be annotated" } */
+        b[m] = a[m];
+  }
+}
+
+/* { dg-final { scan-tree-dump-times "acc loop auto" 0 "original" } } */
diff --git a/gcc/testsuite/c-c++-common/goacc/kernels-loop-annotation-15.c b/gcc/testsuite/c-c++-common/goacc/kernels-loop-annotation-15.c
new file mode 100644 (file)
index 0000000..9a2a7ca
--- /dev/null
@@ -0,0 +1,22 @@
+/* { dg-additional-options "-fopenacc -fopenacc-kernels-annotate-loops" } */
+/* { dg-additional-options "-Wopenacc-kernels-annotate-loops" } */
+/* { dg-additional-options "-fdump-tree-original" } */
+/* { dg-do compile } */
+
+/* Test that an explicit annotation on an inner loop suppresses annotation
+   of outer loops, and produces a diagnostic.  */
+
+void f (float *a, float *b)
+{
+  float t = 0;
+
+#pragma acc kernels
+  {
+    for (int l = 0; l < 20; l++)       /* { dg-warning "loop cannot be annotated" } */
+#pragma acc loop seq
+      for (int m = 0; m < 20; m++)
+        b[m] = a[m];
+  }
+}
+
+/* { dg-final { scan-tree-dump-times "acc loop auto" 0 "original" } } */
diff --git a/gcc/testsuite/c-c++-common/goacc/kernels-loop-annotation-16.c b/gcc/testsuite/c-c++-common/goacc/kernels-loop-annotation-16.c
new file mode 100644 (file)
index 0000000..075f897
--- /dev/null
@@ -0,0 +1,26 @@
+/* { dg-additional-options "-fopenacc -fopenacc-kernels-annotate-loops" } */
+/* { dg-additional-options "-Wopenacc-kernels-annotate-loops" } */
+/* { dg-additional-options "-fdump-tree-original" } */
+/* { dg-do compile } */
+
+/* Test that a loop with a modification of the loop variable in the
+   body cannot be annotated.  */
+
+float f (float *a, float *b, int n)
+{
+  float t = 0;
+
+#pragma acc kernels
+  {
+    for (int i = 0; i < n; i++)        /* { dg-warning "loop cannot be annotated" } */
+      {
+       if (a[i] < 0 || b[i] < 0)
+         i = n;
+       else
+         t += a[i] * b[i];
+      }
+  }
+  return t;
+}
+
+/* { dg-final { scan-tree-dump-times "acc loop auto" 0 "original" } } */
diff --git a/gcc/testsuite/c-c++-common/goacc/kernels-loop-annotation-17.c b/gcc/testsuite/c-c++-common/goacc/kernels-loop-annotation-17.c
new file mode 100644 (file)
index 0000000..5076789
--- /dev/null
@@ -0,0 +1,26 @@
+/* { dg-additional-options "-fopenacc -fopenacc-kernels-annotate-loops" } */
+/* { dg-additional-options "-Wopenacc-kernels-annotate-loops" } */
+/* { dg-additional-options "-fdump-tree-original" } */
+/* { dg-do compile } */
+
+/* Test that a loop with a modification of the loop iteration count
+   variable in the body cannot be annotated.  */
+
+float f (float *a, float *b, int n)
+{
+  float t = 0;
+
+#pragma acc kernels
+  {
+    for (int i = 0; i < n; i++)        /* { dg-warning "loop cannot be annotated" } */
+      {
+       if (a[i] < 0 || b[i] < 0)
+         n = i;
+       else
+         t += a[i] * b[i];
+      }
+  }
+  return t;
+}
+
+/* { dg-final { scan-tree-dump-times "acc loop auto" 0 "original" } } */
diff --git a/gcc/testsuite/c-c++-common/goacc/kernels-loop-annotation-2.c b/gcc/testsuite/c-c++-common/goacc/kernels-loop-annotation-2.c
new file mode 100644 (file)
index 0000000..9e0a946
--- /dev/null
@@ -0,0 +1,21 @@
+/* { dg-additional-options "-fopenacc -fopenacc-kernels-annotate-loops" } */
+/* { dg-additional-options "-Wopenacc-kernels-annotate-loops" } */
+/* { dg-additional-options "-fdump-tree-original" } */
+/* { dg-do compile } */
+
+/* Test that a loop with a variable bound can be annotated.  */
+
+float f (float *a, float *b, int n)
+{
+  float t = 0;
+  int i;
+
+#pragma acc kernels
+  {
+    for (i = 0; i < n; i++)
+      t += a[i] * b[i];
+  }
+  return t;
+}
+
+/* { dg-final { scan-tree-dump-times "acc loop auto" 1 "original" } } */
diff --git a/gcc/testsuite/c-c++-common/goacc/kernels-loop-annotation-3.c b/gcc/testsuite/c-c++-common/goacc/kernels-loop-annotation-3.c
new file mode 100644 (file)
index 0000000..f60070e
--- /dev/null
@@ -0,0 +1,24 @@
+/* { dg-additional-options "-fopenacc -fopenacc-kernels-annotate-loops" } */
+/* { dg-additional-options "-Wopenacc-kernels-annotate-loops" } */
+/* { dg-additional-options "-fdump-tree-original" } */
+/* { dg-do compile } */
+
+/* Test that a loop with a conditional in the body can be annotated.  */
+
+#define n 16
+
+float f (float *a, float *b)
+{
+  float t = 0;
+  int i;
+
+#pragma acc kernels
+  {
+    for (i = 0; i < n; i++)
+      if (a[i] > 0 && b[i] > 0)
+       t += a[i] * b[i];
+  }
+  return t;
+}
+
+/* { dg-final { scan-tree-dump-times "acc loop auto" 1 "original" } } */
diff --git a/gcc/testsuite/c-c++-common/goacc/kernels-loop-annotation-4.c b/gcc/testsuite/c-c++-common/goacc/kernels-loop-annotation-4.c
new file mode 100644 (file)
index 0000000..949871c
--- /dev/null
@@ -0,0 +1,34 @@
+/* { dg-additional-options "-fopenacc -fopenacc-kernels-annotate-loops" } */
+/* { dg-additional-options "-Wopenacc-kernels-annotate-loops" } */
+/* { dg-additional-options "-fdump-tree-original" } */
+/* { dg-do compile } */
+
+/* Test that a loop with a switch and break in the body can be annotated.  */
+
+#define n 16
+
+float f (float *a, float *b, int state)
+{
+  float t = 0;
+  int i;
+
+#pragma acc kernels
+  {
+    for (i = 0; i < n; i++)
+      switch (state)
+       {
+       case 0:
+       default:
+         t += a[i] * b[i];
+         break;
+
+       case 1:
+         if (a[i] > 0 && b[i] > 0)
+           t += a[i] * b[i];
+         break;
+       }
+  }
+  return t;
+}
+
+/* { dg-final { scan-tree-dump-times "acc loop auto" 1 "original" } } */
diff --git a/gcc/testsuite/c-c++-common/goacc/kernels-loop-annotation-5.c b/gcc/testsuite/c-c++-common/goacc/kernels-loop-annotation-5.c
new file mode 100644 (file)
index 0000000..03dfe8f
--- /dev/null
@@ -0,0 +1,27 @@
+/* { dg-additional-options "-fopenacc -fopenacc-kernels-annotate-loops" } */
+/* { dg-additional-options "-Wopenacc-kernels-annotate-loops" } */
+/* { dg-additional-options "-fdump-tree-original" } */
+/* { dg-do compile } */
+
+/* Test that a loop with a continue statement in the body can be annotated.  */
+
+#define n 16
+
+float f (float *a, float *b)
+{
+  float t = 0;
+  int i;
+
+#pragma acc kernels
+  {
+    for (i = 0; i < n; i++)
+      {
+       if (a[i] < 0 || b[i] < 0)
+         continue;
+       t += a[i] * b[i];
+      }
+  }
+  return t;
+}
+
+/* { dg-final { scan-tree-dump-times "acc loop auto" 1 "original" } } */
diff --git a/gcc/testsuite/c-c++-common/goacc/kernels-loop-annotation-6.c b/gcc/testsuite/c-c++-common/goacc/kernels-loop-annotation-6.c
new file mode 100644 (file)
index 0000000..ede6b3c
--- /dev/null
@@ -0,0 +1,27 @@
+/* { dg-additional-options "-fopenacc -fopenacc-kernels-annotate-loops" } */
+/* { dg-additional-options "-Wopenacc-kernels-annotate-loops" } */
+/* { dg-additional-options "-fdump-tree-original" } */
+/* { dg-do compile } */
+
+/* Test that a loop with a break statement in the body cannot be annotated.  */
+
+#define n 16
+
+float f (float *a, float *b)
+{
+  float t = 0;
+  int i;
+
+#pragma acc kernels
+  {
+    for (i = 0; i < n; i++)    /* { dg-warning "loop cannot be annotated" } */
+      {
+       if (a[i] < 0 || b[i] < 0)
+         break;
+       t += a[i] * b[i];
+      }
+  }
+  return t;
+}
+
+/* { dg-final { scan-tree-dump-times "acc loop auto" 0 "original" } } */
diff --git a/gcc/testsuite/c-c++-common/goacc/kernels-loop-annotation-7.c b/gcc/testsuite/c-c++-common/goacc/kernels-loop-annotation-7.c
new file mode 100644 (file)
index 0000000..20ee299
--- /dev/null
@@ -0,0 +1,26 @@
+/* { dg-additional-options "-fopenacc -fopenacc-kernels-annotate-loops" } */
+/* { dg-additional-options "-Wopenacc-kernels-annotate-loops" } */
+/* { dg-additional-options "-fdump-tree-original" } */
+/* { dg-do compile } */
+
+/* Test that a loop with a random function call in the body cannot be
+   annotated.  */
+
+extern float g (float);
+
+#define n 16
+
+float f (float *a, float *b)
+{
+  float t = 0;
+  int i;
+
+#pragma acc kernels
+  {
+    for (i = 0; i < n; i++)    /* { dg-warning "loop cannot be annotated" } */
+      t += g (a[i] * b[i]);
+  }
+  return t;
+}
+
+/* { dg-final { scan-tree-dump-times "acc loop auto" 0 "original" } } */
diff --git a/gcc/testsuite/c-c++-common/goacc/kernels-loop-annotation-8.c b/gcc/testsuite/c-c++-common/goacc/kernels-loop-annotation-8.c
new file mode 100644 (file)
index 0000000..796f048
--- /dev/null
@@ -0,0 +1,27 @@
+/* { dg-additional-options "-fopenacc -fopenacc-kernels-annotate-loops" } */
+/* { dg-additional-options "-Wopenacc-kernels-annotate-loops" } */
+/* { dg-additional-options "-fdump-tree-original" } */
+/* { dg-do compile } */
+
+/* Test that a loop with an openacc function call in the body can be
+   annotated.  */
+
+#pragma acc routine worker
+extern float g (float);
+
+#define n 16
+
+float f (float *a, float *b)
+{
+  float t = 0;
+  int i;
+
+#pragma acc kernels
+  {
+    for (i = 0; i < n; i++)
+      t += g (a[i] * b[i]);
+  }
+  return t;
+}
+
+/* { dg-final { scan-tree-dump-times "acc loop auto" 1 "original" } } */
diff --git a/gcc/testsuite/c-c++-common/goacc/kernels-loop-annotation-9.c b/gcc/testsuite/c-c++-common/goacc/kernels-loop-annotation-9.c
new file mode 100644 (file)
index 0000000..048f1b0
--- /dev/null
@@ -0,0 +1,26 @@
+/* { dg-additional-options "-fopenacc -fopenacc-kernels-annotate-loops" } */
+/* { dg-additional-options "-Wopenacc-kernels-annotate-loops" } */
+/* { dg-additional-options "-fdump-tree-original" } */
+/* { dg-do compile } */
+
+/* Test that a kernels loop with a return in the body triggers a hard
+   error.  */
+
+#define n 16
+
+float f (float *a, float *b)
+{
+  float t = 0;
+  int i;
+
+#pragma acc kernels
+  {
+    for (i = 0; i < n; i++)
+      {
+       if (a[i] < 0 || b[i] < 0)
+         return 0.0;   /* { dg-error "invalid branch" } */
+       t += a[i] * b[i];
+      }
+  }
+  return t;
+}
index fdcfe2ca798cc4c676b5165dda768f9a37cd949c..b9e0458eab1a3fc0aa0d54fcb72601f18756e8e4 100644 (file)
@@ -1,6 +1,7 @@
 /* { dg-additional-options "--param=openacc-kernels=parloops" } as this is
    specifically testing "parloops" handling.  */
 /* { dg-additional-options "-O2" } */
+/* { dg-additional-options "-fno-openacc-kernels-annotate-loops" } */
 /* { dg-additional-options "-fdump-tree-parloops1-all" } */
 /* { dg-additional-options "-fdump-tree-optimized" } */
 
index b4a2a72a95011209f340eea89445531b31d0e75f..9a88e8e0d393bb7829a6e3f527c3398078d76a0a 100644 (file)
@@ -1,6 +1,7 @@
 /* { dg-additional-options "--param=openacc-kernels=parloops" } as this is
    specifically testing "parloops" handling.  */
 /* { dg-additional-options "-O2" } */
+/* { dg-additional-options "-fno-openacc-kernels-annotate-loops" } */
 /* { dg-additional-options "-fdump-tree-parloops1-all" } */
 /* { dg-additional-options "-fdump-tree-optimized" } */
 
index 0eb0f0719de1abca6439cbc7b82fa763baa50d86..0a018820ca17bfd82db9f44533d0d6b01cf5e8d4 100644 (file)
@@ -1,6 +1,7 @@
 /* { dg-additional-options "--param=openacc-kernels=parloops" } as this is
    specifically testing "parloops" handling.  */
 /* { dg-additional-options "-O2" } */
+/* { dg-additional-options "-fno-openacc-kernels-annotate-loops" } */
 /* { dg-additional-options "-fdump-tree-parloops1-all" } */
 /* { dg-additional-options "-fdump-tree-optimized" } */
 
index 6b2a7784444afe5070d5bb760473d981852d366a..4821cb9675ed1147bd37fe8b4a512bdb90b6124e 100644 (file)
@@ -1,6 +1,7 @@
 /* { dg-additional-options "--param=openacc-kernels=parloops" } as this is
    specifically testing "parloops" handling.  */
 /* { dg-additional-options "-O2" } */
+/* { dg-additional-options "-fno-openacc-kernels-annotate-loops" } */
 /* { dg-additional-options "-fdump-tree-parloops1-all" } */
 /* { dg-additional-options "-fdump-tree-optimized" } */
 
index d0ee6ffb1e5ead3886a12a309feeaccfbe74cad3..d650bfa97181ae5422d8a2bdcb468c4667e7dc72 100644 (file)
@@ -1,6 +1,7 @@
 /* { dg-additional-options "--param=openacc-kernels=parloops" } as this is
    specifically testing "parloops" handling.  */
 /* { dg-additional-options "-O2" } */
+/* { dg-additional-options "-fno-openacc-kernels-annotate-loops" } */
 /* { dg-additional-options "-fdump-tree-parloops1-all" } */
 /* { dg-additional-options "-fdump-tree-optimized" } */
 
index 4400a33152b0b04f97f5a4fd27d065cf7e03fe78..8aaddebe366b731a2c1a69c2ed4cba2d53e3b2b4 100644 (file)
@@ -2,6 +2,7 @@
    specifically testing "parloops" handling.  */
 /* { dg-additional-options "-O2" } */
 /* { dg-additional-options "-g" } */
+/* { dg-additional-options "-fno-openacc-kernels-annotate-loops" } */
 /* { dg-additional-options "-fdump-tree-parloops1-all" } */
 /* { dg-additional-options "-fdump-tree-optimized" } */
 
index b3812b791314fc204d4485bfc5556ff055672438..bcc2555837343eb5d7fbf95cd7ddf26a48eba386 100644 (file)
@@ -1,6 +1,7 @@
 /* { dg-additional-options "--param=openacc-kernels=parloops" } as this is
    specifically testing "parloops" handling.  */
 /* { dg-additional-options "-O2" } */
+/* { dg-additional-options "-fno-openacc-kernels-annotate-loops" } */
 /* { dg-additional-options "-fdump-tree-parloops1-all" } */
 /* { dg-additional-options "-fdump-tree-optimized" } */
 
index 5708014636d204d9f6364b21d2bfe28cd1443244..b9ffc11bb1c281fb641f08eea0d3d69d23b383d7 100644 (file)
@@ -1,6 +1,7 @@
 /* { dg-additional-options "--param=openacc-kernels=parloops" } as this is
    specifically testing "parloops" handling.  */
 /* { dg-additional-options "-O2" } */
+/* { dg-additional-options "-fno-openacc-kernels-annotate-loops" } */
 /* { dg-additional-options "-fdump-tree-parloops1-all" } */
 /* { dg-additional-options "-fdump-tree-optimized" } */
 
index f6d9391a5490c4e2fd42b7e6335b449f975aabb0..a7cec7fab9ede4116b8254ac94a81550fd2f6364 100644 (file)
@@ -1,6 +1,7 @@
 /* { dg-additional-options "--param=openacc-kernels=parloops" } as this is
    specifically testing "parloops" handling.  */
 /* { dg-additional-options "-O2" } */
+/* { dg-additional-options "-fno-openacc-kernels-annotate-loops" } */
 /* { dg-additional-options "-fdump-tree-parloops1-all" } */
 /* { dg-additional-options "-fdump-tree-optimized" } */
 
index 5ef928ff2e3eb60698fa6607e689bc72cbda1906..954902edbd993bd27f8907b82737cba51a47a70e 100644 (file)
@@ -1,6 +1,7 @@
 /* { dg-additional-options "--param=openacc-kernels=parloops" } as this is
    specifically testing "parloops" handling.  */
 /* { dg-additional-options "-O2" } */
+/* { dg-additional-options "-fno-openacc-kernels-annotate-loops" } */
 /* { dg-additional-options "-fdump-tree-parloops1-all" } */
 /* { dg-additional-options "-fdump-tree-optimized" } */
 
index 928125684668f0c450215ccc4febd3ff5adcb61c..e2f27af76ed5c5f7b57dc9a54c5336167a69a1fe 100644 (file)
@@ -1,6 +1,7 @@
 /* { dg-additional-options "--param=openacc-kernels=parloops" } as this is
    specifically testing "parloops" handling.  */
 /* { dg-additional-options "-O2" } */
+/* { dg-additional-options "-fno-openacc-kernels-annotate-loops" } */
 /* { dg-additional-options "-fdump-tree-parloops1-all" } */
 /* { dg-additional-options "-fdump-tree-optimized" } */
 
index 8f57cd6b8592c890392956c7f533a2417bf52a37..305974b88cedaccd9086073821e5fc68178043a6 100644 (file)
@@ -1,6 +1,7 @@
 /* { dg-additional-options "--param=openacc-kernels=parloops" } as this is
    specifically testing "parloops" handling.  */
 /* { dg-additional-options "-O2" } */
+/* { dg-additional-options "-fno-openacc-kernels-annotate-loops" } */
 /* { dg-additional-options "-fdump-tree-parloops1-all" } */
 /* { dg-additional-options "-fdump-tree-optimized" } */
 
index 419b08e5f2686b5662a65751c0d3b578c6b8f580..1449f7a066d4def029ab1234d4bfc93c832800d7 100644 (file)
@@ -1,6 +1,7 @@
 /* { dg-additional-options "--param=openacc-kernels=parloops" } as this is
    specifically testing "parloops" handling.  */
 /* { dg-additional-options "-O2" } */
+/* { dg-additional-options "-fno-openacc-kernels-annotate-loops" } */
 /* { dg-additional-options "-fdump-tree-parloops1-all" } */
 /* { dg-additional-options "-fdump-tree-optimized" } */