[PATCH] D30147: AMDGPU/SI: Add new SISched policy to reduce register usage

[Axel Davy via Phabricator via llvm-commits]

axeldavy added a comment.

To ease rebasing, I'll move this patch to after the patch adding support for subreg lane tracking.



================
Comment at: lib/Target/AMDGPU/SIMachineScheduler.cpp:1801
+  DenseMap<unsigned, SmallPtrSet<SIScheduleBlock*, 16>> RegisterConsumers;
+  SIBlockInfo() : Dependencies(), ConsumedRegisters(),
+  ProducedConsumedRegisters(), ProducedRegisters(), BlockInRegs(),
----------------
vpykhtin wrote:
> do not need a constructor here, default would do just what you've written
DenseMap has only an explicit constructor.
The compiler is very unhappy if I don't define the constructor of the structure, which calls the explicit constructor of the DenseMap.


Repository:
  rL LLVM

https://reviews.llvm.org/D30147