0xMiden / 0xMiden/miden-vm

Reduce stack footprint for error handling

Aperta
#2,576 3 commenti 0 reazioni 0 assegnatari Vedi su GitHub
processor
Lingua principale
Rust
Stelle
772
Fork
352
Merge medio
1g 12h
PR unite (30g)
93

Descrizione

The error handling changes in #2544 result in a large stack footprint due to:
1. Many arguments passed to error mapping functions
2. Growing size of `ExecutionError`

Potential solutions:
- Re-introduce `ErrorContext` as a struct passed by reference (reduces argument count)
- Box inner error types (reduces `ExecutionError` size)

Note: Boxing is not trivial since `Diagnostic` is not implemented for `Box`.

See [comment](https://github.com/0xMiden/miden-vm/pull/2544#issuecomment-3724332474).

Guida per i contributori

Apri la guida per i contributori

Valutazione

Questa issue non è ancora stata valutata.

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.