[PATCH] D31834: Remove unnecessary bitvector clear

Xin Tong via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 22 22:39:03 PDT 2017


trentxintong added a comment.

My understanding is only 1 class DeadMachineInstructionElim object will be constructed and used for all the functions in the module by the pass manager. However the LivePhysRegs is re-assigned at the beginning of the runOnMachineFunction for every basic block.  So the LivePhysRegs.clear() is really unnecessary.


https://reviews.llvm.org/D31834





More information about the llvm-commits mailing list