[PATCH] D100748: [Greedy RA] Do not split interval on tied-def

Serguei Katkov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 29 03:09:29 PDT 2021


skatkov added inline comments.


================
Comment at: llvm/lib/CodeGen/SplitKit.cpp:822
   LLVM_DEBUG(dbgs() << "    overlapIntv [" << Start << ';' << End << "):");
   RegAssign.insert(Start, End, OpenIdx);
   LLVM_DEBUG(dump());
----------------
Generally, we cannot set end of OpenIdx to End if it is a statepoint instruction and register represents tied-def on this instructions.

Setting end we split tied-use and tied-def in different live intervals allowing to assign them to different registers breaking the tied-ness property.


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

https://reviews.llvm.org/D100748



More information about the llvm-commits mailing list