tree-nested.c has this code: 2333 for (c = gimple_omp_taskreg_clauses (stmt); 2334 c; 2335 c = OMP_CLAUSE_CHAIN (c)) 2336 if ((OMP_CLAUSE_CODE (c) == OMP_CLAUSE_FIRSTPRIVATE 2337 || OMP_CLAUSE_CODE (c) == OMP_CLAUSE_SHARED) 2338 && OMP_CLAUSE_DECL (c) == decl) 2339 break; 2340 if (c == NULL && gimple_code (stmt) != GIMPLE_OMP_TARGET) 2341 {
which leads to this warning from -Wmisleading-indentation (justifiably, in my opinion): ../../../src/gcc/tree-nested.c: In function ‘tree_node* convert_tramp_reference_stmt(gimple_stmt_iterator*, bool*, walk_stmt_info* ’: ../../../src/gcc/tree-nested.c:2340:8: error: statement is indented as if it were guarded by... [-Werror=misleading-indentation] if (c == NULL && gimple_code (stmt) != GIMPLE_OMP_TARGET) ^ ../../../src/gcc/tree-nested.c:2333:6: note: ...this ‘for’ clause, but it is not for (c = gimple_omp_taskreg_clauses (stmt); ^ This patch fixes the indentation of lines 2340-2360 to make it clear that they're not part of the "for" loop. gcc/ChangeLog: * tree-nested.c (convert_tramp_reference_stmt): Fix indentation. --- gcc/tree-nested.c | 42 +++++++++++++++++++++--------------------- 1 file changed, 21 insertions(+), 21 deletions(-) diff --git a/gcc/tree-nested.c b/gcc/tree-nested.c index c04c429..a5435f9 100644 --- a/gcc/tree-nested.c +++ b/gcc/tree-nested.c @@ -2337,27 +2337,27 @@ convert_tramp_reference_stmt (gimple_stmt_iterator *gsi, bool *handled_ops_p, || OMP_CLAUSE_CODE (c) == OMP_CLAUSE_SHARED) && OMP_CLAUSE_DECL (c) == decl) break; - if (c == NULL && gimple_code (stmt) != GIMPLE_OMP_TARGET) - { - c = build_omp_clause (gimple_location (stmt), - i ? OMP_CLAUSE_FIRSTPRIVATE - : OMP_CLAUSE_SHARED); - OMP_CLAUSE_DECL (c) = decl; - OMP_CLAUSE_CHAIN (c) = gimple_omp_taskreg_clauses (stmt); - gimple_omp_taskreg_set_clauses (stmt, c); - } - else if (c == NULL) - { - c = build_omp_clause (gimple_location (stmt), - OMP_CLAUSE_MAP); - OMP_CLAUSE_DECL (c) = decl; - OMP_CLAUSE_SET_MAP_KIND (c, - i ? GOMP_MAP_TO : GOMP_MAP_TOFROM); - OMP_CLAUSE_SIZE (c) = DECL_SIZE_UNIT (decl); - OMP_CLAUSE_CHAIN (c) = gimple_omp_target_clauses (stmt); - gimple_omp_target_set_clauses (as_a <gomp_target *> (stmt), - c); - } + if (c == NULL && gimple_code (stmt) != GIMPLE_OMP_TARGET) + { + c = build_omp_clause (gimple_location (stmt), + i ? OMP_CLAUSE_FIRSTPRIVATE + : OMP_CLAUSE_SHARED); + OMP_CLAUSE_DECL (c) = decl; + OMP_CLAUSE_CHAIN (c) = gimple_omp_taskreg_clauses (stmt); + gimple_omp_taskreg_set_clauses (stmt, c); + } + else if (c == NULL) + { + c = build_omp_clause (gimple_location (stmt), + OMP_CLAUSE_MAP); + OMP_CLAUSE_DECL (c) = decl; + OMP_CLAUSE_SET_MAP_KIND (c, + i ? GOMP_MAP_TO : GOMP_MAP_TOFROM); + OMP_CLAUSE_SIZE (c) = DECL_SIZE_UNIT (decl); + OMP_CLAUSE_CHAIN (c) = gimple_omp_target_clauses (stmt); + gimple_omp_target_set_clauses (as_a <gomp_target *> (stmt), + c); + } } info->new_local_var_chain = save_local_var_chain; info->static_chain_added |= save_static_chain_added; -- 1.8.5.3