[LLVM] Update inlining maintainers #120579
Conversation
|
@chandlerc On a semi-related note, you are also listed as the maintainer of SROA/Mem2Reg. I don't really have a replacement for that area, so I can either just leave you listed there, or drop that category entirely... |
|
I'd suggest adding @kazutakahirata as well. lgtm for me. |
There was a problem hiding this comment.
LGTM!
@chandlerc On a semi-related note, you are also listed as the maintainer of SROA/Mem2Reg. I don't really have a replacement for that area, so I can either just leave you listed there, or drop that category entirely...
From recent changes, you or @aeubanks might be OK?
Going back further, both @LebedevRI and @arsenm have made substantial changes to SROA.
It's still near and dear to my heart, but definitely good to have a more current and active maintainer. =D
nikic
force-pushed
the
inlining-maintainer
branch
2 times, most recently
from
726ed10 to
5e1832e
Compare
Done. I've added "module inliner and inline order" as the focus area -- does that sound correct? |
|
Ping @kazutakahirata, to confirm whether you want to be listed as a maintainer for inlining. |
Yes, thanks! |
nikic
force-pushed
the
inlining-maintainer
branch
from
5e1832e to
9f66c3a
Compare
We currently list @chandlerc as the maintainer for inlining, but I don't think he works on that area nowadays.
I'd like to nominate @aeubanks and @mtrofin as new maintainers for inlining. I'm also adding @aeubanks as the maintainer for NewPM/CGCSS/LazyCallGraph, which are quite closely intertwined with inlining.