Skip to content

Remove -swift-version 3 checks for "expression too complex". #11338

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
Aug 4, 2017
Merged

Remove -swift-version 3 checks for "expression too complex". #11338

merged 1 commit into from
Aug 4, 2017

Conversation

rudkx
Copy link
Contributor

@rudkx rudkx commented Aug 4, 2017

This was initially added to avoid "expression was too complex" in a
case where we were not previously reporting it for -swift-version 3
but should have been. In retrospect this seems misguided since
although we would not like to regress on "too complex" expressions, we
really don't want to silently continue in the cases where we decide an
expression is "too complex", but where we have a solution that we
could use. It's better to fail.

@rudkx
Copy link
Contributor Author

rudkx commented Aug 4, 2017

@swift-ci Please smoke test and merge

This was initially added to avoid "expression was too complex" in a
case where we were not previously reporting it for -swift-version 3
but should have been. In retrospect this seems misguided since
although we would not like to regress on "too complex" expressions, we
really don't want to silently continue in the cases where we decide an
expression is "too complex", but where we have a solution that we
could use. It's better to fail.
@rudkx
Copy link
Contributor Author

rudkx commented Aug 4, 2017

@swift-ci Please smoke test and merge

@rudkx
Copy link
Contributor Author

rudkx commented Aug 4, 2017

@swift-ci Please smoke test

@rudkx
Copy link
Contributor Author

rudkx commented Aug 4, 2017

Hmmm, Linux failed with:

note: testing using 'lldb': /home/buildnode/jenkins/workspace/swift-PR-Linux-smoke-test@3/branch-master/buildbot_linux/none-swift_package_sandbox_linux-x86_64/usr/bin/lldb
lit.py: /home/buildnode/jenkins/workspace/swift-PR-Linux-smoke-test@3/branch-master/swift-integration-tests/lit.cfg:180: fatal: lldb does not exist!
/home/buildnode/jenkins/workspace/swift-PR-Linux-smoke-test@3/branch-master/swift/utils/build-script: fatal error: command terminated with a non-zero exit status 2, aborting
/home/buildnode/jenkins/workspace/swift-PR-Linux-smoke-test@3/branch-master/swift/utils/build-script: fatal error: command terminated with a non-zero exit status 1, aborting
Build step 'Execute shell' marked build as failure

@rudkx
Copy link
Contributor Author

rudkx commented Aug 4, 2017

@swift-ci Please smoke test Linux platform

@rudkx
Copy link
Contributor Author

rudkx commented Aug 4, 2017

@swift-ci Please clean smoke test Linux platform

@gottesmm
Copy link
Contributor

gottesmm commented Aug 4, 2017

@rudkx I think lldb has broken PR testing now.

@rudkx rudkx merged commit 0b173de into swiftlang:master Aug 4, 2017
@rudkx rudkx deleted the bail-on-too-complex branch August 4, 2017 21:45
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