[llvm] [NFC][LLVM] Add RealtimeSanitizer LLVM code owners (PR #101231)

Chris Apple via llvm-commits llvm-commits at lists.llvm.org
Wed Jul 31 13:06:56 PDT 2024


cjappl wrote:

> > > > CC Reviewers @vitalybuka @MaskRay @davidtrevelyan
> > > > Because we don't have commiter access, not sure if this should just be closed! I defer to the reviewers if adding this in is appropriate or not.
> > > > [>
> > > > Thanks as always.
> > > 
> > > 
> > > I don't know if this process is still relevant https://llvm.org/docs/DeveloperPolicy.html#obtaining-commit-access
> > 
> > 
> > Thanks for that link @vitalybuka! Do you think it's appropriate at this stage to apply for commit access? I have a couple commits, and only one large one (the compiler-rt work for rtsan). With a few additional big changes needed for the first version of rtsan.
> > I defer to your expertise, seeing as you're the one I've interacted the most with on PRs. I do not want to apply "too early" if it is inappropriate.
> > Thank you for any advice.
> 
> https://discourse.llvm.org/t/rfc-new-criteria-for-commit-access/76290/81 Looks like LLVM it trying to limit the number of commiters https://discourse.llvm.org/t/rfc-new-criteria-for-commit-access/76290 And LLVM will likely soon going to have better commiter access requesting process.
> 
> We can try proceed as-is, without commiter, and you can apply when you think access is beneficial to you. I assume wrong review assignees will figure out how to contact you if needed. I suspect some OWNERS already lost commiter access.

Sounds good. I think perhaps once the main pieces of rtsan have landed and are stable, that would be a good time to apply. 

https://github.com/llvm/llvm-project/pull/101231


More information about the llvm-commits mailing list