diff options
| author | Alexey Kardashevskiy <aik@linux.ibm.com> | 2021-09-13 16:41:01 +1000 |
|---|---|---|
| committer | Dmitry Vyukov <dvyukov@google.com> | 2021-09-16 21:37:48 +0200 |
| commit | 5b989942e1043028009d3a08610a8f52efad7330 (patch) | |
| tree | 0babaeb60c5b7c6f936a9b1f835b9a5099609cdb /executor/executor_test.h | |
| parent | ec8573b1f61d4321a736204c9e14d62f8a5e5405 (diff) | |
pkg/ifuzz/powerpc: fix instructions
The PowerISA pdf parser made several wrong assumptions about the format
("copy"/"paste."/...) and missed that some fields are split and have
reverse order ("rldicr" and similar). Another problem with parsing
2 column page layout.
This makes powerpc.Insn.Fields value type from tuple to an array of
tuples and fixes encodeBits() accordingly.
This fixes powerisa30_to_syz to store all the bits and split "MB"/"ME"
in 2 halves sorted in the way encodeBits() loops.
This should not change the coverage dramatically but this improves
pseudo instructions as "rldicr" is used to preload registers although
no huge difference expected there either as the problem was with top
32bits and hypercall/rtastoken numbers are 16bit anyway.
While at this, this fixes powerisa30_to_syz to make "make generate" not
change insns.go. This also drops comments from the generated file as are
proven to be useless and just increase lengths of already long lines
(vim hates it).
Signed-off-by: Alexey Kardashevskiy <aik@linux.ibm.com>
Diffstat (limited to 'executor/executor_test.h')
0 files changed, 0 insertions, 0 deletions