evan/qemu-xteink
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

docs(wasm32): correct store-TCI comment to upstream root cause

Browse Source
This commit is contained in:
Evan Reichard
2026-07-21 16:14:22 -04:00
parent 962fc6e71d
commit 7cab93ede7
1 changed files with 5 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files
tcg/wasm32/tcg-target.c.inc
+5 -6
View File
@@ -3372,12 +3372,11 @@ static void tcg_out_qemu_st(TCGContext *s, TCGOpcode opc, const TCGArg *args, bo
{
tcg_tci_out_qemu_ldst(s, opc, args);
tcg_wasm_out_qemu_st(s, args, is_64);
/* ponytail: Store-Containing TBs Run on TCI - The compiled wasm store path miscompiles a TB
* containing a guest store (corrupts a pointer a later load faults on; reproduces in uzlib
* inflate during reader font prewarm, native is fine). Forcing these TBs to the interpreter is
* the working workaround. Ceiling: every store-containing TB is interpreted (perf cost). Upgrade
* path: localize the miscompiled op in tcg_wasm_out_qemu_st and drop this. See
* docs/wasm-inflate-panic.md. */
/* ponytail: Store-Containing TBs Run on TCI - Upstream qemu-wasm sets this for every guest store;
* the compiled restartable-block store path is unfinished upstream (never runs there) and corrupts
* state (uzlib inflate reader font prewarm faults; native is fine). Ceiling: store-containing TBs
* interpreted (perf cost, dominates book open). Upgrade path: make compiled stores rewind-safe in
* tcg_wasm_out_qemu_st, then drop this. See docs/wasm-inflate-panic.md. */
wasm_tci_only_tb = true;
}
static void tcg_out_deposit_i32(TCGContext *s, TCGOpcode opc, TCGReg dest, TCGReg arg1, TCGReg arg2, int pos, int len)