Skip to content

[flang][common] return ENUM_CLASS names definition to original state #143553

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

Merged
merged 1 commit into from
Jun 10, 2025

Conversation

akuhlens
Copy link
Contributor

@akuhlens akuhlens commented Jun 10, 2025

This PR simply reverts a few lines in bf60aa1 to their state in bcba39a so that they are constant for some of the build tests that require it. This should fix the breakage caused by #142022.

@llvmbot llvmbot added the flang Flang issues not falling into any other category label Jun 10, 2025
@akuhlens akuhlens requested review from jplehr and klausler June 10, 2025 15:27
@akuhlens
Copy link
Contributor Author

Any suggestion on how to test that the PR actually fixes the broken build bots?

@klausler
Copy link
Contributor

Any suggestion on how to test that the PR actually fixes the broken build bots?

Only one way to be sure.

@Kewen12
Copy link
Contributor

Kewen12 commented Jun 10, 2025

Hello! May I ask if we are going to merge this fix any time soon?

@akuhlens akuhlens merged commit d502c68 into llvm:main Jun 10, 2025
9 checks passed
rorth pushed a commit to rorth/llvm-project that referenced this pull request Jun 11, 2025
…lvm#143553)

This PR simply reverts a few lines in
bf60aa1 to their state in
bcba39a so that they are constant for
some of the build tests that require it. This should fix the breakage
caused by llvm#142022.
tomtor pushed a commit to tomtor/llvm-project that referenced this pull request Jun 14, 2025
…lvm#143553)

This PR simply reverts a few lines in
bf60aa1 to their state in
bcba39a so that they are constant for
some of the build tests that require it. This should fix the breakage
caused by llvm#142022.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
flang Flang issues not falling into any other category
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants