diff mbox series

linearize.h: fix some 'selfcheck' warnings

Message ID 89a7aecb-1a5c-0a43-c9a9-ccee0b12fca6@ramsayjones.plus.com (mailing list archive)
State Mainlined, archived
Headers show
Series linearize.h: fix some 'selfcheck' warnings | expand

Commit Message

Ramsay Jones Feb. 26, 2021, 11:28 p.m. UTC
Commits 34c57a7f ("asm-mem: does it clobber memory?", 2021-02-20) and
d6721b38 ("asm-mem: does it output to memory?", 2021-02-20) both add
a single bit bitfield to the 'struct asm' part of the union contained
within the 'struct instruction'. This causes the 'selfcheck' target
to issue several 'dubious one-bit signed bitfield' errors.

In order to suppress these errors, change the type of the bitfields to
an unsigned type.

Signed-off-by: Ramsay Jones <ramsay@ramsayjones.plus.com>
---
 linearize.h | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

Comments

Luc Van Oostenryck Feb. 28, 2021, 9:33 p.m. UTC | #1
On Fri, Feb 26, 2021 at 11:28:43PM +0000, Ramsay Jones wrote:
> 
> Commits 34c57a7f ("asm-mem: does it clobber memory?", 2021-02-20) and
> d6721b38 ("asm-mem: does it output to memory?", 2021-02-20) both add
> a single bit bitfield to the 'struct asm' part of the union contained
> within the 'struct instruction'. This causes the 'selfcheck' target
> to issue several 'dubious one-bit signed bitfield' errors.
> 
> In order to suppress these errors, change the type of the bitfields to
> an unsigned type.

Thank you,
-- Luc
diff mbox series

Patch

diff --git a/linearize.h b/linearize.h
index cf0cf066..01ee656c 100644
--- a/linearize.h
+++ b/linearize.h
@@ -150,8 +150,8 @@  struct instruction {
 		struct /* asm */ {
 			const char *string;
 			struct asm_rules *asm_rules;
-			int clobber_memory:1;
-			int output_memory:1;
+			unsigned int clobber_memory:1;
+			unsigned int output_memory:1;
 		};
 	};
 };