[PATCH] D157894: InstSimplify: Handle exp10(log10(x)) -> x

[Matt Arsenault via Phabricator via llvm-commits]

arsenm created this revision.
arsenm added reviewers: jcranmer-intel, foad, kpn, sepavloff, andrew.w.kaylor.
Herald added subscribers: StephenFan, hiraditya.
Herald added a project: All.
arsenm requested review of this revision.
Herald added a subscriber: wdng.
Herald added a project: LLVM.

Copy from exp/exp2 case.


https://reviews.llvm.org/D157894

Files:
  llvm/lib/Analysis/InstructionSimplify.cpp
  llvm/test/Transforms/InstSimplify/log-exp-intrinsic.ll


Index: llvm/test/Transforms/InstSimplify/log-exp-intrinsic.ll
===================================================================
--- llvm/test/Transforms/InstSimplify/log-exp-intrinsic.ll
+++ llvm/test/Transforms/InstSimplify/log-exp-intrinsic.ll
@@ -246,9 +246,7 @@
 
 define double @exp10_reassoc_log10_strict(double %a) {
 ; CHECK-LABEL: @exp10_reassoc_log10_strict(
-; CHECK-NEXT:    [[TMP1:%.*]] = call double @llvm.log10.f64(double [[A:%.*]])
-; CHECK-NEXT:    [[TMP2:%.*]] = call reassoc double @llvm.exp10.f64(double [[TMP1]])
-; CHECK-NEXT:    ret double [[TMP2]]
+; CHECK-NEXT:    ret double [[A:%.*]]
 ;
   %1 = call double @llvm.log10.f64(double %a)
   %2 = call reassoc double @llvm.exp10.f64(double %1)
@@ -283,11 +281,7 @@
 
 define double @exp10_log10_exp10_log10_reassoc(double %a) {
 ; CHECK-LABEL: @exp10_log10_exp10_log10_reassoc(
-; CHECK-NEXT:    [[TMP1:%.*]] = call double @llvm.log10.f64(double [[A:%.*]])
-; CHECK-NEXT:    [[TMP2:%.*]] = call reassoc double @llvm.exp10.f64(double [[TMP1]])
-; CHECK-NEXT:    [[TMP3:%.*]] = call double @llvm.log10.f64(double [[TMP2]])
-; CHECK-NEXT:    [[TMP4:%.*]] = call reassoc double @llvm.exp10.f64(double [[TMP3]])
-; CHECK-NEXT:    ret double [[TMP4]]
+; CHECK-NEXT:    ret double [[A:%.*]]
 ;
   %1 = call double @llvm.log10.f64(double %a)
   %2 = call reassoc double @llvm.exp10.f64(double %1)
Index: llvm/lib/Analysis/InstructionSimplify.cpp
===================================================================
--- llvm/lib/Analysis/InstructionSimplify.cpp
+++ llvm/lib/Analysis/InstructionSimplify.cpp
@@ -6153,6 +6153,12 @@
         match(Op0, m_Intrinsic<Intrinsic::log2>(m_Value(X))))
       return X;
     break;
+  case Intrinsic::exp10:
+    // exp10(log10(x)) -> x
+    if (Q.CxtI->hasAllowReassoc() &&
+        match(Op0, m_Intrinsic<Intrinsic::log10>(m_Value(X))))
+      return X;
+    break;
   case Intrinsic::log:
     // log(exp(x)) -> x
     if (Q.CxtI->hasAllowReassoc() &&


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D157894.549972.patch
Type: text/x-patch
Size: 1960 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20230814/277358bf/attachment.bin>