Address comments

Author: madhur13490

llvm/lib/Transforms/Scalar/LoopInterchange.cpp

@@ -75,10 +75,14 @@ using CharMatrix = std::vector<std::vector<char>>;
 } // end anonymous namespace
 
 // Minimum loop depth supported.
-static const unsigned MinLoopNestDepth = 2;
+static cl::opt<unsigned int> MinLoopNestDepth(
+    "loop-interchange-min-loop-nest-depth", cl::init(2), cl::Hidden,
+    cl::desc("Minimum depth of loop nest considered for the transform"));
 
 // Maximum loop depth supported.
-static const unsigned MaxLoopNestDepth = 10;
+static cl::opt<unsigned int> MaxLoopNestDepth(
+    "loop-interchange-max-loop-nest-depth", cl::init(10), cl::Hidden,
+    cl::desc("Maximum depth of loop nest considered for the transform"));
 
 #ifndef NDEBUG
 static void printDepMatrix(CharMatrix &DepMatrix) {

llvm/test/Transforms/LoopInterchange/deep-loop-nest.ll

@@ -1,14 +1,14 @@
-; REQUIRES: asserts
-; RUN: opt < %s -passes=loop-interchange -debug -disable-output 2>&1| FileCheck %s
+; RUN: opt < %s -passes=loop-interchange -pass-remarks-missed='loop-interchange' \
+; RUN:          -disable-output 2>&1 | FileCheck %s
 
-target datalayout = "e-m:e-i64:64-f80:128-n8:16:32:64-S128"
+; RUN: opt < %s -passes=loop-interchange -pass-remarks-missed='loop-interchange' \
+; RUN:          -loop-interchange-max-loop-nest-depth=12 -disable-output 2>&1 | \
+; RUN:          FileCheck --allow-empty -check-prefix=CHECK-MAX %s
 
-; For deep loop nest, delinearization should not be run.
+target datalayout = "e-m:e-i64:64-f80:128-n8:16:32:64-S128"
 
-; CHECK-NOT: Delinearizing
-; CHECK-NOT: Strides:
-; CHECK-NOT: Terms:
-; CHECK: Unsupported depth of loop nest 11, the supported range is [2, 10].
+; CHECK: Unsupported depth of loop nest, the supported range is [2, 10].
+; CHECK-MAX-NOT: Unsupported depth of loop nest, the supported range is [2, 10].
 define void @big_loop_nest() {
 entry:
   br label %for1.header