Skip to content

Commit

Permalink
Disable MatMul QDQ selector on DML EP until MatMulIntegerToFloat is r…
Browse files Browse the repository at this point in the history
…e-enabled
  • Loading branch information
jeffbloo committed Jan 4, 2024
1 parent 70a6f81 commit f4ad940
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -216,7 +216,8 @@ void MatMulQDQRules(SelectorActionRegistry& qdq_selector_action_registry, bool i

#if !defined(ORT_MINIMAL_BUILD)
// TODO: Enable 16-bit types in selector when QLinearMatMul and MatMulInteger support 16-bit.
std::unique_ptr<NodeSelector> selector = std::make_unique<QDQ::MatMulSelector>(is_int8_allowed);
std::vector<const char*> providers = {kCpuExecutionProvider};
std::unique_ptr<NodeSelector> selector = std::make_unique<QDQ::MatMulSelector>(providers, is_int8_allowed);
qdq_selector_action_registry.RegisterSelectorAndAction(action_name,
{{"MatMul", {}}},
std::move(selector),
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -337,9 +337,10 @@ class WhereSelector : public BaseSelector {
// 2 DQ nodes for input -> node -> optional Q if QLinearMatMul, MatMulIntegerToFloat if not
class MatMulSelector : public BaseSelector {
public:
MatMulSelector(bool int8_allowed, bool allow_16bit = false)
MatMulSelector(gsl::span<const char*> compatible_providers, bool int8_allowed, bool allow_16bit = false)
: BaseSelector(std::make_unique<MatMulNodeGroupSelector>(int8_allowed, /*matmulintegertofloat_allowed*/ true,
allow_16bit)) {}
allow_16bit),
compatible_providers) {}
};

// Input: DQ nodes for A, B and optional C
Expand Down

0 comments on commit f4ad940

Please sign in to comment.