[PATCH] D149587: InstSimplify: Simplifications for ldexp

Kevin P. Neal via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue May 2 05:27:53 PDT 2023


kpn added inline comments.


================
Comment at: llvm/lib/Analysis/InstructionSimplify.cpp:6034
+
+  // ldexp(x, 0) -> x
+  if (match(Op1, PatternMatch::m_ZeroInt()))
----------------
foad wrote:
> Why is this not strictfp-safe?
It would pass through an SNaN instead of quieting it I expect.


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

https://reviews.llvm.org/D149587



More information about the llvm-commits mailing list