[PATCH] D33105: [AMDGPU] Turn register pressure estimation into forward tracker

[Stanislav Mekhanoshin via Phabricator via llvm-commits]

rampitec added inline comments.


================
Comment at: lib/Target/AMDGPU/GCNRegPressure.cpp:313-314
+
+  if (MI.isDebugValue())
+    return;
+
----------------
arsenm wrote:
> Should this be using something like isTransient (as it is now looks inappropriate) to avoid IMPLICIT_DEF from impacting the pressure estimate?
I believe these defs should be accounted towards pressure. RA will finally allocate something for it, and in our case it is better to overestimate pressure rather than underestimate it. Anyway, this is not a functional change, it basically factors out an existing algorithm into a common interface. I do not want to change its behavior with this change.


Repository:
  rL LLVM

https://reviews.llvm.org/D33105