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

Matthias Braun via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Sep 24 17:31:38 PDT 2018


MatzeB accepted this revision.
MatzeB added a comment.
This revision is now accepted and ready to land.

I'm convinced now the clobbering works correctly because as clobberRegister does not just clobber the register at hand but the whole destination register of the copy touched by the COPY at hand.
LGTM (this time with the right review :)


https://reviews.llvm.org/D52374





More information about the llvm-commits mailing list