[PATCH] D76596: Backport 4878aa36d4a [ValueLattice] Add new state for undef constants.

[Tom Stellard via Phabricator via llvm-commits]

tstellar added inline comments.


================
Comment at: llvm/include/llvm/Analysis/ValueLattice.h:37
+    /// assuming all uses of the result will be replaced.
+    undef,
 
----------------
I'm fairly certain this is an API and ABI breaking change, I'm fixing up the abi check job on GitHub actions to try to confirm this.


================
Comment at: llvm/include/llvm/Analysis/ValueLattice.h:142
 
-  bool isUndefined() const { return Tag == undefined; }
   bool isConstant() const { return Tag == constant; }
----------------
Removing this is an ABI breaking change, so the patch would need to be modified to keep this in.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D76596