Address comments

Author: Groverkss

mlir/test/Conversion/VectorToLLVM/vector-to-llvm.mlir

@@ -1295,7 +1295,8 @@ func.func @extract_scalar_from_vec_2d_f32_inner_dynamic_idx(%arg0: vector<1x16xf
   return %0 : f32
 }
 
-// Multi-dim vectors are supported if the innermost index is dynamic.
+// Lowering supports extracting from multi-dim vectors with dynamic indices
+// provided that only the trailing index is dynamic.
 
 // CHECK-LABEL: @extract_scalar_from_vec_2d_f32_inner_dynamic_idx(
 //       CHECK:   llvm.extractvalue
@@ -1306,7 +1307,8 @@ func.func @extract_scalar_from_vec_2d_f32_inner_dynamic_idx_scalable(%arg0: vect
   return %0 : f32
 }
 
-// Multi-dim vectors are supported if the innermost index is dynamic.
+// Lowering supports extracting from multi-dim vectors with dynamic indices
+// provided that only the trailing index is dynamic.
 
 // CHECK-LABEL: @extract_scalar_from_vec_2d_f32_inner_dynamic_idx_scalable(
 //       CHECK:   llvm.extractvalue
@@ -1319,7 +1321,8 @@ func.func @extract_scalar_from_vec_2d_f32_outer_dynamic_idx(%arg0: vector<1x16xf
   return %0 : f32
 }
 
-// Multi-dim vectors are supported if the inner most dimension is dynamic.
+// Lowering supports extracting from multi-dim vectors with dynamic indices
+// provided that only the trailing index is dynamic.
 
 // CHECK-LABEL: @extract_scalar_from_vec_2d_f32_outer_dynamic_idx(
 //       CHECK:   vector.extract
@@ -1329,8 +1332,8 @@ func.func @extract_scalar_from_vec_2d_f32_outer_dynamic_idx_scalable(%arg0: vect
   return %0 : f32
 }
 
-// Multi-dim vectors with outer indices as dynamic are not supported, but it
-// shouldn't crash.
+// Lowering does not support extracting from multi-dim vectors with non trailing
+// dynamic index, but it shouldn't crash.
 
 // CHECK-LABEL: @extract_scalar_from_vec_2d_f32_outer_dynamic_idx_scalable(
 //       CHECK:   vector.extract