[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