[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