[PATCH] D71126: Adding PoisonValue for representing poison value explicitly in IR

Tue Nov 24 15:46:35 PST 2020

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

LGTM.

p.s. I'm very unsure about the merits of having poison being a sub-class of undef, but I'm also completely fine landing this as is and adjusting later.  Being able to spell poison directly in tests is a huge win, and I'm more than comfortable evolving in tree as warranted.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D71126