[PATCH] D125319: [AMDGPU] gfx11 BUF Instructions

Joe Nash via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed May 11 06:41:52 PDT 2022


Joe_Nash added a comment.

In D125319#3506059 <https://reviews.llvm.org/D125319#3506059>, @foad wrote:

>> Includes a small change in SMInstructions.td to correct encoded bits.
>
> Sounds like that should be a separate patch. Why doesn't it affect any MC tests?

It has to do with SIRegisterInfo.td:214. We now have pregfx11 and postgfx11 version of M0 and SGPR_NULL encoding instead of a common one. So SIRegisterInfo, BUFInstructions and SMInstructions need to be changed in the same commit. Test output doesn't change because the same value is encoded, just referring to a new def name.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D125319



More information about the llvm-commits mailing list