mirror of
https://github.com/espressif/binutils-gdb.git
synced 2025-10-12 18:36:10 +08:00
opcodes: make use of __builtin_popcount when available
This commit provides a small performance improvement when starting up CGEN based disassemblers by making use of __builtin_popcount. The #if check used in this commit was copied from bfd/elf32-arm.c where __builtin_popcount is also used. I ran into this code while investigating some GDB tests that would occasionally timeout. One of the reason these tests were having problems is that the m16c and m32c disassemblers take so long to initialise themselves. Speeding up count_decodable_bits helps, but is not a total solution. Still, this felt like an easy win which added minimal extra complexity, so I figure its worth doing. opcodes/ChangeLog: * cgen-dis.c (count_decodable_bits): Use __builtin_popcount when available.
This commit is contained in:
@ -1,3 +1,8 @@
|
||||
2021-06-22 Andrew Burgess <andrew.burgess@embecosm.com>
|
||||
|
||||
* cgen-dis.c (count_decodable_bits): Use __builtin_popcount when
|
||||
available.
|
||||
|
||||
2021-06-22 Alan Modra <amodra@gmail.com>
|
||||
|
||||
* pj-dis.c (print_insn_pj): Don't print trailing tab. Do
|
||||
|
@ -40,6 +40,9 @@ static int
|
||||
count_decodable_bits (const CGEN_INSN *insn)
|
||||
{
|
||||
unsigned mask = CGEN_INSN_BASE_MASK (insn);
|
||||
#if GCC_VERSION >= 3004
|
||||
return __builtin_popcount (mask);
|
||||
#else
|
||||
int bits = 0;
|
||||
unsigned m;
|
||||
|
||||
@ -49,6 +52,7 @@ count_decodable_bits (const CGEN_INSN *insn)
|
||||
++bits;
|
||||
}
|
||||
return bits;
|
||||
#endif
|
||||
}
|
||||
|
||||
/* Add an instruction to the hash chain. */
|
||||
|
Reference in New Issue
Block a user