[PATCH] D127744: freeze does not change the constant property

[Philip Reames via Phabricator via llvm-commits]

reames added a comment.

Non blocking comment..

freeze(undef) being equally null or base as undef seems questionable to me.  I think in practice it might work out okay, but there's no guarantee that the value chosen for undef isn't 0 when frozen, and 1 when not.  You might want to step back and take a deeper look at the problem you're solving here.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D127744