[LoopVectorize] Relax a FCmpInst assert to dyn_cast after D95690

The instruction may be `icmp eq i32`. Noticed in an internal Halide+wasm JIT test.
vmaksimo · Feb 2, 2021 · 0fa6130 · 0fa6130
1 parent 41877b8
commit 0fa6130
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/llvm/lib/Analysis/IVDescriptors.cpp b/llvm/lib/Analysis/IVDescriptors.cpp
@@ -309,8 +309,8 @@ bool RecurrenceDescriptor::AddReductionVar(PHINode *Phi, RecurKind Kind,
           // TODO: This is a hack to work-around the fact that FMF may not be
           //       assigned/propagated correctly. If that problem is fixed or we
           //       standardize on fmin/fmax via intrinsics, this can be removed.
-          assert(isa<FCmpInst>(Sel->getCondition()) && "Expected fcmp min/max");
-          CurFMF |= cast<FCmpInst>(Sel->getCondition())->getFastMathFlags();
+          if (auto *FCmp = dyn_cast<FCmpInst>(Sel->getCondition()))
+            CurFMF |= FCmp->getFastMathFlags();
         }
         FMF &= CurFMF;
       }