tvm icon indicating copy to clipboard operation
tvm copied to clipboard

[Relax][Bugfix] FCallPacked not checked in CodegenVMTIR

Open Lunderberg opened this issue 1 year ago • 1 comments

Prior to this commit, an operator's FCallPacked attribute, used to specify a 1:1 mapping between a relax operator and a PackedFunc that implements it, was only checked in CodegenVM. Any operator with FCallPacked would raise an error when compiled using CodegenVMTIR.

This commit removes the FCallPacked handling from CodegenVM altogether, and instead checks for this attribute as part of LegalizeOps. This provides the same functionality across both backends.

Lunderberg avatar Jun 07 '24 15:06 Lunderberg

@tvm-bot rerun

Lunderberg avatar Jun 07 '24 21:06 Lunderberg

@Lunderberg need rebase

tqchen avatar Sep 05 '24 14:09 tqchen

Thank you, and rebased to resolve conflicts. (And again to resolve a conflict that wasn't flagged by git.)

Lunderberg avatar Sep 05 '24 16:09 Lunderberg