Skip to content

Improve CHECK_OP macro#29539

Closed
shinh wants to merge 3 commits intopytorch:masterfrom
shinh:check-op
Closed

Improve CHECK_OP macro#29539
shinh wants to merge 3 commits intopytorch:masterfrom
shinh:check-op

Conversation

@shinh
Copy link
Contributor

@shinh shinh commented Nov 11, 2019

  • Show values in question like glog.
  • Handle expressions with logical operators properly by adding
    parentheses around expressions.
  • Allow outputting nullptr (some build failed without this)

- Show values in question like glog.
- Handle expressions with logical operators properly by adding
  parentheses around expressions.
@shinh
Copy link
Contributor Author

shinh commented Nov 15, 2019

@ljk53 @Yangqing Could you review this PR since you have modified this file recently?

@ezyang ezyang requested a review from ljk53 February 3, 2020 17:40
@ezyang ezyang added the triaged This issue has been looked at a team member, and triaged and prioritized into an appropriate module label Feb 3, 2020
Copy link
Contributor

@facebook-github-bot facebook-github-bot left a comment

Choose a reason for hiding this comment

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

@ljk53 has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

@facebook-github-bot
Copy link
Contributor

@ljk53 merged this pull request in 612e621.

BowenBao pushed a commit to BowenBao/pytorch that referenced this pull request Feb 12, 2020
Summary:
- Show values in question like glog.
- Handle expressions with logical operators properly by adding
  parentheses around expressions.
- Allow outputting nullptr (some build failed without this)
Pull Request resolved: pytorch#29539

Reviewed By: dreiss

Differential Revision: D19698991

Pulled By: ljk53

fbshipit-source-id: e329c01622cfc386ac009904092519a4adfe94a8
ttumiel pushed a commit to ttumiel/pytorch that referenced this pull request Mar 4, 2020
Summary:
- Show values in question like glog.
- Handle expressions with logical operators properly by adding
  parentheses around expressions.
- Allow outputting nullptr (some build failed without this)
Pull Request resolved: pytorch#29539

Reviewed By: dreiss

Differential Revision: D19698991

Pulled By: ljk53

fbshipit-source-id: e329c01622cfc386ac009904092519a4adfe94a8
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Merged 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.

5 participants