[PATCH] D78667: [ValueLattice] Merging unknown with empty CR is unknown.

Nikita Popov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Apr 22 14:09:35 PDT 2020


nikic added inline comments.


================
Comment at: llvm/include/llvm/Analysis/ValueLattice.h:359
+        return false;
       return markOverdefined();
+    }
----------------
Can't we do this for the `isUndef()` case as well, i.e. return false here unconditionally?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D78667





More information about the llvm-commits mailing list