<div dir="ltr"><br><br><div class="gmail_quote"><div dir="ltr">On Tue, Sep 15, 2015 at 5:13 PM Duncan P. N. Exon Smith <<a href="mailto:dexonsmith@apple.com">dexonsmith@apple.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
> On 2015-Sep-15, at 16:52, Manuel Klimek <<a href="mailto:klimek@google.com" target="_blank">klimek@google.com</a>> wrote:<br>
><br>
><br>
><br>
> On Tue, Sep 15, 2015 at 4:33 PM Duncan P. N. Exon Smith <<a href="mailto:dexonsmith@apple.com" target="_blank">dexonsmith@apple.com</a>> wrote:<br>
> +llvm-commits<br>
><br>
> > On 2015-Sep-15, at 15:49, Teresa Johnson <<a href="mailto:tejohnson@google.com" target="_blank">tejohnson@google.com</a>> wrote:<br>
> ><br>
> > I just realized that none of the discussion starting with Duncan's<br>
> > review is going to llvm-commits, since llvm-commits is a subscriber in<br>
> > Phab, but the review by email doesn't actually update Phab or cc the<br>
> > subscribers (only the reviewers).<br>
><br>
> And I just added two more emails :(.<br>
><br>
> I'm not sure I understand how this happened.  Is it my fault somehow?  I just hit "reply-all" and I guess I didn't check the CC list.<br>
><br>
> If there was an email that was not cc'ed to llvm-commits, llvm-commits was not put in as subsriber before creating the revision (the creator of the revision has to make sure, just like if you manually send an email)<br>
><br>
<br>
This seems like a different case.  I've attached the original email as an attachment, but I'll quote the relevant part inline:<br>
<br>
> Begin forwarded message:<br>
><br>
> From: Teresa Johnson <<a href="mailto:tejohnson@google.com" target="_blank">tejohnson@google.com</a>><br>
> Date: 2015 September 1 at 10:53:51 PDT<br>
> To: <a href="mailto:tejohnson@google.com" target="_blank">tejohnson@google.com</a>, <a href="mailto:dexonsmith@apple.com" target="_blank">dexonsmith@apple.com</a>, <a href="mailto:davidxl@google.com" target="_blank">davidxl@google.com</a>, <a href="mailto:mehdi.amini@apple.com" target="_blank">mehdi.amini@apple.com</a><br>
> Cc: <a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a><br>
> Subject: [PATCH] D12536: Function bitcode index in Value Symbol Table and lazy reading support<br>
> Reply-To: <a href="mailto:reviews%2BD12536%2Bpublic%2B4f470f0312a65d2b@reviews.llvm.org" target="_blank">reviews+D12536+public+4f470f0312a65d2b@reviews.llvm.org</a><br>
> List-Id: <a href="mailto:reviews@reviews.llvm.org" target="_blank">reviews@reviews.llvm.org</a><br>
><br>
> tejohnson created this revision.<br>
> tejohnson added reviewers: dexonsmith, davidxl, joker.eph.<br>
> tejohnson added a subscriber: llvm-commits.<br>
><br>
<br>
Obviously I'm not a phab user, but it looks to me like this revision was created with llvm-commits as a subscriber.  Is there another way for llvm-commits to get dropped?<br></blockquote><div><br></div><div>This also seems to have been cc' llvm-commits, so I'd expect an email program to include it in reply-all?</div></div></div>