[v2] target/avr: Convert to TranslatorOps

[PATCH v2 0/3] target/avr: Convert to TranslatorOps

Richard Henderson posted 3 patches 2 years, 10 months ago

Diff against v1
Download series mbox

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/next-importer-push tags/patchew/20210620215022.1510617-1-richard.henderson@linaro.org

Maintainers: Michael Rolnik <mrolnik@gmail.com>

target/avr/translate.c | 284 ++++++++++++++++++++++-------------------
1 file changed, 154 insertions(+), 130 deletions(-)

Expand all Fold all

[PATCH v2 0/3] target/avr: Convert to TranslatorOps

Posted by Richard Henderson 2 years, 10 months ago

I've reached a point where *all* targets must use the translator loop.  

Changes for v2:
  * Fix (drop) singlestep check for max_insns.
    We already do that generically.
  * Fully initialize DisasContext.

It was that last that caused the check-acceptance regression for v1.
I simply got lucky the first time (with optimization) and the
uninitialized struct members happened to have some zeros.


r~


Richard Henderson (3):
  target/avr: Add DisasContextBase to DisasContext
  target/avr: Change ctx to DisasContext* in gen_intermediate_code
  target/avr: Convert to TranslatorOps

 target/avr/translate.c | 284 ++++++++++++++++++++++-------------------
 1 file changed, 154 insertions(+), 130 deletions(-)

-- 
2.25.1