Skip to content

[QNN] Fix padding changes due to #3739 - #3989

Merged
tqchen merged 1 commit into
apache:masterfrom
anijain2305:fix_pad_qnn
Sep 23, 2019
Merged

[QNN] Fix padding changes due to #3739#3989
tqchen merged 1 commit into
apache:masterfrom
anijain2305:fix_pad_qnn

Conversation

@anijain2305

@anijain2305anijain2305 commented Sep 21, 2019

Copy link
Copy Markdown
Contributor

Somehow #3739 went in without failing the Jenkins. It added a new pad_mode which was not propagated to QNN. This PR fixes that

@kevinthesun@alexgl-github@tqchen

@cchung100mcchung100m left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM :)

@tqchen
tqchen merged commit 8eb3157 into apache:masterSep 23, 2019
@tqchen

Copy link
Copy Markdown
Member

Thanks @cchung100m@anijain2305

wweic pushed a commit to wweic/tvm that referenced this pull request Sep 30, 2019
wweic pushed a commit to wweic/tvm that referenced this pull request Sep 30, 2019
wweic pushed a commit to neo-ai/tvm that referenced this pull request Oct 1, 2019
@anijain2305
anijain2305 deleted the fix_pad_qnn branch November 13, 2019 00:31
Sign up for freeto 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

@anijain2305@tqchen@cchung100m