[PATCH] D149587: InstSimplify: Simplifications for ldexp

[Matt Arsenault via Phabricator via llvm-commits]

arsenm added inline comments.


================
Comment at: llvm/lib/Analysis/InstructionSimplify.cpp:6103
+  if (!IsStrict) {
+    // ldexp(undef, x) -> nan
+    if (Q.isUndefValue(Op0))
----------------
foad wrote:
> I still don't understand why this one isn't strictfp-safe, if you simplify -> qnan.
if undef could be anything, it could have been a signaling nan that would demand quieting


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

https://reviews.llvm.org/D149587