[PATCH] D996: hasPath

Chandler Carruth via llvm-commits llvm-commits at lists.llvm.org
Wed Oct 5 19:04:34 PDT 2016


On Wed, Oct 5, 2016 at 1:25 PM Eugene Zelenko via llvm-commits <
llvm-commits at lists.llvm.org> wrote:

> Eugene.Zelenko added a comment.
> This revision now requires review to proceed.
>
> Looks like patch was not committed.
>

I would like to ask that you are a bit less eager with auditing all of
phabricator for patches that haven't been committed. I think this is
starting to add more noise than signal.

In particular, I would suggest only pinging in this way if there is some
functionality you are interested in seeing specifically and if you have
verified that in fact it doesn't exist in tree, not just that the
phabricator review isn't closed.

Keep in mind, phabricator is not the system of record for LLVM. It is just
a tool we use to make writing and reading code reviews a bit easier.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20161006/e44f0902/attachment.html>


More information about the llvm-commits mailing list