[v1] target/hexagon: Minor idef-parser cleanup

[PATCH 0/4] target/hexagon: Minor idef-parser cleanup

Anton Johansson via posted 4 patches 6 months, 3 weeks ago

Diff against v2 v3 v4
Download series mbox

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20240506183117.32268-1-anjo@rev.ng

Maintainers: Alessandro Di Federico <ale@rev.ng>, Anton Johansson <anjo@rev.ng>

There is a newer version of this series

target/hexagon/idef-parser/idef-parser.h    | 10 --------
target/hexagon/idef-parser/idef-parser.y    |  2 --
target/hexagon/idef-parser/parser-helpers.c | 27 ++++++++++++---------
target/hexagon/idef-parser/parser-helpers.h | 13 ----------
4 files changed, 16 insertions(+), 36 deletions(-)

Expand all Fold all

[PATCH 0/4] target/hexagon: Minor idef-parser cleanup

Posted by Anton Johansson via 6 months, 3 weeks ago

Was running idef-parser with valgrind and noticed we were leaking the
init_list GArray, which is used to hold instruction arguments that may
need initialization.  This patchset fixes the leak, removes unused
macros and undefined functions, and simplifies gen_inst_init_args() to
only handle predicate values.

Anton Johansson (4):
  target/hexagon: idef-parser remove unused defines
  target/hexagon: idef-parser remove undefined functions
  target/hexagon: idef-parser fix leak of init_list
  target/hexagon: idef-parser simplify predicate init

 target/hexagon/idef-parser/idef-parser.h    | 10 --------
 target/hexagon/idef-parser/idef-parser.y    |  2 --
 target/hexagon/idef-parser/parser-helpers.c | 27 ++++++++++++---------
 target/hexagon/idef-parser/parser-helpers.h | 13 ----------
 4 files changed, 16 insertions(+), 36 deletions(-)

-- 
2.44.0