<div dir="ltr">Btw, I'm seeing other inconsistencies, e.g., I just committed <a href="https://reviews.llvm.org/D42460" target="_blank">https://reviews.<wbr>llvm.org/D42460</a> and herald ended up adding both cfe-commits and llvm-commits to subscribers.<div><br></div><div>When I created it, I left subscribers blank, so herald added cfe-commits -- which seems to be correct.</div><div>Then, when I committed it, herald added llvm-commits -- which doesn't seem correct.</div><div><br></div><div><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Jan 25, 2018 at 10:04 AM, Don Hinton <span dir="ltr"><<a href="mailto:hintonda@gmail.com" target="_blank">hintonda@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Thanks Ben...</div><div class="HOEnZb"><div class="h5"><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Jan 25, 2018 at 9:53 AM, Ben Hamilton <span dir="ltr"><<a href="mailto:benhamilton@google.com" target="_blank">benhamilton@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Ah, I see from <a href="http://bcain-llvm.readthedocs.io/projects/libunwind/en/latest/" target="_blank">http://bcain-llvm.readthe<wbr>docs.io/projects/libunwind/en/<wbr>latest/</a> that libunwind review discussion does take place on cfe-commits.<div><br></div><div>Added. Sorry about not doing that earlier!</div><span class="m_8434331081452894173HOEnZb"><font color="#888888"><div><br></div><div>Ben</div></font></span><div><div class="m_8434331081452894173h5"><div><br><div class="gmail_quote"><div dir="ltr">On Thu, Jan 25, 2018 at 8:37 AM Ben Hamilton <<a href="mailto:benhamilton@google.com" target="_blank">benhamilton@google.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><p dir="ltr">Great question. Is there an existing separate list for libunwind? </p>
<p dir="ltr">If libunwind discussion already happens on cfe-commits, then I'll make reviews for libunwind subscribe that list.</p>
<br><div class="gmail_quote"><div dir="ltr">On Tue, Jan 23, 2018, 21:40 Don Hinton <<a href="mailto:hintonda@gmail.com" target="_blank">hintonda@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div>Currently, herald doesn't subscribe libunwind to any group.  Should it be added to cfe-commits?</div><div><br></div>  <a href="https://reviews.llvm.org/H268" target="_blank">https://reviews.llvm.org/H268</a><br></div>
</blockquote></div></blockquote></div></div></div></div></div>
</blockquote></div><br></div>
</div></div></blockquote></div><br></div>