cfallin opened Issue #2109:
We have temporarily disabled the new x64 backend's CI tests (#2100) because it would intermittently fail these tests with a crash. We need to resolve the underlying issue (perhaps a codegen bug) and re-enable the tests.
cfallin commented on Issue #2109:
cc @julian-seward1 @bnjbvr
cfallin labeled Issue #2109:
We have temporarily disabled the new x64 backend's CI tests (#2100) because it would intermittently fail these tests with a crash. We need to resolve the underlying issue (perhaps a codegen bug) and re-enable the tests.
bnjbvr commented on Issue #2109:
I don't have strong evidence, but I think this has been fixed by https://github.com/bytecodealliance/wasmtime/commit/3849dc18b1bf3560c43950f049fe553034c424b2.
See https://github.com/bytecodealliance/wasmtime/commit/3849dc18b1bf3560c43950f049fe553034c424b2#diff-234d9b09197fd3d4bc69bf1a2ca4f2f8L2410. Before the PR, when emitting a constant
null
for a reference type, we wouldn't match theis_int
, so a floating-point xorps/xorpd would be used to generate the register. According to the result of regalloc, this could result in incorrect code in the best case, uninitialized registers in the worst case. The above patch is a likely fix since the issue was intermittent, and matches this behavior of randomness, and crashes happened around reference type tests (I remember seeing double frees, etc.).I'll close the issue now; if the intermittent failure happens again, I'm happy to look into it a bit more.
bnjbvr closed Issue #2109:
We have temporarily disabled the new x64 backend's CI tests (#2100) because it would intermittently fail these tests with a crash. We need to resolve the underlying issue (perhaps a codegen bug) and re-enable the tests.
bnjbvr labeled Issue #2109:
We have temporarily disabled the new x64 backend's CI tests (#2100) because it would intermittently fail these tests with a crash. We need to resolve the underlying issue (perhaps a codegen bug) and re-enable the tests.
github-actions[bot] commented on Issue #2109:
Subscribe to Label Action
cc @bnjbvr
<details>
This issue or pull request has been labeled: "cranelift"Thus the following users have been cc'd because of the following labels:
- bnjbvr: cranelift
To subscribe or unsubscribe from this label, edit the <code>.github/subscribe-to-label.json</code> configuration file.
Learn more.
</details>
Last updated: Dec 23 2024 at 12:05 UTC