| Message ID | 59688AB3.6030901@foss.arm.com |
|---|---|
| State | New |
| Headers | show |
diff --git a/gcc/gimple-ssa-store-merging.c b/gcc/gimple-ssa-store-merging.c index 17a95a50b1fee8ae989343fe63955ab5c045f43b..05b8cd9516a3f863a0f8ce598a69ab8f52a1c441 100644 --- a/gcc/gimple-ssa-store-merging.c +++ b/gcc/gimple-ssa-store-merging.c @@ -331,8 +331,8 @@ clear_bit_region (unsigned char *ptr, unsigned int start, else if (start == 0 && len > BITS_PER_UNIT) { unsigned int nbytes = len / BITS_PER_UNIT; - /* We could recurse on each byte but do the loop here to avoid - recursing too deep. */ + /* We could recurse on each byte but we clear whole bytes, so a simple + memset will do. */ memset (ptr, '\0', nbytes); /* Clear the remaining sub-byte region if there is one. */ if (len % BITS_PER_UNIT != 0)
Hi all, Jakub replaced the simple loop zeroing parts of the array in a previous patch. This patchlet just adjusts the comment to be a bit more accurate. Committed as obvious. Thanks, Kyrill 2017-07-14 Kyrylo Tkachov <kyrylo.tkachov@arm.com> * gimple-ssa-store-merging.c (clear_bit_region): Replace reference to loop in comment with memset.