[PATCH] D52374: [MachineCopyPropagation] Reimplement CopyTracker in terms of register units

Justin Bogner via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Sep 21 12:08:52 PDT 2018


bogner created this revision.
bogner added a reviewer: MatzeB.
Herald added subscribers: llvm-commits, kristof.beyls, mcrosier.
Herald added a reviewer: javed.absar.
bogner added a dependency: D52370: [MachineCopyPropagation] Rework how we manage RegMask clobbers.

Change the copy tracker to keep a single map of register units instead
of 3 maps of registers. This gives a very significant compile time
performance improvement to the pass. I measured a 30-40% decrease in
time spent in MCP on x86 and AArch64 and much more significant
improvements on an out of tree target with more registers.


Repository:
  rL LLVM

https://reviews.llvm.org/D52374

Files:
  lib/CodeGen/MachineCopyPropagation.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D52374.166531.patch
Type: text/x-patch
Size: 5475 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180921/39950ce6/attachment.bin>


More information about the llvm-commits mailing list