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

[CIR][NFC] remove redundant test in CIR/IR/data-member-ptr.cir #582

Merged
merged 1 commit into from
May 5, 2024

Conversation

Lancern
Copy link
Member

@Lancern Lancern commented May 5, 2024

As suggested in #401, this PR removes the get_global_member test in CIR/IR/data-member-ptr.cir as it is redundant.

The original comment: #401 (comment)

As suggested in llvm#401, this patch removes the `get_global_member` test in
`CIR/IR/data-member-ptr.cir` as it is redundant.
@bcardosolopes bcardosolopes merged commit f690ca9 into llvm:main May 5, 2024
7 checks passed
@Lancern Lancern deleted the nfc-get-data-member-test branch May 6, 2024 03:47
bruteforceboy pushed a commit to bruteforceboy/clangir that referenced this pull request Oct 2, 2024
)

As suggested in llvm#401, this PR removes the `get_global_member` test in
`CIR/IR/data-member-ptr.cir` as it is redundant.

The original comment:
llvm#401 (comment)
Hugobros3 pushed a commit to shady-gang/clangir that referenced this pull request Oct 2, 2024
)

As suggested in llvm#401, this PR removes the `get_global_member` test in
`CIR/IR/data-member-ptr.cir` as it is redundant.

The original comment:
llvm#401 (comment)
keryell pushed a commit to keryell/clangir that referenced this pull request Oct 19, 2024
)

As suggested in llvm#401, this PR removes the `get_global_member` test in
`CIR/IR/data-member-ptr.cir` as it is redundant.

The original comment:
llvm#401 (comment)
lanza pushed a commit that referenced this pull request Nov 5, 2024
As suggested in #401, this PR removes the `get_global_member` test in
`CIR/IR/data-member-ptr.cir` as it is redundant.

The original comment:
#401 (comment)
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