[PATCH] D78425: [ValueLattice] Add move constructor

Florian Hahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Apr 19 12:17:29 PDT 2020


fhahn accepted this revision.
fhahn added a comment.
This revision is now accepted and ready to land.

LGTM, thanks. I think it would be good to update the description  after the latest changes.



================
Comment at: include/llvm/Analysis/ValueLattice.h:107
+public:
+  // Const and Range are initialized on-demand.
+  ValueLatticeElement() : Tag(unknown) {}
----------------
might be a good opportunity to update Const -> ConstVal.


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

https://reviews.llvm.org/D78425





More information about the llvm-commits mailing list