[PATCH] D90231: [GVN] Don't replace argument to @llvm.is.constant.*()

[Joerg Sonnenberger via Phabricator via llvm-commits]

joerg added a comment.

I completely agree with Roman that this patch is very undesirable since it has a high chance of breaking other reasonable uses of the intrinsic. In fact, I would say that the observed behavior here is perfectly reasonable under the definition of @lllvm.is.constant. The original code for LLVM at least should be just `__builtin_constant_p(b) && b > -129 && b < 128)`.


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

https://reviews.llvm.org/D90231