[PATCH] D82638: [MachineCopyPropagation] BackwardPropagatableCopy: add check for hasOverlappingMultipleDef

Kai Luo via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jul 29 08:27:02 PDT 2020


lkail added a comment.

In D82638#2118048 <https://reviews.llvm.org/D82638#2118048>, @efriedma wrote:

> If the definition is earlyclobber, do we also need to check for overlapping uses?

Prior `propagateDefs`, there is an invalidation of earlyclobber registers, so in `propagateDefs`, if `MODef` is earlyclobber, there should not be a corresponding COPY to propagate from.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D82638/new/

https://reviews.llvm.org/D82638



More information about the llvm-commits mailing list