[PATCH] D75845: [ValueLattice] Add singlecrfromundef lattice value.

Florian Hahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Mar 11 12:29:44 PDT 2020


fhahn marked 2 inline comments as done.
fhahn added inline comments.


================
Comment at: llvm/include/llvm/Analysis/ValueLattice.h:31
     /// state in our local meet rules.  In this usage, it's taken to mean
     /// "nothing known yet".
     unknown,
----------------
efriedma wrote:
> I think it would make sense to list out the possible state transitions for each state here.
Done! Is the format along the lines you imagined? It does not list explicitly that we can stay in the existing state, if nothing changes.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D75845/new/

https://reviews.llvm.org/D75845





More information about the llvm-commits mailing list