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

[GPU] Make permuteWeights inline #47634

Closed
wants to merge 3 commits into from
Closed

Conversation

xta0
Copy link
Contributor

@xta0 xta0 commented Nov 9, 2020

Stack from ghstack:

Follow up on @dust's diff - D24710102. Make the function inline in order to get rid of the compiler checking -Werror,-Wunused-function.

Differential Revision: D24824637

Follow up on @dust's diff - D24710102. Make the function inline in order to get rid of the compiler checking `-Werror,-Wunused-function`.

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

[ghstack-poisoned]
@dr-ci
Copy link

dr-ci bot commented Nov 10, 2020

💊 CI failures summary and remediations

As of commit 1711d57 (more details on the Dr. CI page):


  • 2/2 failures possibly* introduced in this PR
    • 2/2 non-CircleCI failure(s)

Extra GitHub checks: 1 failed


codecov.io: 1 failed


This comment was automatically generated by Dr. CI (expand for details).Follow this link to opt-out of these comments for your Pull Requests.

Please report bugs/suggestions on the GitHub issue tracker or post in the (internal) Dr. CI Users group.

See how this bot performed.

This comment has been revised 10 times.

Follow up on @dust's diff - D24710102. Make the function inline in order to get rid of the compiler checking `-Werror,-Wunused-function`.

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

[ghstack-poisoned]
Follow up on @dust's diff - D24710102. Make the function inline in order to get rid of the compiler checking `-Werror,-Wunused-function`.

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

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

This pull request has been merged in 2ed3430.

@facebook-github-bot facebook-github-bot deleted the gh/xta0/69/head branch November 16, 2020 15:18
tugsbayasgalan pushed a commit to tugsbayasgalan/pytorch that referenced this pull request Nov 16, 2020
Summary:
Pull Request resolved: pytorch#47634

Follow up on d16r's diff - D24710102. Make the function inline in order to get rid of the compiler checking `-Werror,-Wunused-function`.
ghstack-source-id: 116607200

Test Plan:
1. Sandcastle Tests
2. CircleCI jobs

Reviewed By: d16r

Differential Revision: D24824637

fbshipit-source-id: c17e219b384b91ac4620aa23112a6cda1200a605
jasperzhong pushed a commit to jasperzhong/swift that referenced this pull request Nov 25, 2021
Pull Request resolved: pytorch/pytorch#47634

Follow up on @dust's diff - D24710102. Make the function inline in order to get rid of the compiler checking `-Werror,-Wunused-function`.
ghstack-source-id: 116607200

Differential Revision: [D24824637](https://our.internmc.facebook.com/intern/diff/D24824637/)
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

3 participants