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

[QNNPACK:Sparsity] Add A matrix pretransformed based sparse kernels for FC #50587

Closed
wants to merge 17 commits into from

Conversation

kimishpatel
Copy link
Contributor

@kimishpatel kimishpatel commented Jan 15, 2021

Stack from ghstack:

Summary:
This diff introduces two kernesl. One is to pretransform A to do block
wise transforms.
And then the kernel that directly works on top pretransformed weights.

Test Plan:
./build/local/q8gemm-sparse-test
./build/local/fully-connected-sparse-test

Reviewers:

Subscribers:

Tasks:

Tags:

Differential Revision: D25925504

…or FC

Summary:
This diff introduces two kernesl. One is to pretransform A to do block
wise transforms.
And then the kernel that directly works on top pretransformed weights.

Test Plan:
./build/local/q8gemm-sparse-test
./build/local/fully-connected-sparse-test

Reviewers:

Subscribers:

Tasks:

Tags:

[ghstack-poisoned]
…e kernels for FC"

Summary:
This diff introduces two kernesl. One is to pretransform A to do block
wise transforms.
And then the kernel that directly works on top pretransformed weights.

Test Plan:
./build/local/q8gemm-sparse-test
./build/local/fully-connected-sparse-test

Reviewers:

Subscribers:

Tasks:

Tags:

