Stream: git-wasmtime

Topic: wasmtime / issue #6050 Cranelift: Consider emitting jump ...


view this post on Zulip Wasmtime GitHub notifications bot (Mar 17 2023 at 17:50):

fitzgen opened issue #6050:

Right now they are emitted inline right after the code that evaluates the table index and loads the target out of the table. This seems like it could waste instruction pipeline bandwidth and icache space. Might be better to emit them at the end of the function, similar to @alexcrichton's recent changes for emitting MInst::TrapIf's trap instructions at the end of the function. Would need to integrate with the whole islands business, of course.

view this post on Zulip Wasmtime GitHub notifications bot (Mar 17 2023 at 17:50):

fitzgen labeled issue #6050:

Right now they are emitted inline right after the code that evaluates the table index and loads the target out of the table. This seems like it could waste instruction pipeline bandwidth and icache space. Might be better to emit them at the end of the function, similar to @alexcrichton's recent changes for emitting MInst::TrapIf's trap instructions at the end of the function. Would need to integrate with the whole islands business, of course.


Last updated: Oct 23 2024 at 20:03 UTC