From: Richard Sandiford Date: Tue, 20 Nov 2012 22:49:41 +0000 (+0000) Subject: stor-layout.c (bit_field_mode_iterator::next_mode): Fix signedness. X-Git-Tag: releases/gcc-4.8.0~1875 X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=ec593a8fc3c739005538ffbe097a6ad38240be6d;p=thirdparty%2Fgcc.git stor-layout.c (bit_field_mode_iterator::next_mode): Fix signedness. gcc/ * stor-layout.c (bit_field_mode_iterator::next_mode): Fix signedness. From-SVN: r193680 --- diff --git a/gcc/ChangeLog b/gcc/ChangeLog index 59982dfb1e67..8990122d3ab3 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,3 +1,7 @@ +2012-11-20 Richard Sandiford + + * stor-layout.c (bit_field_mode_iterator::next_mode): Fix signedness. + 2012-11-20 Vladimir Makarov PR rtl-optimization/55396 diff --git a/gcc/stor-layout.c b/gcc/stor-layout.c index 03da59e0f1f6..b1b7cb29fc4f 100644 --- a/gcc/stor-layout.c +++ b/gcc/stor-layout.c @@ -2670,10 +2670,6 @@ bit_field_mode_iterator::next_mode (enum machine_mode *out_mode) if (unit != GET_MODE_PRECISION (mode_)) continue; - /* Skip modes that are too small. */ - if ((bitpos_ % unit) + bitsize_ > unit) - continue; - /* Stop if the mode is too wide to handle efficiently. */ if (unit > MAX_FIXED_MODE_SIZE) break; @@ -2683,11 +2679,18 @@ bit_field_mode_iterator::next_mode (enum machine_mode *out_mode) if (count_ > 0 && unit > BITS_PER_WORD) break; + /* Skip modes that are too small. */ + unsigned HOST_WIDE_INT substart = (unsigned HOST_WIDE_INT) bitpos_ % unit; + unsigned HOST_WIDE_INT subend = substart + bitsize_; + if (subend > unit) + continue; + /* Stop if the mode goes outside the bitregion. */ - HOST_WIDE_INT start = bitpos_ - (bitpos_ % unit); + HOST_WIDE_INT start = bitpos_ - substart; if (bitregion_start_ && start < bitregion_start_) break; - if (start + unit > bitregion_end_ + 1) + HOST_WIDE_INT end = start + unit; + if (end > bitregion_end_ + 1) break; /* Stop if the mode requires too much alignment. */