Differential Revision: [D25925504](https://our.internmc.facebook.com/intern/diff/D25925504)

[ghstack-poisoned]
…e kernels for FC"

Summary:
This diff introduces two kernesl. One is to pretransform A to do block
wise transforms.
And then the kernel that directly works on top pretransformed weights.

Test Plan:
./build/local/q8gemm-sparse-test
./build/local/fully-connected-sparse-test

Reviewers:

Subscribers:

Tasks:

Tags:

Differential Revision: [D25925504](https://our.internmc.facebook.com/intern/diff/D25925504)

[ghstack-poisoned]
…e kernels for FC"

Summary:
This diff introduces two kernesl. One is to pretransform A to do block
wise transforms.
And then the kernel that directly works on top pretransformed weights.

Test Plan:
./build/local/q8gemm-sparse-test
./build/local/fully-connected-sparse-test

Reviewers:

Subscribers:

Tasks:

Tags:

Differential Revision: [D25925504](https://our.internmc.facebook.com/intern/diff/D25925504)

[ghstack-poisoned]
…e kernels for FC"

Summary:
This diff introduces two kernesl. One is to pretransform A to do block
wise transforms.
And then the kernel that directly works on top pretransformed weights.

Test Plan:
./build/local/q8gemm-sparse-test
./build/local/fully-connected-sparse-test

Reviewers:

Subscribers:

Tasks:

Tags:

Differential Revision: [D25925504](https://our.internmc.facebook.com/intern/diff/D25925504)

[ghstack-poisoned]
…e kernels for FC"

Summary:
This diff introduces two kernesl. One is to pretransform A to do block
wise transforms.
And then the kernel that directly works on top pretransformed weights.

Test Plan:
./build/local/q8gemm-sparse-test
./build/local/fully-connected-sparse-test

Reviewers:

Subscribers:

Tasks:

Tags:

Differential Revision: [D25925504](https://our.internmc.facebook.com/intern/diff/D25925504)

[ghstack-poisoned]
const uint32_t* kernel_row_values;
const uint8_t* kernel_values;
const float* bias;
float* c; // can be float or uint8)t
Copy link
Contributor

Choose a reason for hiding this comment

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

typo: uint8_t

…e kernels for FC"

Summary:
This diff introduces two kernesl. One is to pretransform A to do block
wise transforms.
And then the kernel that directly works on top pretransformed weights.

Test Plan:
./build/local/q8gemm-sparse-test
./build/local/fully-connected-sparse-test

Reviewers:

Subscribers:

Tasks:

Tags:

Differential Revision: [D25925504](https://our.internmc.facebook.com/intern/diff/D25925504)

[ghstack-poisoned]
…e kernels for FC"

Summary:
This diff introduces two kernesl. One is to pretransform A to do block
wise transforms.
And then the kernel that directly works on top pretransformed weights.

Test Plan:
./build/local/q8gemm-sparse-test
./build/local/fully-connected-sparse-test

Reviewers:

Subscribers:

Tasks:

Tags:

Differential Revision: [D25925504](https://our.internmc.facebook.com/intern/diff/D25925504)

[ghstack-poisoned]
…e kernels for FC"

Summary:
This diff introduces two kernesl. One is to pretransform A to do block
wise transforms.
And then the kernel that directly works on top pretransformed weights.

Test Plan:
./build/local/q8gemm-sparse-test
./build/local/fully-connected-sparse-test

Reviewers:

Subscribers:

Tasks:

Tags:

Differential Revision: [D25925504](https://our.internmc.facebook.com/intern/diff/D25925504)

[ghstack-poisoned]
…e kernels for FC"

Summary:
This diff introduces two kernesl. One is to pretransform A to do block
wise transforms.
And then the kernel that directly works on top pretransformed weights.

Test Plan:
./build/local/q8gemm-sparse-test
./build/local/fully-connected-sparse-test

Reviewers:

Subscribers:

Tasks:

Tags:

Differential Revision: [D25925504](https://our.internmc.facebook.com/intern/diff/D25925504)

[ghstack-poisoned]
…e kernels for FC"

Summary:
This diff introduces two kernesl. One is to pretransform A to do block
wise transforms.
And then the kernel that directly works on top pretransformed weights.

Test Plan:
./build/local/q8gemm-sparse-test
./build/local/fully-connected-sparse-test

Reviewers:

Subscribers:

Tasks:

Tags:

Differential Revision: [D25925504](https://our.internmc.facebook.com/intern/diff/D25925504)

[ghstack-poisoned]
…e kernels for FC"

Summary:
This diff introduces two kernesl. One is to pretransform A to do block
wise transforms.
And then the kernel that directly works on top pretransformed weights.

Test Plan:
./build/local/q8gemm-sparse-test
./build/local/fully-connected-sparse-test

Reviewers:

Subscribers:

Tasks:

Tags:

Differential Revision: [D25925504](https://our.internmc.facebook.com/intern/diff/D25925504)

[ghstack-poisoned]
…e kernels for FC"

Summary:
This diff introduces two kernesl. One is to pretransform A to do block
wise transforms.
And then the kernel that directly works on top pretransformed weights.

Test Plan:
./build/local/q8gemm-sparse-test
./build/local/fully-connected-sparse-test

Reviewers:

Subscribers:

Tasks:

Tags:

Differential Revision: [D25925504](https://our.internmc.facebook.com/intern/diff/D25925504)

[ghstack-poisoned]
…e kernels for FC"

Summary:
This diff introduces two kernesl. One is to pretransform A to do block
wise transforms.
And then the kernel that directly works on top pretransformed weights.

Test Plan:
./build/local/q8gemm-sparse-test
./build/local/fully-connected-sparse-test

Reviewers:

Subscribers:

Tasks:

Tags:

Differential Revision: [D25925504](https://our.internmc.facebook.com/intern/diff/D25925504)

[ghstack-poisoned]
…e kernels for FC"

Summary:
This diff introduces two kernesl. One is to pretransform A to do block
wise transforms.
And then the kernel that directly works on top pretransformed weights.

Test Plan:
./build/local/q8gemm-sparse-test
./build/local/fully-connected-sparse-test

Reviewers:

Subscribers:

Tasks:

Tags:

Differential Revision: [D25925504](https://our.internmc.facebook.com/intern/diff/D25925504)

[ghstack-poisoned]
…e kernels for FC"

Summary:
This diff introduces two kernesl. One is to pretransform A to do block
wise transforms.
And then the kernel that directly works on top pretransformed weights.

Test Plan:
./build/local/q8gemm-sparse-test
./build/local/fully-connected-sparse-test

Reviewers:

Subscribers:

Tasks:

Tags:

Differential Revision: [D25925504](https://our.internmc.facebook.com/intern/diff/D25925504)

[ghstack-poisoned]
…e kernels for FC"

Summary:
This diff introduces two kernesl. One is to pretransform A to do block
wise transforms.
And then the kernel that directly works on top pretransformed weights.

Test Plan:
./build/local/q8gemm-sparse-test
./build/local/fully-connected-sparse-test

Reviewers:

Subscribers:

Tasks:

Tags:

Differential Revision: [D25925504](https://our.internmc.facebook.com/intern/diff/D25925504)

[ghstack-poisoned]
@facebook-github-bot
Copy link
Contributor

This pull request has been merged in 7360ce3.

@facebook-github-bot facebook-github-bot deleted the gh/kimishpatel/36/head branch February 6, 2021 15:15
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.

None yet

4 participants