RESOLVED FIXED 218781
[JSC] wasm fault trampoline should be C code since it is tagged as CFunctionPtr 
https://bugs.webkit.org/show_bug.cgi?id=218781
Summary [JSC] wasm fault trampoline should be C code since it is tagged as CFunctionPtr
Yusuke Suzuki
Reported 2020-11-10 19:18:53 PST
[JSC] wasm fault trampoline should be C code since it is tagged as CFunctionPtr
Attachments
Patch (5.34 KB, patch)
2020-11-10 19:20 PST, Yusuke Suzuki 		keith_miller: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Yusuke Suzuki
Comment 1 2020-11-10 19:20:12 PST
Created attachment 413762 [details] Patch
Mark Lam
Comment 2 2020-11-11 09:58:40 PST
Comment on attachment 413762 [details] Patch In AirIRGenerator::AirIRGenerator(), for the MemoryMode::Signaling case, it checks if (UNLIKELY(!Thunks::singleton().stub(throwExceptionFromWasmThunkGenerator))). Is that still needed, or is it now obsolete? Ditto for B3IRGenerator::B3IRGenerator().
Yusuke Suzuki
Comment 3 2020-11-11 11:23:39 PST
(In reply to Mark Lam from comment #2) > Comment on attachment 413762 [details] > Patch > > In AirIRGenerator::AirIRGenerator(), for the MemoryMode::Signaling case, it > checks if > (UNLIKELY(!Thunks::singleton().stub(throwExceptionFromWasmThunkGenerator))). > Is that still needed, or is it now obsolete? > > Ditto for B3IRGenerator::B3IRGenerator(). Right! This is not necessary. Removed.
Yusuke Suzuki
Comment 4 2020-11-11 11:51:26 PST
Committed r269694: <https://trac.webkit.org/changeset/269694>
Radar WebKit Bug Importer
Comment 5 2020-11-11 11:52:17 PST
<rdar://problem/71292659>
Note You need to log in before you can comment on or make changes to this bug.