Skip to content

Fix issue mutating if expressions - #2601

Merged
tqchen merged 1 commit into
apache:masterfrom
jroesch:misc-bug-fix
Feb 18, 2019
Merged

tqchen merged 1 commit into
apache:masterfrom
jroesch:misc-bug-fix

Conversation

@jroesch

@jroesch jroesch commented Feb 15, 2019

Copy link
Copy Markdown
Member

Fixes a small bug in the Python impl of the expression mutator.

@tqchen

tqchen commented Feb 15, 2019

Copy link
Copy Markdown
Member

Please add a regression test as per https://docs.tvm.ai/contribute/code_review.html#ensure-test-coverage

@icemelon icemelon added the status: need update need update based on feedbacks label Feb 15, 2019
@tqchen

tqchen commented Feb 16, 2019

Copy link
Copy Markdown
Member

@ZihengJiang @yzhliu we might need to consider cherry-pick this patch after release tag

@tqchen
tqchen merged commit f6be4d6 into apache:master Feb 18, 2019
@tqchen

tqchen commented Feb 18, 2019

Copy link
Copy Markdown
Member

This is merged because we need to cherry pick it for the 0.5 release. @jroesch please send a test case in a followup PR

@tqchen tqchen added status: accepted and removed status: need update need update based on feedbacks labels Feb 18, 2019
ZihengJiang pushed a commit that referenced this pull request Feb 18, 2019
ZihengJiang pushed a commit that referenced this pull request Feb 18, 2019
wweic pushed a commit to neo-ai/tvm that referenced this pull request Feb 20, 2019
wweic pushed a commit to neo-ai/tvm that referenced this pull request Feb 20, 2019
@yzhliu yzhliu mentioned this pull request Mar 2, 2019
9 of 28 tasks
@jroesch
jroesch deleted the misc-bug-fix branch February 4, 2021 04:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants