Skip to content

Remove InplaceVariables and the problems they cause

Javier Duarte requested to merge github/fork/jmitrevs/remove_inplace_variable into master

Created by: jmitrevs

This attempts to fix the problem that removing nodes can cause to InplaceVariables by removing InplaceVariables altogether. These variables are only used for the Reshape layar. In io_stream we already replace Reshape nodes with Repack nodes; this pull request additionally removes those nodes in io_parallel (and flatten), changing the shape of the preceding variable. (We may want to rename the ReshapeStream optimizer pass to reflect on the additional duties it performs.)

I added a basic test, though it does not test the flatten case.

@vloncar, is this the direction we should go?

Merge request reports