[PATCH] D52010: RegAllocFast: Rewrite and improve

[Matt Arsenault via Phabricator via llvm-commits]

arsenm added a comment.

I see a few different assertions in tests with this patch. Did you see these before or are they regressions?

In a few ARM tests during ARMExpandPseudo:

Assertion failed: (!(isKill && isDef) && "Kill flag on def"), function CreateReg, file ../include/llvm/CodeGen/MachineOperand.h, line 76

In one test:
Assertion failed: ((LRI->LastUse == &MI || MO.isTied() || !isRegUsedInInstr(LRI->PhysReg)) && "assigned reg free"), function useVirtReg, file ../lib/CodeGen/RegAllocFast.cpp, line 723.

In a few different targets' tests:
Assertion failed: (MBB != &MBB->getParent()->front() && "no reload in start block"), function reloadAtBegin, file ../lib/CodeGen/RegAllocFast.cpp, line 320.


Repository:
  rL LLVM

https://reviews.llvm.org/D52010