x86: never set i386_cpu_flags' "unused" field
authorJan Beulich <jbeulich@suse.com>
Thu, 17 Mar 2022 10:05:11 +0000 (11:05 +0100)
committerJan Beulich <jbeulich@suse.com>
Thu, 17 Mar 2022 10:05:11 +0000 (11:05 +0100)
commit13ed231a0fad6e5988673dc8fde3996ad0136735
treea07c547405db4bbd487bc1983b2f9bd5e5e57e1b
parentad9de929c3ef0b3b6c32db05cf3dc33ff3904990
x86: never set i386_cpu_flags' "unused" field

Setting this field risks cpu_flags_all_zero() mistakenly returning
"false" when the object passed in was e.g. the result of ANDing together
two objects which had the bit set, or ANDNing together an object with
the field set and one with the field clear.

While there also avoid setting CpuNo64: Like Cpu64 this is driven
differently anyway and hence shouldn't be set anywhere by default.

Note that the moving of the two items in i386-gen.c's cpu_flags[] is
only for documentation purposes (and slight reducing of overhead), as
the fields are sorted anyway upon program start.
opcodes/i386-gen.c
opcodes/i386-init.h
opcodes/i386-opc.h