[PATCH] D28137: [Devirtualization] MemDep returns non-local !invariant.group dependencies

Daniel Berlin via llvm-commits llvm-commits at lists.llvm.org
Fri Dec 30 15:26:55 PST 2016


>
>
> PS: consider sending replies in phabricator, because email replies
> sometimes doesn't appear there (like this one)
>

I'm strongly against that, in the sense that phabricator needs to integrate
with existing workflows.  I do real reviews in phab, but just overall
commenting, don't have time or energy to click when i'm reading email :)

If email to phabricator is not currently working, we should fix that.
(I believe it is *supposed to be working*)



>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20161230/178d16a8/attachment.html>


More information about the llvm-commits mailing list