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

Add eager and lazy future support. #114

Merged
merged 1 commit into from
Apr 14, 2015

Conversation

albertnetymk
Copy link
Contributor

It seems that thread local variables are not updated properly on mac,
which might be a bug,
filed anyway.

Currently, on mac, only one OS thread is used.

Test result on my box (Linux):
Tests passed: 53/58
Tests failed: 5/58
await
deadlock_yourself
parametricClasses
parametricPrint
suspend

It seems that thread local variables are not updated properly on mac,
which might be a [bug](https://llvm.org/bugs/show_bug.cgi?id=23154),
filed anyway.

Currently, on mac, only one OS thread is used.
@kikofernandez kikofernandez self-assigned this Apr 13, 2015
@kikofernandez
Copy link
Contributor

I get the same tests results on my Mac. I would like to get an overview of the changes with you, since there are many interesting stuff happening in the scheduler.

On another note, not really related to this commit: I believe we need to re-think where we are going. We keep breaking stuff in Mac and continue with a "I'll solve it later". If we continue with this approach, our baggage will grow up to a point where we will not be able to support Mac (this is not related to the commit, or you, but to the way we are doing things ever since we moved to the new RT).

Let me know when you have some time to give me an overview of your changes.

@albertnetymk
Copy link
Contributor Author

Good to know. After lunch would be great. As for the mac breaking issues, we could discuss it more in the morning meeting.

kikofernandez pushed a commit that referenced this pull request Apr 14, 2015
@kikofernandez kikofernandez merged commit 26fa3c3 into master Apr 14, 2015
@kikofernandez kikofernandez deleted the features/eager-lazy-futures branch April 14, 2015 11:47
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.

2 participants