Skip to content

Conversation

@smessmer
Copy link
Contributor

@smessmer smessmer commented Jul 3, 2019

Stack:
    :white_circle:  #22239 Open up AliasAnalysisKind for any ops  💛
    :white_circle:  #22175 AliasAnalysisKind::CONSERVATIVE/FROM_SCHEMA  💛
    :white_circle:  #22520 Don't use DeadCodeElimination from LowerAllTuples  💛
    :black_circle:  #22499 Another dce fix  💛

Another place where onnx export is running dead code elimination after making the jit graph invalid. Fixing it.

Differential Revision: D16111969

Differential Revision: D16111969
Differential Version: 86024667
@pytorchbot pytorchbot added oncall: jit Add this issue/PR to JIT oncall triage queue module: onnx Related to torch.onnx labels Jul 3, 2019
@smessmer smessmer requested review from dzhulgakov and houseroad July 3, 2019 19:33
Differential Revision: D16111969
Differential Version: 86025315
Copy link
Member

@houseroad houseroad left a comment

Choose a reason for hiding this comment

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

LGTM

@facebook-github-bot
Copy link
Contributor

This pull request has been merged in 2732a5e.

xzhu1900 pushed a commit to xzhu1900/pytorch that referenced this pull request Jul 5, 2019
Summary:
Pull Request resolved: pytorch#22499

Another place where onnx export is running dead code elimination after making the jit graph invalid. Fixing it.

Reviewed By: houseroad

Differential Revision: D16111969

fbshipit-source-id: 5ba80340c06d091988858077f142ea4e3da0638c
@ezyang ezyang deleted the export-D16111969 branch July 19, 2019 15:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Merged module: onnx Related to torch.onnx 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