diff options
| author | Paul Menzel <pmenzel@molgen.mpg.de> | 2018-01-23 00:13:57 +0100 |
|---|---|---|
| committer | Patrick Georgi <pgeorgi@google.com> | 2019-01-17 13:05:26 +0000 |
| commit | 60132a43a670276add0e3ea0e56511aabd2ce099 (patch) | |
| tree | 8db9663f730f4918b84a142cfa75c03237535bb3 | |
| parent | d266710d2cca38f417e96447d76b9bfd3fda18f1 (diff) | |
| download | coreboot-60132a43a670276add0e3ea0e56511aabd2ce099.tar.xz | |
lib/libgcc.c: Fix shift warnings
```
if (!(a & (0xffff << 16))) {
^~
src/lib/libgcc.c:40:18: error: result of '255 << 24' requires 33 bits to represent, but 'int' only has 32 bits [-Werror=shift-overflow=]
if (!(a & (0xff << 24))) {
^~
src/lib/libgcc.c:45:17: error: result of '15 << 28' requires 33 bits to represent, but 'int' only has 32 bits [-Werror=shift-overflow=]
if (!(a & (0xf << 28))) {
^~
```
Change-Id: I7bdd75c20a76b303743d7e7e0d3a6503760284fd
Signed-off-by: Paul Menzel <pmenzel@molgen.mpg.de>
Reviewed-on: https://review.coreboot.org/c/23361
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Patrick Rudolph <siro@das-labor.org>
Reviewed-by: Julius Werner <jwerner@chromium.org>
Reviewed-by: HAOUAS Elyes <ehaouas@noos.fr>
| -rw-r--r-- | src/lib/libgcc.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/src/lib/libgcc.c b/src/lib/libgcc.c index 369346cd7d..99a0749a45 100644 --- a/src/lib/libgcc.c +++ b/src/lib/libgcc.c @@ -32,17 +32,17 @@ int __clzsi2(u32 a) }; int r = 0; - if (!(a & (0xffff << 16))) { + if (!(a & (0xffffU << 16))) { r += 16; a <<= 16; } - if (!(a & (0xff << 24))) { + if (!(a & (0xffU << 24))) { r += 8; a <<= 8; } - if (!(a & (0xf << 28))) { + if (!(a & (0xfU << 28))) { r += 4; a <<= 4; } |