Skip to content

Commit 51ecf47

Browse files
authored
Re-enable fused matmul rules (#2370)
Enable fused matmul rules again, since they were commented out
1 parent cabd83b commit 51ecf47

File tree

2 files changed

+4
-6
lines changed

2 files changed

+4
-6
lines changed

onnxscript/rewriter/ort_fusions/_core.py

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,7 @@
44

55
import onnxscript.ir as ir
66
import onnxscript.ir.passes.common as common_passes
7+
import onnxscript.rewriter.ort_fusions.fused_matmul_rule_sets as fused_matmul_rule_sets
78
import onnxscript.rewriter.ort_fusions.shape_optimization as shape_optimization
89
from onnxscript.optimizer import optimize
910
from onnxscript.rewriter import rewrite
@@ -37,9 +38,7 @@
3738
*instance_to_group_normalization.rules.rules,
3839
# NOTE: group normalization merge silu should be applied after instance to group normalization
3940
# *group_normalization_merge_silu.rules.rules,
40-
# NOTE: The rules below are broken:
41-
# https://github.com/microsoft/onnxscript/pull/2317#issuecomment-2896058483
42-
# *fused_matmul_rule_sets.fused_matmul_rule_sets(),
41+
*fused_matmul_rule_sets.fused_matmul_rule_sets().rules,
4342
]
4443

4544

onnxscript/rewriter/ort_fusions/fused_matmul_rule_sets.py

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -327,9 +327,8 @@ def pattern(self, op, x, y):
327327

328328

329329
def fused_matmul_rule_sets() -> orp.RewriteRuleSet:
330-
"""Returns a set of rules introducing onnxruntime contrib obs.
331-
This requires onnxruntime to run the model after
332-
it is rewritten.
330+
"""Returns a set of rules introducing onnxruntime contrib ops.
331+
This requires onnxruntime to run the model after it is rewritten.
333332
334333
Returns:
335334
RewriteRuleSet

0 commit comments

Comments
 (0)