[PATCH] D30150: AMDGPU/SI: Update colorEndsAccordingToDependencies

[Valery Pykhtin via Phabricator via llvm-commits]

vpykhtin added inline comments.


================
Comment at: lib/Target/AMDGPU/SIMachineScheduler.cpp:840
+  // everything in one block.
+  if (*std::max_element(CurrentBottomUpReservedDependencyColoring.begin(),
+                        CurrentBottomUpReservedDependencyColoring.end()) == 0 &&
----------------
axeldavy wrote:
> vpykhtin wrote:
> > max_element can return end iterator only if the range is empty, so could you place an assert for this before these comparisons?
> The two vectors are supposed to be the size of DAGSize.
> Should I assert their size, and that DAGSize is >= 1 ?
It would look a bit paranoid :-) but I think it would make more sence than just asserting its not empty.


Repository:
  rL LLVM

https://reviews.llvm.org/D30150