mbox series

[v3,0/9] target/nios2: Convert to TranslatorOps

Message ID 20210628220810.2919600-1-richard.henderson@linaro.org (mailing list archive)
Headers show
Series target/nios2: Convert to TranslatorOps | expand

Message

Richard Henderson June 28, 2021, 10:08 p.m. UTC 
I've reached a point where *all* targets must use the translator loop.  
Do that, plus some other obvious cleanups.

Changes for v3:
  * Improve the commentary on patch 4 (pmm).
  * Inline handle_instruction.
  * Use pc_next for pc+4 (pmm).

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


r~


Richard Henderson (9):
  target/nios2: Replace DISAS_TB_JUMP with DISAS_NORETURN
  target/nios2: Use global cpu_env
  target/nios2: Use global cpu_R
  target/nios2: Add DisasContextBase to DisasContext
  target/nios2: Convert to TranslatorOps
  target/nios2: Remove assignment to env in handle_instruction
  target/nios2: Clean up goto in handle_instruction
  target/nios2: Inline handle_instruction
  target/nios2: Use pc_next for pc + 4

 target/nios2/translate.c | 318 +++++++++++++++++++--------------------
 1 file changed, 153 insertions(+), 165 deletions(-)

Comments

Richard Henderson June 29, 2021, 5:12 p.m. UTC | #1 
On 6/28/21 3:08 PM, Richard Henderson wrote:
> I've reached a point where *all* targets must use the translator loop.
> Do that, plus some other obvious cleanups.
> 
> Changes for v3:
>    * Improve the commentary on patch 4 (pmm).
>    * Inline handle_instruction.
>    * Use pc_next for pc+4 (pmm).
> 
> Changes for v2:
>    * Fix (drop) singlestep check for max_insns.
>      We already do that generically.

Now fully reviewed.  Queuing to tcg-next.


r~