This is an archive of the discontinued LLVM Phabricator instance.

[AVR][NFC] Change the owner of the AVR backend
ClosedPublic

Authored by benshi001 on May 14 2023, 2:57 AM.

Details

Diff Detail

Event Timeline

benshi001 created this revision.May 14 2023, 2:57 AM
Herald added a project: Restricted Project. · View Herald TranscriptMay 14 2023, 2:57 AM
Herald added a subscriber: Jim. · View Herald Transcript
benshi001 requested review of this revision.May 14 2023, 2:57 AM
Herald added a project: Restricted Project. · View Herald TranscriptMay 14 2023, 2:57 AM

LGTM after we see some support on https://discourse.llvm.org/t/apply-for-the-ownership-of-the-avr-backend/70624 . Please also add the link to the description.

benshi001 edited the summary of this revision. (Show Details)May 14 2023, 5:28 PM
benshi001 edited the summary of this revision. (Show Details)

LGTM after we see some support on https://discourse.llvm.org/t/apply-for-the-ownership-of-the-avr-backend/70624 . Please also add the link to the description.

Dylan has agreed in our private disscussion. And he will confirm via https://discourse.llvm.org/t/apply-for-the-ownership-of-the-avr-backend/70624

LGTM after we see some support on https://discourse.llvm.org/t/apply-for-the-ownership-of-the-avr-backend/70624 . Please also add the link to the description.

Dylan has agreed in our private disscussion. And he will confirm via https://discourse.llvm.org/t/apply-for-the-ownership-of-the-avr-backend/70624

I wonder whether Dylan has lost access to Phabricator? It would be good for Dylan to accept this patch as well.
We can help restore the password if needed...

MaskRay accepted this revision.May 16 2023, 5:57 PM
This revision is now accepted and ready to land.May 16 2023, 5:57 PM
lattner accepted this revision.May 21 2023, 5:29 PM
lattner added a subscriber: lattner.

Makes sense to me, thank you!

This revision was automatically updated to reflect the committed changes.

Thanks for all of your trusts !