[PATCH] D86335: [RegisterPressure] Use Register instead of unsigned for RegUnit

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Aug 21 07:25:35 PDT 2020


arsenm added a comment.

In D86335#2230504 <https://reviews.llvm.org/D86335#2230504>, @foad wrote:

> In D86335#2230402 <https://reviews.llvm.org/D86335#2230402>, @arsenm wrote:
>
>> RegUnits are *not* registers, so you should not use Register for them
>
> Fair enough. What are they?

They are the smallest tracked piece of a physical register. They aren't necessarily addressable. In VGPR0_VGPR1, since recently we have regunits VGPR0.lo16, VGPR0.hi16, VGPR1.lo16, VGPR1.hi16, even though the high halves can't be encoded in instructions


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D86335



More information about the llvm-commits mailing list