Skip to content

Conversation

@nikic
Copy link
Contributor

@nikic nikic commented Nov 15, 2024

@bogner I currently have you listed as the SDAG maintainer, but I think that nowadays you mostly work on DirectX and aren't actively involved with SDAG anymore, right?

If so, I'd like to nominate new maintainers for this component. My suggestion would @RKSimon and @topperc, if one or both of you are up for it.

@nikic nikic requested review from RKSimon, bogner and topperc November 15, 2024 16:48
@topperc
Copy link
Collaborator

topperc commented Nov 15, 2024

I accept

Copy link
Collaborator

@RKSimon RKSimon left a comment

Choose a reason for hiding this comment

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

No problem, happy to take it on.

Copy link
Collaborator

@topperc topperc left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@bogner bogner left a comment

Choose a reason for hiding this comment

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

Yep, I’ve been negligent on maintaining SelectionDAG for a long time, happy for others to take this on.

@nikic nikic merged commit d49ee7d into llvm:main Nov 20, 2024
9 checks passed
@nikic nikic deleted the sdag-maintainer branch November 20, 2024 09:02
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.

4 participants