[PATCH] D4276: Added llvm.is.constant intrinsic

[Eli Friedman via Phabricator via llvm-commits]

efriedma added a comment.

> The SCCP bug is already be exhibited by the tests in test/CodeGen/Generic/is-constant.ll which pass a struct.

I'd prefer a testcase that explicitly calls "opt -sccp", as opposed to implicitly relying on the fact that opt -O2 includes SCCP.  (You can just add a small test to test/Transforms/SCCP/ipsccp-basic.ll .)


https://reviews.llvm.org/D4276