-
Notifications
You must be signed in to change notification settings - Fork 10.5k
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
Conversation
@eeckstein Can you please review? |
LGTM! |
Will it be re-enabled again in 4.0 branch? A lot of users are looking forward to this. |
No. |
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. |
😰 Wh.... Why? What's |
571e360
to
17effee
Compare
@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 |
@swift-ci Please smoke test |
@NachoSoto Looks like there were some bad interactions between this transform and the rest of the optimizer. |
Thanks @jckarter! What's the plan moving forward then? I've been waiting for this change for several years, so I was pretty excited. |
@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. |
"At some point"? |
🤷♂️ |
radar rdar://problem/32064645
Disable large types irgen pass (for now)
Swift 4.0 PR #9447