[PATCH] D156852: [AMDGPU] Use inreg for hint to preload kernel arguments

Austin Kerbow via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Sep 12 12:07:39 PDT 2023


kerbowa added a comment.

In D156852#4644070 <https://reviews.llvm.org/D156852#4644070>, @arsenm wrote:

> I don't want any usage of metadata, it's confusing and unnecessary. The layout should be fully deterministic from just the argument list and inreg, you don't need a parallel encoding scheme

Thanks for the review Matt, I've removed the metadata system entirely.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D156852



More information about the llvm-commits mailing list