Skip to content

[passmanager] Change SIL pass pipeline plan to use an LLVM YAML representation. #29111

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

This eliminates a bunch of code and will make it significantly easier to
maintain/add to this code/use this code.

Just had this on a branch. This should make it really easy to fuzz the optimizer by manipulating the pass pipeline. I wonder if we could even use a libfuzzer based approach.

@gottesmm gottesmm requested a review from atrick January 10, 2020 03:57
@gottesmm
Copy link
Contributor Author

@swift-ci smoke test and merge

@gottesmm
Copy link
Contributor Author

@swift-ci test linux platform

@swift-ci
Copy link
Contributor

Build failed
Swift Test Linux Platform
Git Sha - 33ae6ec67866be92e1fc559ec1750c64d044f185

@gottesmm
Copy link
Contributor Author

I want to fix a couple of things in this area. So I am going to fix this real quick.

@gottesmm gottesmm force-pushed the pr-d5a69902a451af42884c0e9cc4b6f18ecf246ada branch from 33ae6ec to b69615b Compare March 11, 2020 19:19
…sentation.

This eliminates a bunch of code and will make it significantly easier to
maintain/add to this code/use this code.
@gottesmm gottesmm force-pushed the pr-d5a69902a451af42884c0e9cc4b6f18ecf246ada branch from b69615b to 52c5f72 Compare March 11, 2020 21:14
@gottesmm
Copy link
Contributor Author

@swift-ci smoke test

@gottesmm gottesmm merged commit e976aa9 into swiftlang:master Mar 11, 2020
@gottesmm gottesmm deleted the pr-d5a69902a451af42884c0e9cc4b6f18ecf246ada branch March 11, 2020 23:11
Copy link
Contributor

@atrick atrick left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks simpler to me!

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.

3 participants