Skip to content
This repository has been archived by the owner on Jan 24, 2024. It is now read-only.

Revert "[Add Pass] CommonSubexpressionElimination " #1164

Merged
merged 1 commit into from
Jan 30, 2023

Conversation

CtfGo
Copy link
Collaborator

@CtfGo CtfGo commented Jan 29, 2023

Reverts #1116

test_common_subexpression_elimination测试用例随机挂,先revert,待修复后合入,以下是CI流水线中不同pr在该单测出现的两种错误:

  1. 死循环超时 https://xly.bce.baidu.com/paddlepaddle/paddle-cinn/newipipe/detail/7646573/job/21507820

image

  1. 报错退出https://xly.bce.baidu.com/paddlepaddle/paddle-cinn/newipipe/detail/7675541/job/21546552

image

Copy link
Collaborator

@SunNy820828449 SunNy820828449 left a comment

Choose a reason for hiding this comment

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

LGTM

@SunNy820828449 SunNy820828449 merged commit 31cdf07 into develop Jan 30, 2023
@SunNy820828449 SunNy820828449 deleted the revert-1116-dce_pass branch January 30, 2023 02:11
@luotao1
Copy link
Collaborator

luotao1 commented Jan 30, 2023

@zrr1999 请知晓 PR 先被 revert 了。

@zrr1999
Copy link
Member

zrr1999 commented Jan 30, 2023

我在 #1166 里修复了一些可能导致出现错误的bug

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants