Skip to content

[unified build] Default hidden visibility#10752

Closed
orionr wants to merge 16 commits intopytorch:masterfrom
orionr:even-more-hidden
Closed

[unified build] Default hidden visibility#10752
orionr wants to merge 16 commits intopytorch:masterfrom
orionr:even-more-hidden

Conversation

@orionr
Copy link
Contributor

@orionr orionr commented Aug 21, 2018

Flipping to hidden visibility one more time. Let's see what fails.

cc @mingzhe09088 @pjh5 @Yangqing

@orionr orionr force-pushed the even-more-hidden branch 7 times, most recently from bac7aaf to 97717f1 Compare August 27, 2018 14:46
@orionr orionr changed the title [WIP][unified build] Default hidden visibility [unified build] Default hidden visibility Aug 27, 2018
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.

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


// TODO: Merge the *_API macros.
#define AT_API AT_CORE_API
#define AT_EXPORT AT_CORE_EXPORT

This comment was marked as off-topic.

zdevito pushed a commit to zdevito/ATen that referenced this pull request Aug 29, 2018
Summary:
Flipping to hidden visibility one more time. Let's see what fails.

cc mingzhe09088 pjh5 Yangqing
Pull Request resolved: pytorch/pytorch#10752

Reviewed By: ezyang

Differential Revision: D9526343

Pulled By: orionr

fbshipit-source-id: c0e9c29270e95e1b2e21c598095f720c199e1e52
PenghuiCheng pushed a commit to PenghuiCheng/pytorch that referenced this pull request Sep 11, 2018
Summary:
Flipping to hidden visibility one more time. Let's see what fails.

cc mingzhe09088 pjh5 Yangqing
Pull Request resolved: pytorch#10752

Reviewed By: ezyang

Differential Revision: D9526343

Pulled By: orionr

fbshipit-source-id: c0e9c29270e95e1b2e21c598095f720c199e1e52
@ezyang ezyang added the merged label Jun 26, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants