openmp: Ensure proper diagnostics for -> in map/to/from clauses [PR104532]
authorJakub Jelinek <jakub@redhat.com>
Thu, 17 Feb 2022 09:29:06 +0000 (10:29 +0100)
committerJakub Jelinek <jakub@redhat.com>
Thu, 17 Feb 2022 09:29:06 +0000 (10:29 +0100)
The following patch uses the functions normal CPP_DEREF parsing uses,
i.e. convert_lvalue_to_rvalue and build_indirect_ref, instead of
blindly calling build_simple_mem_ref, so that if the variable does not
have correct type, we properly diagnose it instead of ICEing on it.

2022-02-17  Jakub Jelinek  <jakub@redhat.com>

PR c/104532
* c-parser.cc (c_parser_omp_variable_list): For CPP_DEREF, use
convert_lvalue_to_rvalue and build_indirect_ref instead of
build_simple_mem_ref.

* gcc.dg/gomp/pr104532.c: New test.

gcc/c/c-parser.cc
gcc/testsuite/gcc.dg/gomp/pr104532.c [new file with mode: 0644]

index 3b1d2d4..84deac0 100644 (file)
@@ -13145,7 +13145,16 @@ c_parser_omp_variable_list (c_parser *parser,
                {
                  location_t op_loc = c_parser_peek_token (parser)->location;
                  if (c_parser_next_token_is (parser, CPP_DEREF))
-                   t = build_simple_mem_ref (t);
+                   {
+                     c_expr t_expr;
+                     t_expr.value = t;
+                     t_expr.original_code = ERROR_MARK;
+                     t_expr.original_type = NULL;
+                     set_c_expr_source_range (&t_expr, op_loc, op_loc);
+                     t_expr = convert_lvalue_to_rvalue (op_loc, t_expr,
+                                                        true, false);
+                     t = build_indirect_ref (op_loc, t_expr.value, RO_ARROW);
+                   }
                  c_parser_consume_token (parser);
                  if (!c_parser_next_token_is (parser, CPP_NAME))
                    {
diff --git a/gcc/testsuite/gcc.dg/gomp/pr104532.c b/gcc/testsuite/gcc.dg/gomp/pr104532.c
new file mode 100644 (file)
index 0000000..1bf8e03
--- /dev/null
@@ -0,0 +1,15 @@
+/* PR c/104532 */
+/* { dg-do compile } */
+
+void
+foo (int x)
+{
+  #pragma omp target enter data map (to: x->vectors)   /* { dg-error "invalid type argument of '->'" } */
+}                                                      /* { dg-error "must contain at least one" "" { target *-*-* } .-1 } */
+
+void
+bar (int x)
+{
+  #pragma omp target enter data map (to: x->vectors[]) /* { dg-error "invalid type argument of '->'" } */
+}                                                      /* { dg-error "must contain at least one" "" { target *-*-* } .-1 } */
+                                                        /* { dg-error "expected expression before" "" { target *-*-* } .-2 } */