Skip to content

Conversation

Lancern
Copy link
Member

@Lancern Lancern commented Feb 23, 2025

The CIRGen support is already there. This PR adds LLVM lowering support for comparisons between pointers to member functions. Note that pointers to member functions could only be compared for equality.

Copy link
Member

@bcardosolopes bcardosolopes left a comment

Choose a reason for hiding this comment

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

LGTM after nit

The CIRGen support is already there. This patch adds LLVM lowering support for
comparisons between pointers to member functions. Note that pointers to member
functions could only be compared for equality.
@Lancern Lancern merged commit aff6245 into llvm:main Feb 25, 2025
6 checks passed
@Lancern Lancern deleted the memfunc-ptr-cmp branch February 25, 2025 06:11
lanza pushed a commit that referenced this pull request Mar 18, 2025
#1390)

The CIRGen support is already there. This PR adds LLVM lowering support
for comparisons between pointers to member functions. Note that pointers
to member functions could only be compared for equality.
terapines-osc-cir pushed a commit to Terapines/clangir that referenced this pull request Sep 2, 2025
llvm#1390)

The CIRGen support is already there. This PR adds LLVM lowering support
for comparisons between pointers to member functions. Note that pointers
to member functions could only be compared for equality.
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