[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