[PATCH] D99284: [RegAllocFast] properly handle STATEPOINT instruction.

Denis Antrushin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 1 08:00:48 PDT 2021


dantrushin updated this revision to Diff 334698.
dantrushin added a comment.

Rework algorithm a bit:

- Store intruction RegMask (if any) in a global;
- Look at RegMask when allocating register if we care about physical register use. (Re)Use LookAtPhysRegUses for this.

With this change, registers clobbered by regmask should not be assigned
to tied defs and `AssignedTiedDefs` set is not strictly needed. I use
it for assertion check, but can eliminate it.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D99284

Files:
  llvm/lib/CodeGen/RegAllocFast.cpp
  llvm/test/CodeGen/X86/statepoint-fastregalloc.mir

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D99284.334698.patch
Type: text/x-patch
Size: 4157 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20210401/66328c9b/attachment.bin>


More information about the llvm-commits mailing list