Skip to content

Conversation

@AdUhTkJm
Copy link
Contributor

See issue #1331 for more details.

I'm not sure how to test this change; would it be enough that this does not cause any regressions?

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.

LGTM, should be sufficient to merge.

@bcardosolopes bcardosolopes merged commit a322191 into llvm:main Feb 11, 2025
7 checks passed
lanza pushed a commit that referenced this pull request Mar 18, 2025
See issue #1331 for more details.

I'm not sure how to test this change; would it be enough that this does
not cause any regressions?
terapines-osc-cir pushed a commit to Terapines/clangir that referenced this pull request Sep 2, 2025
See issue llvm#1331 for more details.

I'm not sure how to test this change; would it be enough that this does
not cause any regressions?
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.

3 participants