-
Notifications
You must be signed in to change notification settings - Fork 13.6k
[NewPM] Remove CFGPrinterLegacyPass #73414
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
[NewPM] Remove CFGPrinterLegacyPass #73414
Conversation
@llvm/pr-subscribers-llvm-analysis Author: Aiden Grossman (boomanaiden154) ChangesThis pass has no test coverage in upstream LLVM, is not used anywhere in upstream LLVM, and has a NewPM equivalent. For these reasons, remove it. Full diff: https://github.com/llvm/llvm-project/pull/73414.diff 4 Files Affected:
|
✅ With the latest revision this PR passed the C/C++ code formatter. |
This pass has no test coverage in upstream LLVM, is not used anywhere in upstream LLVM, and has a NewPM equivalent. For these reasons, remove it.
55e97c5
to
dcfe328
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Local branch amd-gfx 7bc40bb Merged main:1449b349ac40 into amd-gfx:ba7e2d1a3aa2 Remote branch main 9ba74c2 [NewPM] Remove CFGPrinterLegacyPass (llvm#73414)
This reverts commit 9ba74c2. Continue to support CFGPrinterLegacyPass for LGC, until it has been converted to new pass manager. Change-Id: I147a44afa8a011bbed28490de1f4dfb564c2fb58
Local branch amd-gfx 2b56631 Revert "[NewPM] Remove CFGPrinterLegacyPass (llvm#73414)" Remote branch main 6318dd8 [mlir] Fix a warning
This pass has no test coverage in upstream LLVM, is not used anywhere in upstream LLVM, and has a NewPM equivalent. For these reasons, remove it.