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

[Relay][TOPI] Fix compute and schedule bugs for conv2d_winograd_nhwc on mali device. #8091

Merged
merged 1 commit into from
May 25, 2021
Merged

[Relay][TOPI] Fix compute and schedule bugs for conv2d_winograd_nhwc on mali device. #8091

merged 1 commit into from
May 25, 2021

Conversation

i4oolish
Copy link
Contributor

This PR fix the schedule and compute bugs when running AutoScheduler om mali device.

  1. add argument auto_scheduler_rewritten_layout="" in conv2d_winograd_nhwc_mali;
  2. add need_auto_scheduler_layout=True for conv2d_strategy_mali and
    conv2d_winograd_without_weight_transfrom_strategy_mali.

Thank you for your time on reviewing this PR :)

…on mali device.

1. add argument `auto_scheduler_rewritten_layout=""` in conv2d_winograd_nhwc_mali;
2. add `need_auto_scheduler_layout=True` for conv2d_strategy_mali and
conv2d_winograd_without_weight_transfrom_strategy_mali.

Signed-off-by: haizhu.shao <haizhu.shao@gmail.com>
Copy link
Contributor

@jcf94 jcf94 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

@jcf94 jcf94 merged commit 0cf7ac3 into apache:main May 25, 2021
trevor-m pushed a commit to trevor-m/tvm that referenced this pull request Jun 17, 2021
…on mali device. (apache#8091)

1. add argument `auto_scheduler_rewritten_layout=""` in conv2d_winograd_nhwc_mali;
2. add `need_auto_scheduler_layout=True` for conv2d_strategy_mali and
conv2d_winograd_without_weight_transfrom_strategy_mali.

Signed-off-by: haizhu.shao <haizhu.shao@gmail.com>
trevor-m pushed a commit to neo-ai/tvm that referenced this pull request Jun 17, 2021
…on mali device. (apache#8091)

1. add argument `auto_scheduler_rewritten_layout=""` in conv2d_winograd_nhwc_mali;
2. add `need_auto_scheduler_layout=True` for conv2d_strategy_mali and
conv2d_winograd_without_weight_transfrom_strategy_mali.

Signed-off-by: haizhu.shao <haizhu.shao@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants