[PATCH] D144897: [SPIRV] fix several issues in builds with expensive checks
Ilia Diachkov via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Feb 27 13:28:01 PST 2023
iliya-diyachkov 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);
----------------
arsenm wrote:
> I'd expect the class to never be set at this point (if not, I'd expect it to be always set)
18 LIT tests will fail if I remove this setRegClass(). On other hand, I found calls with register classes already set here in 27 LIT tests.
So I suppose it makes sense to leave this check and setting.
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