Skip to content

Conversation

@JamesL425
Copy link
Contributor

No description provided.

Copy link
Member

@Lancern Lancern left a comment

Choose a reason for hiding this comment

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

Thanks for working on this! LGTM.

@Lancern
Copy link
Member

Lancern commented Feb 24, 2025

The tests are failing.

@JamesL425
Copy link
Contributor Author

Sorry I forgot to change back a file. Should be sorted now!

@JamesL425 JamesL425 requested a review from Lancern February 25, 2025 14:07
@Lancern
Copy link
Member

Lancern commented Feb 25, 2025

CI jobs started. Let's see if it would turn green!

@JamesL425 JamesL425 requested a review from Lancern February 25, 2025 14:56
@github-actions
Copy link

github-actions bot commented Feb 25, 2025

✅ With the latest revision this PR passed the C/C++ code formatter.

@JamesL425
Copy link
Contributor Author

Just made a fix (and a clang format fix), fingers crossed it works now!

Copy link
Member

@Lancern Lancern left a comment

Choose a reason for hiding this comment

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

Thanks, LGTM!

@JamesL425 JamesL425 closed this Feb 25, 2025
@JamesL425 JamesL425 reopened this Feb 25, 2025
@JamesL425
Copy link
Contributor Author

@Lancern I think this one is good to merge?

@Lancern
Copy link
Member

Lancern commented Feb 26, 2025

Merging this!

@Lancern Lancern merged commit 89293bf into llvm:main Feb 26, 2025
9 checks passed
xlauko pushed a commit to xlauko/clangir that referenced this pull request Feb 28, 2025
xlauko pushed a commit to xlauko/clangir that referenced this pull request Feb 28, 2025
lanza pushed a commit that referenced this pull request Mar 18, 2025
terapines-osc-cir pushed a commit to Terapines/clangir that referenced this pull request Sep 2, 2025
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.

2 participants