[llvm-dev] Submitting patches for LLVM -- llvm-commits vs. Phabricator?

[Christoph Kindl via llvm-dev]

Hi,


> On 03 Jan 2018, at 18:40, Adrian Prantl wrote:
> 
> 
> 
>> On Jan 2, 2018, at 11:31 AM, Christoph Kindl via llvm-dev <llvm-dev at lists.llvm.org <mailto:llvm-dev at lists.llvm.org>> wrote:
>> 
>> Hi,
>> 
>>> Date: Sat, 30 Dec 2017 09:59:56 -0600
>>> From: Krzysztof Parzyszek via llvm-dev <llvm-dev at lists.llvm.org <mailto:llvm-dev at lists.llvm.org>>
>>> To: llvm-dev at lists.llvm.org <mailto:llvm-dev at lists.llvm.org>
>>> Subject: Re: [llvm-dev] Submitting patches for LLVM -- llvm-commits
>>> 	vs. Phabricator?
>>> 
>>> Hi,
>>> The current practice is to upload a patch to phabricator and add 
>>> llvm-commits as a subscriber. This will send an email to the mailing 
>>> list. You can (and should) add some reviewers too.
>>> 
>>> -Krzysztof
>> 
>>> On 31 Dec 2017, at 04:49, Shoaib Meenai wrote:
>>> 
>>> Yup, Phabricator is generally preferred for patches.
>>>  
>>> Additionally, are you subscribed to the mailing list? I can't find where I read it now, but I believe your messages are held for moderation if you aren't subscribed. You can subscribe at http://lists.llvm.org/mailman/listinfo/llvm-commits <http://lists.llvm.org/mailman/listinfo/llvm-commits> if needed.
>> 
>> Thanks guys! This is my first attempt to submit a patch; I've sent it to llvm-commits because the Developer Guide mentions this as preferred method:
>> 
>>> Once your patch is ready, submit it by emailing it to the appropriate project’s commit mailing list (or commit it directly if applicable). Alternatively, some patches get sent to the project’s development list or component of the LLVM bug tracker, but the commit list is the primary place for reviews and should generally be preferred.
>> 
>> I guess this part should be updated then to mention Phabricator as the primary place for submitting and reviewing patches?
> 
> Would you mind updating the section and uploading the patch to reviews.llvm.org <http://reviews.llvm.org/> :-) ?
> -- adrian
> 

Of course not, I will do so. I just wanted to make sure I understood the current review procedure before doing so :-)

BR,
Chris

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-dev/attachments/20180105/232c6abb/attachment.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: smime.p7s
Type: application/pkcs7-signature
Size: 3854 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-dev/attachments/20180105/232c6abb/attachment.bin>