diff options
author | Jeff Law <jeffreyalaw@gmail.com> | 2022-07-09 11:11:00 -0400 |
---|---|---|
committer | Jeff Law <jeffreyalaw@gmail.com> | 2022-07-09 11:11:00 -0400 |
commit | 46dc26fdfbf3e64f82188e21aa6a13ec23108e8e (patch) | |
tree | b45385e17e48aab10eba7b334cb6ae9379092982 /gcc/expr.cc | |
parent | 8f1802003d2b0c516362ea816698317129838bcd (diff) | |
download | gcc-46dc26fdfbf3e64f82188e21aa6a13ec23108e8e.zip gcc-46dc26fdfbf3e64f82188e21aa6a13ec23108e8e.tar.gz gcc-46dc26fdfbf3e64f82188e21aa6a13ec23108e8e.tar.bz2 |
[RFA] Improve initialization of objects when the initializer has trailing zeros.
gcc/
* expr.cc (store_expr): Identify trailing NULs in a STRING_CST
initializer and use clear_storage rather than copying the
NULs to the destination array.
Diffstat (limited to 'gcc/expr.cc')
-rw-r--r-- | gcc/expr.cc | 11 |
1 files changed, 11 insertions, 0 deletions
diff --git a/gcc/expr.cc b/gcc/expr.cc index eb280e6..f9753d4 100644 --- a/gcc/expr.cc +++ b/gcc/expr.cc @@ -6088,6 +6088,17 @@ store_expr (tree exp, rtx target, int call_param_p, } str_copy_len = TREE_STRING_LENGTH (str); + + /* Trailing NUL bytes in EXP will be handled by the call to + clear_storage, which is more efficient than copying them from + the STRING_CST, so trim those from STR_COPY_LEN. */ + while (str_copy_len) + { + if (TREE_STRING_POINTER (str)[str_copy_len - 1]) + break; + str_copy_len--; + } + if ((STORE_MAX_PIECES & (STORE_MAX_PIECES - 1)) == 0) { str_copy_len += STORE_MAX_PIECES - 1; |