[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