Skip to content

[passmanager] Move SILOptimizerRequests.cpp into ./lib/SILOptimizer/PassManager/* #30571

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

I am doing this move for two reasons:

  1. This file only works directly with parts of the PassManager.
  2. In the SILOptimizer, we do not have files in the top level of the file tree
    and instead group them in one of the subject subfolders.

…assManager/*

I am doing this move for two reasons:

1. This file only works directly with parts of the PassManager.
2. In the SILOptimizer, we do not have files in the top level of the file tree
and instead group them in one of the subject subfolders.
@gottesmm gottesmm requested a review from atrick March 22, 2020 23:23
@gottesmm
Copy link
Contributor Author

@swift-ci smoke test

@gottesmm
Copy link
Contributor Author

@swift-ci smoke test OS X platform

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.

thanks

@gottesmm
Copy link
Contributor Author

@swift-ci smoke test os x platform

2 similar comments
@gottesmm
Copy link
Contributor Author

@swift-ci smoke test os x platform

@gottesmm
Copy link
Contributor Author

@swift-ci smoke test os x platform

@gottesmm
Copy link
Contributor Author

@swift-ci clean smoke test OS X platform

2 similar comments
@gottesmm
Copy link
Contributor Author

@swift-ci clean smoke test OS X platform

@gottesmm
Copy link
Contributor Author

@swift-ci clean smoke test OS X platform

@gottesmm gottesmm merged commit 51bc44e into swiftlang:master Mar 23, 2020
@gottesmm gottesmm deleted the pr-10e12f0db62b5bcf06e30b8e9ad3da073f2213e6 branch March 23, 2020 03:53
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