Skip to content

Conversation

@xuhdev
Copy link
Collaborator

@xuhdev xuhdev commented Jun 26, 2019

Stack from ghstack:

Related to #20024, #22246, #22262

Differential Revision: D16183576

@pytorchbot pytorchbot added module: cpu CPU specific problem (e.g., perf, algorithm) module: docs Related to our documentation, both in docs/ and docblocks module: internals Related to internal abstractions in c10 and ATen module: operators labels Jun 26, 2019
…CPU)."

Add a bitwise NOT operator for integer and Boolean types (CPU).

Related to #20024, #22246, #22262

gh-metadata: pytorch pytorch 22283 gh/xuhdev/6/head
xuhdev added 2 commits June 26, 2019 16:36
…CPU)."

Add a bitwise NOT operator for integer and Boolean types (CPU).

Related to #20024, #22246, #22262

gh-metadata: pytorch pytorch 22283 gh/xuhdev/6/head
…CPU)."

Add a bitwise NOT operator for integer and Boolean types (CPU).

Related to #20024, #22246, #22262

gh-metadata: pytorch pytorch 22283 gh/xuhdev/6/head
@xuhdev
Copy link
Collaborator Author

xuhdev commented Jun 27, 2019

In subsequent PRs, I will implement a GPU version and replace the ~ operator implementation with this function. But this PR by itself can also be seen as an independent contribution.

…CPU)."

Add a bitwise NOT operator for integer and Boolean types (CPU).

Related to #20024, #22246, #22262

gh-metadata: pytorch pytorch 22283 gh/xuhdev/6/head
@xuhdev xuhdev requested a review from colesbury June 27, 2019 01:04
…CPU)."

Add a bitwise NOT operator for integer and Boolean types (CPU).

Related to #20024, #22246, #22262

gh-metadata: pytorch pytorch 22283 gh/xuhdev/6/head
@ailzhang ailzhang requested a review from izdeby June 27, 2019 04:07
@ailzhang ailzhang added the triaged This issue has been looked at a team member, and triaged and prioritized into an appropriate module label Jun 27, 2019
…CPU)."

Add a bitwise NOT operator for integer and Boolean types (CPU).

Related to #20024, #22246, #22262

gh-metadata: pytorch pytorch 22283 gh/xuhdev/6/head
…CPU)."

Add a bitwise NOT operator for integer and Boolean types (CPU).

Related to #20024, #22246, #22262

gh-metadata: pytorch pytorch 22283 gh/xuhdev/6/head
@xuhdev xuhdev requested a review from izdeby June 27, 2019 16:46
izdeby
izdeby previously approved these changes Jun 27, 2019
@izdeby
Copy link
Contributor

izdeby commented Jun 27, 2019

are you going to add support for CUDA?

@xuhdev
Copy link
Collaborator Author

xuhdev commented Jun 27, 2019

@izdeby Yes

…CPU)."

Add a bitwise NOT operator for integer and Boolean types (CPU).

Related to #20024, #22246, #22262

gh-metadata: pytorch pytorch 22283 gh/xuhdev/6/head
colesbury
colesbury previously approved these changes Jun 27, 2019
Copy link
Member

@colesbury colesbury left a comment

Choose a reason for hiding this comment

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

lgtm too

xuhdev added 2 commits June 27, 2019 15:33
…CPU)."

Add a bitwise NOT operator for integer and Boolean types (CPU).

Related to #20024, #22246, #22262

gh-metadata: pytorch pytorch 22283 gh/xuhdev/6/head
…CPU)."

Add a bitwise NOT operator for integer and Boolean types (CPU).

Related to #20024, #22246, #22262

gh-metadata: pytorch pytorch 22283 gh/xuhdev/6/head
xuhdev added 2 commits June 30, 2019 09:23
…CPU)."

Add a bitwise NOT operator for integer and Boolean types (CPU).

Related to #20024, #22246, #22262

