Address code review comments

Author: AmrDeveloper

clang/include/clang/CIR/Dialect/IR/CIROps.td

@@ -2160,7 +2160,8 @@ def VecShuffleDynamicOp : CIR_Op<"vec.shuffle.dynamic",
     vector from the indices indicated by the elements of the second vector.
 
     ```mlir
-    %new_vec = cir.vec.shuffle.dynamic %vec : !cir.vector<4 x !s32i>, %indices : !cir.vector<4 x !s32i>
+    %new_vec = cir.vec.shuffle.dynamic %vec : !cir.vector<4 x !s32i>, %indices
+        : !cir.vector<4 x !s32i>
     ```
   }];
 

clang/lib/CIR/Lowering/DirectToLLVM/LowerToLLVM.cpp

@@ -1891,6 +1891,11 @@ mlir::LogicalResult CIRToLLVMVecShuffleDynamicOpLowering::matchAndRewrite(
       elementTypeIfVector(op.getIndices().getType()));
   uint64_t numElements =
       mlir::cast<cir::VectorType>(op.getVec().getType()).getSize();
+
+  if (llvm::isPowerOf2_64(numElements))
+    return op.emitError() << "unsupported VecShuffleDynamic for VectorType "
+                             "with size not power of 2";
+
   mlir::Value maskValue = rewriter.create<mlir::LLVM::ConstantOp>(
       loc, llvmIndexType,
       mlir::IntegerAttr::get(llvmIndexType, numElements - 1));