Improve some comments explaining the "handle kills" stuff better.
authorBill Wendling <isanbard@gmail.com>
Wed, 20 Feb 2008 19:35:34 +0000 (19:35 +0000)
committerBill Wendling <isanbard@gmail.com>
Wed, 20 Feb 2008 19:35:34 +0000 (19:35 +0000)
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@47395 91177308-0d34-0410-b5e6-96231b3b80d8

lib/CodeGen/LiveVariables.cpp

index ea27b2e08e08256913011e5344412f0fa32569ce..809048e8ef7c634e53cdc4f258bd7d6a52d4536e 100644 (file)
@@ -296,6 +296,7 @@ void LiveVariables::addRegisterKills(unsigned Reg, MachineInstr *MI,
 ///     last def/use of the register, or
 ///   - The register has sub-registers and none of them are killed elsewhere.
 ///
+/// SubKills is filled with the set of sub-registers that are killed elsewhere.
 bool LiveVariables::HandlePhysRegKill(unsigned Reg, const MachineInstr *RefMI,
                                       SmallSet<unsigned, 4> &SubKills) {
   const unsigned *SubRegs = RegInfo->getImmediateSubRegisters(Reg);
@@ -320,8 +321,9 @@ bool LiveVariables::HandlePhysRegKill(unsigned Reg, const MachineInstr *RefMI,
   return false;
 }
 
-/// HandlePhysRegKill - Calls the recursive version of HandlePhysRegKill. (See
-/// above for details.)
+/// HandlePhysRegKill - Returns true if the whole register is killed in the
+/// machine instruction. If only some of its sub-registers are killed in this
+/// machine instruction, then mark those as killed and return false.
 bool LiveVariables::HandlePhysRegKill(unsigned Reg, MachineInstr *RefMI) {
   SmallSet<unsigned, 4> SubKills;