[PATCH] D74671: [AMDGPU] Use generated RegisterPressureSets enum

Nicolai Hähnle via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Feb 16 03:38:40 PST 2020


nhaehnle accepted this revision.
nhaehnle added a comment.
This revision is now accepted and ready to land.

LGTM. I agree with Matt that it would be great not to create those other pressure sets in the first place, possibly by adding some .td file construct to explicitly list the ones that are required. That said, this change is clearly a step in the right direction.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D74671/new/

https://reviews.llvm.org/D74671





More information about the llvm-commits mailing list