<div dir="ltr"><div>Thank you for the guidance Nate! <br></div><div><br></div><div>I'll do my best to contribute ASAP :)</div><div><br></div><div>Cheers,</div><div>Avishay.<br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, 30 Sep 2019 at 20:58, Nathan Ridge <<a href="mailto:zeratul976@hotmail.com">zeratul976@hotmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hi Avishay,<br>
<br>
Thanks for your interesting in contributing to clangd! Yes, this mailing list is a fine place to ask questions of this sort.<br>
<br>
> the feature I miss the most is the macro referencing<br>
<br>
I would find this useful as well.<br>
<br>
> I glanced a over the source code and I think I can write the feature and <br>
> submit it for review, but I am unsure about the overall procedure. <br>
> Where do I submit it to? Who looks over it and who accepts it?<br>
<br>
The procedures followed for clangd are largely the same as those for the larger LLVM community, which are documented here [1]. One thing that's a bit different is clangd has its own issue tracker on Github [2].<br>
<br>
Patches are submitted to LLVM's Phabricator instance [3]. In terms of who to ask for review, you can look at who has reviewed other recent changes to the files you're touching. However, you can also submit a patch without explicitly setting reviewers. I think the clangd team monitors and will notice all review requests with "[clangd]" in the commit message. <br>
<br>
Hope that helps. Feel free to ask more questions if you have them!<br>
<br>
Regards,<br>
Nate<br>
<br>
[1] <a href="http://llvm.org/docs/DeveloperPolicy.html" rel="noreferrer" target="_blank">http://llvm.org/docs/DeveloperPolicy.html</a><br>
[2] <a href="https://github.com/clangd/clangd/issues/" rel="noreferrer" target="_blank">https://github.com/clangd/clangd/issues/</a><br>
[3] <a href="https://reviews.llvm.org/" rel="noreferrer" target="_blank">https://reviews.llvm.org/</a></blockquote></div>