Skip to content

Add option to force disabling debug info#4645

Merged
llvm-beanz merged 2 commits into
microsoft:mainfrom
llvm-beanz:cbieneman/opt-disable-di
Sep 16, 2022
Merged

Add option to force disabling debug info#4645
llvm-beanz merged 2 commits into
microsoft:mainfrom
llvm-beanz:cbieneman/opt-disable-di

Conversation

@llvm-beanz
Copy link
Copy Markdown
Collaborator

This is (admittedly) a little hacky. DXC spends a lot of compile time updating debug information because we always generate it so that we can generate diagnostics for late-running validation.

This flag disables that validation resulting in about a 15% compile time improvement on a pessimistic inliner test.

This is (admittedly) a little hacky. DXC spends a lot of compile time
updating debug information because we always generate it so that we can
generate diagnostics for late-running validation.

This flag disables that validation resulting in about a 15% compile
time improvement on a pessimistic inliner test.
Comment thread include/dxc/Support/HLSLOptions.td Outdated
@AppVeyorBot
Copy link
Copy Markdown

@adam-yang
Copy link
Copy Markdown
Contributor

What happens you give -Zi and -fdisable-debug-info together?

@llvm-beanz
Copy link
Copy Markdown
Collaborator Author

If you pass -Zi -fdisable-debug-info is ignored.

@AppVeyorBot
Copy link
Copy Markdown

Copy link
Copy Markdown
Collaborator

@pow2clk pow2clk left a comment

Choose a reason for hiding this comment

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

Looks good

@llvm-beanz llvm-beanz merged commit b49ecd1 into microsoft:main Sep 16, 2022
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.

4 participants