gh-metadata: pytorch pytorch 22283 gh/xuhdev/6/head
…CPU)."

Add a bitwise NOT operator for integer and Boolean types (CPU).

Related to #20024, #22246, #22262

gh-metadata: pytorch pytorch 22283 gh/xuhdev/6/head
@xuhdev
Copy link
Collaborator Author

xuhdev commented Jun 30, 2019

@pytorchbot retest this please

1 similar comment
@xuhdev
Copy link
Collaborator Author

xuhdev commented Jul 1, 2019

@pytorchbot retest this please

xuhdev added 2 commits July 1, 2019 13:04
…CPU)."

Add a bitwise NOT operator for integer and Boolean types (CPU).

Related to #20024, #22246, #22262

gh-metadata: pytorch pytorch 22283 gh/xuhdev/6/head
…CPU)."

Add a bitwise NOT operator for integer and Boolean types (CPU).

Related to #20024, #22246, #22262

gh-metadata: pytorch pytorch 22283 gh/xuhdev/6/head
@xuhdev xuhdev requested review from colesbury and izdeby July 1, 2019 22:23
@xuhdev
Copy link
Collaborator Author

xuhdev commented Jul 1, 2019

@colesbury @izdeby I changed the code to avoid the template bitwise_not function (and thus the code becomes simpler)

xuhdev added 6 commits July 1, 2019 15:27
…CPU)."

Add a bitwise NOT operator for integer and Boolean types (CPU).

Related to #20024, #22246, #22262

gh-metadata: pytorch pytorch 22283 gh/xuhdev/6/head
…CPU)."

Add a bitwise NOT operator for integer and Boolean types (CPU).

Related to #20024, #22246, #22262

gh-metadata: pytorch pytorch 22283 gh/xuhdev/6/head
…CPU)."

Add a bitwise NOT operator for integer and Boolean types (CPU).

Related to #20024, #22246, #22262

gh-metadata: pytorch pytorch 22283 gh/xuhdev/6/head
…CPU)."

Add a bitwise NOT operator for integer and Boolean types (CPU).

Related to #20024, #22246, #22262

gh-metadata: pytorch pytorch 22283 gh/xuhdev/6/head
…CPU)."

Add a bitwise NOT operator for integer and Boolean types (CPU).

Related to #20024, #22246, #22262

gh-metadata: pytorch pytorch 22283 gh/xuhdev/6/head
…CPU)."

Add a bitwise NOT operator for integer and Boolean types (CPU).

Related to #20024, #22246, #22262

gh-metadata: pytorch pytorch 22283 gh/xuhdev/6/head
@zou3519 zou3519 dismissed stale reviews from colesbury and izdeby July 8, 2019 18:20

stale

@zou3519
Copy link
Contributor

zou3519 commented Jul 8, 2019

@izdeby @colesbury could you take another look at this please?

…CPU)."

Add a bitwise NOT operator for integer and Boolean types (CPU).

Related to #20024, #22246, #22262

gh-metadata: pytorch pytorch 22283 gh/xuhdev/6/head
@zou3519 zou3519 deleted the gh/xuhdev/6/head branch July 10, 2019 19:20
zdevito pushed a commit to zdevito/ATen that referenced this pull request Jul 10, 2019
Summary: Pull Request resolved: pytorch/pytorch#22283

Test Plan: Imported from OSS

Differential Revision: D16183576

Pulled By: colesbury

fbshipit-source-id: 2e539fab8ff885dddb9bff334d1d784b28d65b8f
@facebook-github-bot
Copy link
Contributor

@colesbury merged this pull request in e2dc1fc.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Merged module: cpu CPU specific problem (e.g., perf, algorithm) module: docs Related to our documentation, both in docs/ and docblocks module: internals Related to internal abstractions in c10 and ATen open source triaged This issue has been looked at a team member, and triaged and prioritized into an appropriate module

Projects

None yet

Development

Successfully merging this pull request may close these issues.

10 participants