[PATCH] D110164: [NVPTX] Add VRFrame and VRFrameLocal to integer register classes

Andrew Savonichev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Sep 21 06:30:51 PDT 2021


asavonic added a comment.

While the patch does silence the error, I don't think that assigning
the same register to different register classes (for 32- and 64-bit
triple) is the right thing to do.

Instead, I guess we can define new set of registers (VRFrame64 and
VRFrameLocal64), and use them depending on a triple. Let me know if
this is an acceptable solution, or we should do something else.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D110164



More information about the llvm-commits mailing list