eopXD added inline comments.
================ Comment at: clang/include/clang/Basic/riscv_vector.td:2602 + unsigned MaxIndex = cast<StructType>(VTupleOperand->getType())->getNumElements(); + Index = std::min(Index, MaxIndex); + ---------------- craig.topper wrote: > Does this `min` do anything or does SemaChecking already guarantee it? Yes, SemaChecking should be able to guard the value correctly and this line is redundant. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D147916/new/ https://reviews.llvm.org/D147916 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
