[PATCH] D43329: [SystemZ, MachineScheduler] Refactor GenericScheduler::tryCandidate() to reuse parts in a new SystemZ scheduling strategy.

[Florian Hahn via Phabricator via llvm-commits]

fhahn accepted this revision.
fhahn added a comment.

Great, thanks Jonas! LGTM too.



================
Comment at: lib/Target/AMDGPU/SIMachineScheduler.cpp:157
 
+namespace llvm { namespace SISched {
 static bool tryLess(int TryVal, int CandVal,
----------------
nit: not sure, should nested namespaces be separated by newlines?


https://reviews.llvm.org/D43329