riscv-non-isa / riscv-trace-spec

RISC-V Processor Trace Specification
https://jira.riscv.org/browse/RVG-88
Creative Commons Attribution 4.0 International
152 stars 47 forks source link

The field width of branch_map in format 1 #133

Closed zhangdujiao closed 1 month ago

zhangdujiao commented 1 month ago

Can the width of branch_map be fixed at 31bit, rather than determined by branches value, since the latter will need lots of registers for implementation.

IainCRobertson commented 1 month ago

No, this is not possible (at least, not whilst remaining compliant to the E-Trace standard).

In our implementation this doesn't cost any registers, but it does require muxes to correctly align the upper part of the trace packet payload. Whilst the approach you suggest would reduce the muxing cost, it would significantly impact the encoding efficiency.

Iain

From: zhangdujiao @.> Sent: 04 August 2024 04:46 To: riscv-non-isa/riscv-trace-spec @.> Cc: Subscribed @.***> Subject: [riscv-non-isa/riscv-trace-spec] The field width of branch_map in format 1 (Issue #133)

Can the width of branch_map be fixed at 31bit, rather than determined by branches value, since the latter will need lots of registers for implementation.

- Reply to this email directly, view it on GitHubhttps://github.com/riscv-non-isa/riscv-trace-spec/issues/133, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ALQOPSXSNKGFPYIR6AL2BETZPWPZDAVCNFSM6AAAAABL6O3OHKVHI2DSMVQWIX3LMV43ASLTON2WKOZSGQ2DMNZXGUYTEMQ. You are receiving this because you are subscribed to this thread.Message ID: @.**@.>>