Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

inference: eliminate more potential widenconst(::TypeofVararg) cases #42971

Merged
merged 1 commit into from
Nov 7, 2021

Conversation

aviatesk
Copy link
Member

@aviatesk aviatesk commented Nov 6, 2021

Follows #42583.
The missing case within const_prop_function_heuristic was originally
reported from RerationalAI's test suite. I added the other two cases by
some code review.

@aviatesk aviatesk requested a review from vtjnash November 6, 2021 10:22
@Sacha0
Copy link
Member

Sacha0 commented Nov 6, 2021

Thanks so much Shuhei! Will give this a whirl and report back :).

@aviatesk
Copy link
Member Author

aviatesk commented Nov 6, 2021

yeah, that should be really helpful !

Follows #42583.
The missing case within `const_prop_function_heuristic` was originally
reported from RationalAI's test suite. I added the other two cases by
some code review.
@aviatesk aviatesk added the merge me PR is reviewed. Merge when all tests are passing label Nov 7, 2021
@aviatesk aviatesk merged commit 71d57d9 into master Nov 7, 2021
@aviatesk aviatesk deleted the avi/morevararg branch November 7, 2021 13:38
@DilumAluthge DilumAluthge removed the merge me PR is reviewed. Merge when all tests are passing label Nov 7, 2021
KristofferC pushed a commit that referenced this pull request Nov 8, 2021
#42971)

Follows #42583.
The missing case within `const_prop_function_heuristic` was originally
reported from RerationalAI's test suite. I added the other two cases by
some code review.

(cherry picked from commit 71d57d9)
LilithHafner pushed a commit to LilithHafner/julia that referenced this pull request Feb 22, 2022
JuliaLang#42971)

Follows JuliaLang#42583.
The missing case within `const_prop_function_heuristic` was originally
reported from RerationalAI's test suite. I added the other two cases by
some code review.
LilithHafner pushed a commit to LilithHafner/julia that referenced this pull request Mar 8, 2022
JuliaLang#42971)

Follows JuliaLang#42583.
The missing case within `const_prop_function_heuristic` was originally
reported from RerationalAI's test suite. I added the other two cases by
some code review.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants