Skip to content

Fix code generation for new ScalarList type#2816

Merged
ailzhang merged 1 commit into
pytorch:masterfrom
wenleix:scalar_list
Mar 8, 2021
Merged

Fix code generation for new ScalarList type#2816
ailzhang merged 1 commit into
pytorch:masterfrom
wenleix:scalar_list

Conversation

@wenleix

@wenleix wenleix commented Mar 8, 2021

Copy link
Copy Markdown
Contributor

We are going to introduce ScalarList = ArrayRef in PyTorch.

See pytorch/pytorch#53470 for context.

We are going to introduce ScalarList = ArrayRef<Scalar> in PyTorch.
@wenleix

wenleix commented Mar 8, 2021

Copy link
Copy Markdown
Contributor Author

cc @smessmer

@JackCaoG JackCaoG requested a review from ailzhang March 8, 2021 19:00

@ailzhang ailzhang left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks!

@ailzhang ailzhang merged commit e537eed into pytorch:master Mar 8, 2021
@wenleix wenleix deleted the scalar_list branch March 8, 2021 23:42
wenleix added a commit to wenleix/xla that referenced this pull request Mar 30, 2021
This reverts commit e537eed.

We decide not to introduce ScalarList as an alias for ArrayRef<Scalar>
but to use ArrayRef<Scalar> directly in
pytorch/pytorch#53583.

The original commit is no longer needed.
wenleix added a commit that referenced this pull request Mar 31, 2021
This reverts commit e537eed.

We decide not to introduce ScalarList as an alias for ArrayRef<Scalar>
but to use ArrayRef<Scalar> directly in
pytorch/pytorch#53583.

The original commit is no longer needed.
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.

2 participants