[PATCH] D37353: [SparsePropagation] Enable interprocedural analysis

Davide Italiano via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Sep 11 14:55:06 PDT 2017


davide added a comment.

In https://reviews.llvm.org/D37353#867141, @mssimpso wrote:

> In https://reviews.llvm.org/D37353#867090, @davide wrote:
>
> > If it's not too hard, you may consider adding some unit tests for the sparse engine.
>
>
> Ah, I didn't think about adding tests to unittests/. That makes sense - I'll give it a shot. I do wonder if it'd be easier to just move SCCP over to the generic engine, though.


That wouldn't  be a bad idea at all :)


https://reviews.llvm.org/D37353





More information about the llvm-commits mailing list