Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[BUG] NCHW4c is an unsupported convolution layout for arm_cpu #16537

Open
lhutton1 opened this issue Feb 7, 2024 · 0 comments
Open

[BUG] NCHW4c is an unsupported convolution layout for arm_cpu #16537

lhutton1 opened this issue Feb 7, 2024 · 0 comments
Labels
flow:relay The overall lowering flow for tvm.relay.build, including BYOC core, excluding tvm.driver.build. needs-triage PRs or issues that need to be investigated by maintainers to find the right assignees to address it topi python/tvm/topi type: bug

Comments

@lhutton1
Copy link
Contributor

lhutton1 commented Feb 7, 2024

Expected behaviour:

When compiled with target arm_cpu the model should compile successfully.

Actual behaviour:

The test fails to run and gives the following error:

def conv2d_strategy_arm_cpu(attrs, inputs, out_type, target):
...
    else:
>               raise RuntimeError(f"Unsupported conv2d layout {layout} for arm cpu")
E               RuntimeError: Unsupported conv2d layout NCHW4c for arm cpu

python/tvm/relay/op/strategy/arm_cpu.py:273: RuntimeError

Environment:

Tested with TVM at 6a3fadc. The issue was found as a result of the changes in #16513, however it can be reproduced without as described below.

How to reproduce:

pytest tests/python/relay/test_pass_alter_op_layout.py -k test_alter_layout_nonscalar_broadcast
pytest tests/python/relay/test_pass_alter_op_layout.py -k test_alter_layout_blocked_no_broadcast
pytest tests/python/relay/test_pass_alter_op_layout.py -k test_alter_layout_blocked_broadcast
pytest tests/python/relay/test_pass_alter_op_layout.py -k test_alter_layout_re_blocking_broadcast
pytest tests/python/relay/test_pass_alter_op_layout.py -k test_broadcast_non_adaptable
pytest tests/python/relay/test_pass_alter_op_layout.py -k test_broadcast_respect_input_layouts

Run any of the above tests with an arm_cpu target. Note: Reminder to remove any skip condition that exists in the test currently.

@lhutton1 lhutton1 added type: bug topi python/tvm/topi needs-triage PRs or issues that need to be investigated by maintainers to find the right assignees to address it flow:relay The overall lowering flow for tvm.relay.build, including BYOC core, excluding tvm.driver.build. labels Feb 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
flow:relay The overall lowering flow for tvm.relay.build, including BYOC core, excluding tvm.driver.build. needs-triage PRs or issues that need to be investigated by maintainers to find the right assignees to address it topi python/tvm/topi type: bug
Projects
None yet
Development

No branches or pull requests

1 participant