Skip to content
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

fix Issue 15047 - "used before set" error with -O #6074

Closed
wants to merge 7 commits into from

Conversation

Cauterite
Copy link
Contributor

Currently there is a case where the optimiser is able to detect an error that is not otherwise detected. This results in a compilation error only when the -O flag is used.
If code is buildable with -O it should be buildable without -O. Therefore, the optimiser now emits a warning instead.

@dlang-bot
Copy link
Contributor

Fix Bugzilla Description
15047 "used before set" error with void initialization (only with -O)

@dlang-bot
Copy link
Contributor

@Cauterite, thanks for your PR! By analyzing the annotation information on this pull request, we identified @yebblies, @CyberShadow and @9rnsr to be potential reviewers. @yebblies: The PR was automatically assigned to you, please reassign it if you were identified mistakenly.

(The DLang Bot is under development. If you experience any issues, please open an issue at its repo.)

@Cauterite
Copy link
Contributor Author

Currently the -w flag is broken with ddoc. If ddoc prints warnings, compilation continues regardless of the -w flag.

With that fixed, certain ddoc tests are failing because they trigger warnings and specify the -w flag.

@Cauterite
Copy link
Contributor Author

Cauterite commented Aug 23, 2016

Looks like fixing the -w flag has a much wider effect than I had expected. Therefore I'm going to split this pull request into two.

TLDR: -w is a very broken feature right now.

@Cauterite Cauterite closed this Aug 23, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants