[PATCH] D126771: [fastalloc] Support allocate specific register class in fastalloc.

LuoYuanke via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 23 05:23:25 PDT 2022


LuoYuanke added a comment.

In D126771#3604703 <https://reviews.llvm.org/D126771#3604703>, @DavidSpickett wrote:

> I also bisected an lldb failure down to this: https://lab.llvm.org/buildbot/#/builders/96/builds/25038/steps/6/logs/stdio
>
> It has backtrace of a sort but it's probably not much help, I will get hold of the reproducer files.

Thank you! I guess it was caused by line 306, I'll try getRegBankOrNull() instead of getRegClass().



================
Comment at: llvm/lib/CodeGen/RegAllocFast.cpp:306
+  assert(Register::isVirtualRegister(Reg));
+  const TargetRegisterClass &RC = *MRI->getRegClass(Reg);
+  return ShouldAllocateClass(*TRI, RC);
----------------
I'll use getRegClassOrNull() to replace getRegClass().


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D126771



More information about the llvm-commits mailing list