[PATCH] D16787: [AArch64] Allocate the modified and used regs once per function.

Chad Rosier via llvm-commits llvm-commits at lists.llvm.org
Tue Feb 2 07:09:00 PST 2016


mcrosier added a comment.

Committed r259510.


================
Comment at: lib/Target/AArch64/AArch64LoadStoreOptimizer.cpp:1020
@@ -1016,2 +1019,3 @@
   // Track which registers have been modified and used between the first insn
   // and the second insn.
+  ModifiedRegs.reset();
----------------
mgrang wrote:
> Can you please update these comments to reflect that the bitfield trackers are being reset here?
Sure, Mandeep.  I think the call to reset makes it pretty clear we're resetting the vector at this point, so I didn't add a comment here.  However, I did add an additional comment where the vectors are being resized.


http://reviews.llvm.org/D16787





More information about the llvm-commits mailing list