[PATCH] D149587: InstSimplify: Simplifications for ldexp

[Jay Foad via Phabricator via llvm-commits]

foad added inline comments.


================
Comment at: llvm/lib/Analysis/ConstantFolding.cpp:1602
   case Intrinsic::amdgcn_fract:
-  case Intrinsic::amdgcn_ldexp:
   case Intrinsic::amdgcn_sin:
----------------
Should keep support for the old intrinsic while it still exists.


================
Comment at: llvm/lib/Analysis/ConstantFolding.cpp:2706
 
-      if (IntrinsicID == Intrinsic::amdgcn_ldexp) {
+      if (IntrinsicID == Intrinsic::ldexp) {
         // FIXME: Should flush denorms depending on FP mode, but that's ignored
----------------
Should keep support for the old intrinsic while it still exists.


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

https://reviews.llvm.org/D149587