Thanks to visit codestin.com
Credit goes to github.com

Skip to content

[jit] refactor register_prim_ops#21001

Closed
wanchaol wants to merge 1 commit into
gh/wanchaol/10/basefrom
gh/wanchaol/10/head
Closed

[jit] refactor register_prim_ops#21001
wanchaol wants to merge 1 commit into
gh/wanchaol/10/basefrom
gh/wanchaol/10/head

Conversation

@wanchaol
Copy link
Copy Markdown
Collaborator

@wanchaol wanchaol commented May 28, 2019

Stack from ghstack:

Differential Revision: D15523445

@pytorchbot pytorchbot added the oncall: jit Add this issue/PR to JIT oncall triage queue label May 28, 2019
@wanchaol wanchaol requested review from driazati, eellison, suo and zdevito May 28, 2019 18:57
Copy link
Copy Markdown
Contributor

@driazati driazati left a comment

Choose a reason for hiding this comment

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

Seems fine, though it's all temporary until we land something along the lines of #19373, bind all these as custom ops, and delete all these macros.

#19373 was blocked on removing assumptions that prim:: and aten:: ops are pure unless annotated otherwise (they should match custom ops, which are assumed to be side-effectful unless specified)

@suo
Copy link
Copy Markdown
Member

suo commented May 28, 2019

@driazati the purity thing is fixed btw so that PR is no longer blocked

@facebook-github-bot
Copy link
Copy Markdown
Contributor

@wanchaol merged this pull request in 0885dd2.

laurentdupin pushed a commit to laurentdupin/pytorch that referenced this pull request Apr 24, 2026
Summary:
Pull Request resolved: pytorch#21001
ghimport-source-id: f1b8e39

Differential Revision: D15523445

Pulled By: wanchaol

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

Labels

Merged oncall: jit Add this issue/PR to JIT oncall triage queue

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants