Skip to content

Sema: Bind context generic params using the current ConstraintSystem's generic environment instead of the decl's DC. #9760

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 19, 2017

Conversation

jckarter
Copy link
Contributor

@jckarter jckarter commented May 19, 2017

Fixes SR-4833.

rdar://problem/32134718

@jckarter
Copy link
Contributor Author

@swift-ci Please smoke test

@jckarter
Copy link
Contributor Author

@slavapestov Does this look good?

@slavapestov
Copy link
Contributor

Looks good to me (and for 4.0 branch as well)

@slavapestov
Copy link
Contributor

Looks like this resolved a compiler crasher too -- nice!

…s generic environment instead of the decl's DC.

Fixes SR-4833.
@jckarter jckarter force-pushed the sibling-nested-generic-call branch from 9ef400d to d16eb33 Compare May 19, 2017 17:48
@jckarter
Copy link
Contributor Author

@swift-ci Please smoke test

1 similar comment
@jckarter
Copy link
Contributor Author

@swift-ci Please smoke test

@jckarter
Copy link
Contributor Author

@ddunbar This llbuild failure on Linux ring a bell?

******************** TEST 'llbuild-unit :: BuildSystem/BuildSystemTests/BuildSystemTaskTests.staleFileRemovalWithManyFiles' FAILED ********************
Note: Google Test filter = BuildSystemTaskTests.staleFileRemovalWithManyFiles
[==========] Running 1 test from 1 test case.
[----------] Global test environment set-up.
[----------] 1 test from BuildSystemTaskTests
[ RUN      ] BuildSystemTaskTests.staleFileRemovalWithManyFiles
/home/buildnode/disk2/workspace/swift-PR-Linux-smoke-test/branch-master/llbuild/unittests/BuildSystem/BuildSystemTaskTests.cpp:732: Failure
Value of: delegate.getMessages()
  Actual: { "commandPreparing(C.1)", "commandStarted(C.1)", "commandFinished(C.1: 0)" }
Expected: std::vector<std::string>({ "commandPreparing(C.1)", "commandStarted(C.1)", "commandFinished(C.1)", })
Which is: { "commandPreparing(C.1)", "commandStarted(C.1)", "commandFinished(C.1)" }
[  FAILED  ] BuildSystemTaskTests.staleFileRemovalWithManyFiles (31 ms)
[----------] 1 test from BuildSystemTaskTests (31 ms total)

[----------] Global test environment tear-down
[==========] 1 test from 1 test case ran. (31 ms total)
[  PASSED  ] 0 tests.
[  FAILED  ] 1 test, listed below:
[  FAILED  ] BuildSystemTaskTests.staleFileRemovalWithManyFiles

 1 FAILED TEST

@aciidgh
Copy link
Contributor

aciidgh commented May 19, 2017

The llbuild failure was just fixed by @neonichu in swiftlang/swift-llbuild#175

@ddunbar
Copy link
Contributor

ddunbar commented May 19, 2017

It was overlapped PRs landing, was fixed in swiftlang/swift-llbuild#175

@ddunbar
Copy link
Contributor

ddunbar commented May 19, 2017

@swift-ci please smoke test linux

@jckarter
Copy link
Contributor Author

Thanks @aciidb0mb3r and @ddunbar!

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.

4 participants