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

scalac show-phases prerecheck twice #21939

Closed
Sparkaar opened this issue Nov 13, 2024 · 1 comment · Fixed by #21942
Closed

scalac show-phases prerecheck twice #21939

Sparkaar opened this issue Nov 13, 2024 · 1 comment · Fixed by #21942
Labels
area:compiler-phases Issues tied with configuration and division of compiler phases. area:settings Issues tied to command line options & settings. itype:bug

Comments

@Sparkaar
Copy link

Compiler version

Scala compiler version 3.5.2 -- Copyright 2002-2024, LAMP/EPFL

Minimized example

scalac -Xshow-phases

Output

Output shortened for readability

             preRecheck  preRecheck
                recheck  recheck
             preRecheck  preRecheck
                     cc  cc

Expectation

The second preRecheck phase seems off, should probably rather be postRecheck

@Sparkaar Sparkaar added the stat:needs triage Every issue needs to have an "area" and "itype" label label Nov 13, 2024
@Gedochao
Copy link
Contributor

scalac -Xshow-phases|grep preRecheck                        
#             preRecheck  preRecheck
#             preRecheck  preRecheck

indeed 🤨

@Gedochao Gedochao added itype:bug area:settings Issues tied to command line options & settings. area:compiler-phases Issues tied with configuration and division of compiler phases. and removed stat:needs triage Every issue needs to have an "area" and "itype" label labels Nov 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:compiler-phases Issues tied with configuration and division of compiler phases. area:settings Issues tied to command line options & settings. itype:bug
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants