Wasmtime GitHub notifications bot (Sep 07 2023 at 13:12): Wasmtime GitHub notifications bot (Sep 07 2023 at 13:12):afonso360 opened PR #6975 from afonso360:riscv-check-ext to bytecodealliance:main:
:wave: Hey,
This PR Is a follow up to #6955 where @alexcrichton suggested that we check that all the flags the backend currently expects.
Most of the backend does not even check these flags, so to avoid emitting instructions where there is no support for them, we reject building the ISA when they aren't present..
This PR also renames the
INSTRUCTION_SIZEconstant to something more suitable, since we are going to have varying instruction lengths.
Wasmtime GitHub notifications bot (Sep 07 2023 at 13:12):afonso360 requested cfallin for a review on PR #6975.
Wasmtime GitHub notifications bot (Sep 07 2023 at 13:12):afonso360 requested wasmtime-compiler-reviewers for a review on PR #6975.
Wasmtime GitHub notifications bot (Sep 07 2023 at 13:13):afonso360 edited PR #6975:
:wave: Hey,
This PR Is a follow up to #6955 where @alexcrichton suggested that we check that all the flags the backend currently expects.
Most of the backend does not even check these flags, so to avoid emitting instructions where there is no support for them, we reject building the ISA when they aren't present.
This PR also renames the
INSTRUCTION_SIZEconstant to something more suitable, since we are going to have varying instruction lengths.
Wasmtime GitHub notifications bot (Sep 07 2023 at 13:32):bjorn3 submitted PR review.
Wasmtime GitHub notifications bot (Sep 07 2023 at 13:32):bjorn3 created PR review comment:
"The RISC-V Backend currently requires all the features in the G Extension enabled".into(),In the future it should be possible to add support for more restricted feature sets, right?
Wasmtime GitHub notifications bot (Sep 07 2023 at 13:36):afonso360 submitted PR review.
Wasmtime GitHub notifications bot (Sep 07 2023 at 13:36):afonso360 created PR review comment:
Oh yeah, we should be able to reduce, ideally all the way until RV64I which is just the 37 base instructions!
Wasmtime GitHub notifications bot (Sep 07 2023 at 13:36):afonso360 updated PR #6975.
Wasmtime GitHub notifications bot (Sep 07 2023 at 13:37):afonso360 edited PR review comment.
Wasmtime GitHub notifications bot (Sep 07 2023 at 13:40):afonso360 updated PR #6975.
Wasmtime GitHub notifications bot (Sep 07 2023 at 14:18):alexcrichton submitted PR review.
Wasmtime GitHub notifications bot (Sep 07 2023 at 16:53):afonso360 merged PR #6975.
Last updated: Dec 23 2024 at 12:05 UTC