-
Notifications
You must be signed in to change notification settings - Fork 10.3k
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
Remove dependency of SILFunction in IRGenOptions #73727
Conversation
@swift-ci smoke test |
lib/SIL/IR/SILFunction.cpp
Outdated
options->LLVMLTOKind != IRGenLLVMLTOKind::None) { | ||
return false; | ||
} | ||
const auto &silOptions = getModule().getOptions(); |
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.
Why not just add a bool ShouldBePreservedByDebugger
to SILOptions? Then you don’t have to much a bunch of stuff around.
99015cf
to
956668f
Compare
@swift-ci smoke test |
SILFunction::shouldBePreservedForDebugger checks if some optimizations are enabled to decide whether a function should be preserved so its accessible form the debugger or not. Some of these settings used to live only in IRGenOptions making SILFunction depend on IRGenOptions.
956668f
to
4aec4e7
Compare
@swift-ci test |
@swift-ci smoke test |
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.
Thanks!
SILFunction::shouldBePreservedForDebugger checks if some optimizations are enabled to decide whether a function should be preserved so its accessible form the debugger or not. Some of these settings used to live only in IRGenOptions making SILFunction depend on IRGenOptions. This patch duplicates those settings into SILOptions to remove that dependency.