Btw, I'm seeing other inconsistencies, e.g., I just committed https://reviews.llvm.org/D42460 and herald ended up adding both cfe-commits and llvm-commits to subscribers.
When I created it, I left subscribers blank, so herald added cfe-commits -- which seems to be correct. Then, when I committed it, herald added llvm-commits -- which doesn't seem correct. On Thu, Jan 25, 2018 at 10:04 AM, Don Hinton <hinto...@gmail.com> wrote: > Thanks Ben... > > On Thu, Jan 25, 2018 at 9:53 AM, Ben Hamilton <benhamil...@google.com> > wrote: > >> Ah, I see from http://bcain-llvm.readthedocs.io/projects/libunwind/en/ >> latest/ that libunwind review discussion does take place on cfe-commits. >> >> Added. Sorry about not doing that earlier! >> >> Ben >> >> On Thu, Jan 25, 2018 at 8:37 AM Ben Hamilton <benhamil...@google.com> >> wrote: >> >>> Great question. Is there an existing separate list for libunwind? >>> >>> If libunwind discussion already happens on cfe-commits, then I'll make >>> reviews for libunwind subscribe that list. >>> >>> On Tue, Jan 23, 2018, 21:40 Don Hinton <hinto...@gmail.com> wrote: >>> >>>> Currently, herald doesn't subscribe libunwind to any group. Should it >>>> be added to cfe-commits? >>>> >>>> https://reviews.llvm.org/H268 >>>> >>> >
_______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits