aaron.ballman added a comment. In D124081#3461742 <https://reviews.llvm.org/D124081#3461742>, @mboehme wrote:
> In D124081#3461731 <https://reviews.llvm.org/D124081#3461731>, @aaron.ballman > wrote: > >> In D124081#3461556 <https://reviews.llvm.org/D124081#3461556>, @mboehme >> wrote: >> >>> I'd like to mark this patch as a draft so that Herald doesn't keep adding >>> reviewers -- but I'm not sure how? >> >> In that case, I usually set the permissions on the review explicitly so that >> I'm the only one who can view the draft (but this means precommit CI won't >> run on it). > > In this case, though, I want to be able to show the draft to others and > discuss it with them. I guess this technique doesn't work for that? FWIW, you can pick the specific list of folks you want to give permissions to if you know it's a limited group. (I've done this with Erich from time to time when I want a second set of eyes on an NFC change but don't want to bother the mailing lists with it.) However, it sounds to me like you want anyone to be able to come by and comment if they'd like while you're prepping the patch, so I'd stick with the [WIP] in the title like you've got. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D124081/new/ https://reviews.llvm.org/D124081 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
