Skip to content

Disable large types irgen pass #9452

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

Merged
merged 1 commit into from
May 10, 2017

Conversation

shajrawi
Copy link

@shajrawi shajrawi commented May 10, 2017

radar rdar://problem/32064645

Disable large types irgen pass (for now)

Swift 4.0 PR #9447

@shajrawi
Copy link
Author

@eeckstein Can you please review?

@eeckstein
Copy link
Contributor

LGTM!

@slavapestov
Copy link
Contributor

Will it be re-enabled again in 4.0 branch? A lot of users are looking forward to this.

@shajrawi
Copy link
Author

No.

@jckarter
Copy link
Contributor

Can we keep a flag around so people can try it out, and we can still get bug reports and feedback on it? That's what we did with bridging pch.

@NachoSoto
Copy link
Contributor

😰 Wh.... Why? What's rdar://problem/32064645? What's the point of having a transparent process if you guys keep linking private bugs...? 😕

@shajrawi shajrawi force-pushed the disable_large_types_irgen_pass_master branch from 571e360 to 17effee Compare May 10, 2017 01:58
@shajrawi
Copy link
Author

@jckarter I plan to add such a flag in a later commit on master, but, since this is an ABI change anyone whom wants to test their code with that flag enabled will need to rebuild the entire toolchain with it. else they'd crash when calling the standard library..etc

@shajrawi
Copy link
Author

@swift-ci Please smoke test

@jckarter
Copy link
Contributor

@NachoSoto Looks like there were some bad interactions between this transform and the rest of the optimizer.

@NachoSoto
Copy link
Contributor

Thanks @jckarter! What's the plan moving forward then? I've been waiting for this change for several years, so I was pretty excited.

@shajrawi
Copy link
Author

@NachoSoto we'll get back to this transformation at some point, making sure it is truly great, but it is not going to be re-enabled on the 4.0 branch.

@NachoSoto
Copy link
Contributor

"At some point"?

@shajrawi shajrawi merged commit f9861fe into master May 10, 2017
@shajrawi shajrawi deleted the disable_large_types_irgen_pass_master branch May 10, 2017 04:46
@NachoSoto
Copy link
Contributor

🤷‍♂️

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