Revert "[Polly] Reject reject regions entered by an indirectbr/callbr."

This reverts commit 91f46bb which
causes test failures when assertions are off.

Author: zeroomega

polly/include/polly/ScopDetectionDiagnostic.h

@@ -73,7 +73,6 @@ enum class RejectReasonKind {
   InvalidTerminator,
   IrreducibleRegion,
   UnreachableInExit,
-  IndirectPredecessor,
   LastCFG,
 
   // Non-Affinity
@@ -276,32 +275,6 @@ class ReportUnreachableInExit : public ReportCFG {
   //@}
 };
 
-//===----------------------------------------------------------------------===//
-/// Captures regions with an IndirectBr predecessor.
-class ReportIndirectPredecessor : public ReportCFG {
-  Instruction *Inst;
-  DebugLoc DbgLoc;
-
-public:
-  ReportIndirectPredecessor(Instruction *Inst, DebugLoc DbgLoc)
-      : ReportCFG(RejectReasonKind::IndirectPredecessor), Inst(Inst),
-        DbgLoc(DbgLoc) {}
-
-  /// @name LLVM-RTTI interface
-  //@{
-  static bool classof(const RejectReason *RR);
-  //@}
-
-  /// @name RejectReason interface
-  //@{
-  std::string getRemarkName() const override;
-  const Value *getRemarkBB() const override;
-  std::string getMessage() const override;
-  std::string getEndUserMessage() const override;
-  const DebugLoc &getDebugLoc() const override;
-  //@}
-};
-
 //===----------------------------------------------------------------------===//
 /// Base class for non-affine reject reasons.
 ///

polly/lib/Analysis/ScopDetection.cpp

@@ -1757,13 +1757,6 @@ bool ScopDetection::isValidRegion(DetectionContext &Context) {
     return false;
   }
 
-  for (BasicBlock *Pred : predecessors(CurRegion.getEntry())) {
-    Instruction *PredTerm = Pred->getTerminator();
-    if (isa<IndirectBrInst>(PredTerm) || isa<CallBrInst>(PredTerm))
-      return invalid<ReportIndirectPredecessor>(
-          Context, /*Assert=*/true, PredTerm, PredTerm->getDebugLoc());
-  }
-
   // SCoP cannot contain the entry block of the function, because we need
   // to insert alloca instruction there when translate scalar to array.
   if (!PollyAllowFullFunction &&

polly/lib/Analysis/ScopDetectionDiagnostic.cpp

@@ -50,9 +50,8 @@ using namespace llvm;
 static Statistic RejectStatistics[] = {
     SCOP_STAT(CFG, ""),
     SCOP_STAT(InvalidTerminator, "Unsupported terminator instruction"),
-    SCOP_STAT(IrreducibleRegion, "Irreducible loops"),
     SCOP_STAT(UnreachableInExit, "Unreachable in exit block"),
-    SCOP_STAT(IndirectPredecessor, "Branch from indirect terminator"),
+    SCOP_STAT(IrreducibleRegion, "Irreducible loops"),
     SCOP_STAT(LastCFG, ""),
     SCOP_STAT(AffFunc, ""),
     SCOP_STAT(UndefCond, "Undefined branch condition"),
@@ -240,37 +239,6 @@ bool ReportUnreachableInExit::classof(const RejectReason *RR) {
   return RR->getKind() == RejectReasonKind::UnreachableInExit;
 }
 
-//===----------------------------------------------------------------------===//
-// IndirectPredecessor.
-
-std::string ReportIndirectPredecessor::getRemarkName() const {
-  return "IndirectPredecessor";
-}
-
-const Value *ReportIndirectPredecessor::getRemarkBB() const {
-  if (Inst)
-    return Inst->getParent();
-  return nullptr;
-}
-
-std::string ReportIndirectPredecessor::getMessage() const {
-  if (Inst)
-    return "Branch from indirect terminator: " + *Inst;
-  return getEndUserMessage();
-}
-
-const DebugLoc &ReportIndirectPredecessor::getDebugLoc() const {
-  return DbgLoc;
-}
-
-std::string ReportIndirectPredecessor::getEndUserMessage() const {
-  return "Branch from indirect terminator.";
-}
-
-bool ReportIndirectPredecessor::classof(const RejectReason *RR) {
-  return RR->getKind() == RejectReasonKind::IndirectPredecessor;
-}
-
 //===----------------------------------------------------------------------===//
 // ReportIrreducibleRegion.