[PATCH] D144897: [SPIRV] fix several issues in builds with expensive checks

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Feb 27 12:14:22 PST 2023


arsenm added inline comments.


================
Comment at: llvm/lib/Target/SPIRV/SPIRVCallLowering.cpp:392
       assert(Arg.Regs.size() == 1 && "Call arg has multiple VRegs");
+      if (!MRI->getRegClassOrNull(Arg.Regs[0]))
+        MRI->setRegClass(Arg.Regs[0], &SPIRV::IDRegClass);
----------------
I'd expect the class to never be set at this point (if not, I'd expect it to be always set)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D144897



More information about the llvm-commits mailing list