Skip to content

[polymorphic-builtin] Rather than asserting in IRGen if we see a buil… #27302

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

gottesmm
Copy link
Contributor

…tin that was not specialized, turn the builtin into a trap.

I forgot about this part of the design when I was working on this. To ensure
that the whole design works as expected, I included a small end-to-end test
using an experimental design for simd that uses polymorphic builtins that test
this functionally.

NOTE: The experimental design is only intended to exercise the code functionally.

rdar://48248417

@gottesmm
Copy link
Contributor Author

@swift-ci smoke test

…tin that was not specialized, turn the builtin into a trap.

I forgot about this part of the design when I was working on this. To ensure
that the whole design works as expected, I included a small end-to-end test
using an experimental design for simd that uses polymorphic builtins that test
this functionally.

NOTE: The experimental design is only intended to exercise the code functionally.

rdar://48248417
@gottesmm gottesmm force-pushed the pr-7a861faa03ec80b01ad9a212afe79e529fb6857e branch from 462fbc8 to b9046c6 Compare September 23, 2019 20:51
@gottesmm
Copy link
Contributor Author

@swift-ci smoke test and merge

2 similar comments
@gottesmm
Copy link
Contributor Author

@swift-ci smoke test and merge

@gottesmm
Copy link
Contributor Author

@swift-ci smoke test and merge

@swift-ci swift-ci merged commit c3d304c into swiftlang:master Sep 23, 2019
@gottesmm gottesmm deleted the pr-7a861faa03ec80b01ad9a212afe79e529fb6857e branch September 23, 2019 22:14
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