[llvm] [AMDGPU][NFC] Minor source cleanups in SIInsertWaitcnts (PR #181095)

[Sameer Sahasrabuddhe via llvm-commits]

================
@@ -2997,8 +2995,7 @@ bool WaitcntBrackets::mergeAsyncMarks(ArrayRef<MergeInfo> MergeInfos,
   unsigned OtherSize = OtherMarks.size();
   unsigned OurSize = AsyncMarks.size();
   unsigned MergeCount = std::min(OtherSize, OurSize);
-  assert(OurSize == MaxSize);
-  for (unsigned Idx = 1; Idx <= MergeCount; ++Idx) {
+  for (auto Idx : seq_inclusive<unsigned>(1, MergeCount)) {
----------------
ssahasra wrote:

But then in that case, the type is already clear from "1u", and using an explicit "unsigned" type for Idx could be considered a repetition.

https://github.com/llvm/llvm-project/pull/181095