[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