[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:

Right. "1u" felt cryptic to me, but I can see how one could get used to it. Yes, template arguments do look ugly especially when C++ does a very good job of inferring them.

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