[PATCH] D37353: [SparsePropagation] Enable interprocedural analysis
Daniel Berlin via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Oct 2 16:08:20 PDT 2017
dberlin added a comment.
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)
https://reviews.llvm.org/D37353
More information about the llvm-commits
mailing list