afonso360 opened PR #3282 from x64-fix-brtables
to main
:
With this PR we fix the compilation crash for
br_table.i64
. We still only support 2^32 targets, but we now allow that number to come from a i64 value.I don't really know enough about x86 to improve the
Inst::JmpTableSeq
and enable 2^64 targets, but I suspect its not likely to be an issue.Fixes #3100. The CLIF Fuzzer found this independently as soon as I added jump tables.
bjorn3 submitted PR review.
bjorn3 created PR review comment:
You will still have to keep the special case in
cranelift/frontend/src/switch.rs
for 128bit ints.
afonso360 updated PR #3282 from x64-fix-brtables
to main
.
afonso360 updated PR #3282 from x64-fix-brtables
to main
.
afonso360 updated PR #3282 from x64-fix-brtables
to main
.
cfallin submitted PR review.
cfallin merged PR #3282.
Last updated: Dec 23 2024 at 12:05 UTC