[PATCH] D28084: Value number stores and memory states so we can detect when memorystates are equivalent (IE store of same value to memory).Tests coming.

Daniel Berlin via llvm-commits llvm-commits at lists.llvm.org
Sat Dec 24 11:50:49 PST 2016


On Sat, Dec 24, 2016 at 7:50 AM, Davide Italiano via Phabricator <
reviews at reviews.llvm.org> wrote:

> davide added a comment.
>
> This is very nice. I think I got the latest revision (unless there's an
> issue with phabricator) and I see many tests in `Transform/NewGVN` failing,
> some of them hitting the assertion in `updateProcessedCount()`.
>

This is my fault, the latest rev is not really uploaded.
It's due to the continue after valueNumberMemoryPhi.
I'll address the rest and upload a new version
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20161224/2fcd7d16/attachment.html>


More information about the llvm-commits mailing list