[PATCH] D37353: [SparsePropagation] Enable interprocedural analysis

Matthew Simpson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Oct 3 07:55:29 PDT 2017


mssimpso added a comment.

In https://reviews.llvm.org/D37353#886435, @dberlin wrote:

> FYI: I'm about to rename getOrInitValueState to getValueState.
>  That is what SCCP calls it, and IMHO, we should be consistent.
>
> I think you should do roughly the same (IE name the return-only ones as getExisting or getLattice, and not add "orInit" to the initializing ones)


Sounds good to me.


https://reviews.llvm.org/D37353





More information about the llvm-commits mailing list