From ad69edbb4b230582ecd1863e68d0c2044f5ad901 Mon Sep 17 00:00:00 2001 From: Tom Tromey Date: Thu, 26 Jul 2018 19:52:47 -0600 Subject: Use unsigned as base type for some enums -fsanitize=undefined complains about using operator~ on various enum types that are used with DEF_ENUM_FLAGS_TYPE. This patch fixes these problems by explicitly setting the base type for these enums to unsigned. It also adds a static assert to enum_flags to ensure that future enums used this way have an unsigned underlying type. gdb/ChangeLog 2018-10-03 Tom Tromey * common/enum-flags.h (enum_flags::operator~): Add static assert. * symfile-add-flags.h (enum symfile_add_flag): Use unsigned as base type. * objfile-flags.h (enum objfile_flag): Use unsigned as base type. * gdbtypes.h (enum type_instance_flag_value): Use unsigned as base type. * c-lang.h (enum c_string_type_values): Use unsigned as base type. * btrace.h (enum btrace_thread_flag): Use unsigned as base type. --- gdb/btrace.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'gdb/btrace.h') diff --git a/gdb/btrace.h b/gdb/btrace.h index bfb0b9f2..0448bd6 100644 --- a/gdb/btrace.h +++ b/gdb/btrace.h @@ -228,7 +228,7 @@ struct btrace_call_history }; /* Branch trace thread flags. */ -enum btrace_thread_flag +enum btrace_thread_flag : unsigned { /* The thread is to be stepped forwards. */ BTHR_STEP = (1 << 0), -- cgit v1.1