From f1ba6a819de6f4aacf9d69725f64496d20edd5a1 Mon Sep 17 00:00:00 2001 From: Jakub Jelinek Date: Fri, 29 Oct 2021 10:20:29 +0200 Subject: gimple-fold: Preserve location in gimple_fold_builtin_memset As mentioned yesterday, gimple_fold_builtin_memset doesn't preserve locus which means e.g. the -Wstringop-overflow warnings are emitted as: In function 'test_max': cc1: warning: writing 1 byte into a region of size 0 [-Wstringop-overflow=] The function emits up to 2 new statements, but the latter (asgn) is added through gsi_replace and therefore the locus is copied over from the call. But store is emitted before the call and optionally the call removed afterwards, so locus needs to be copied over manually. 2021-10-29 Jakub Jelinek * gimple-fold.c (gimple_fold_builtin_memset): Copy over location from call to store. * gcc.dg/Wstringop-overflow-62.c: Adjust expected diagnostics. --- gcc/gimple-fold.c | 1 + 1 file changed, 1 insertion(+) (limited to 'gcc/gimple-fold.c') diff --git a/gcc/gimple-fold.c b/gcc/gimple-fold.c index 7fcfef4..6e25a7c 100644 --- a/gcc/gimple-fold.c +++ b/gcc/gimple-fold.c @@ -1505,6 +1505,7 @@ gimple_fold_builtin_memset (gimple_stmt_iterator *gsi, tree c, tree len) var = fold_build2 (MEM_REF, etype, dest, build_int_cst (ptr_type_node, 0)); gimple *store = gimple_build_assign (var, build_int_cst_type (etype, cval)); gimple_move_vops (store, stmt); + gimple_set_location (store, gimple_location (stmt)); gsi_insert_before (gsi, store, GSI_SAME_STMT); if (gimple_call_lhs (stmt)) { -- cgit v1.1