[PATCH] D12588: Introduce target hook for optimizing register copies

Quentin Colombet via llvm-commits llvm-commits at lists.llvm.org
Tue Sep 8 13:41:40 PDT 2015


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

Thanks Matt for the clarifications.
LGTM.

if shareSameRegisterFile is not used, maybe sink its implementation in the generic shouldRewriteCopySrc.
Targets can still call the parent method when overriding.

This is just a suggestion though, I am fine with the patch as it is. Up to you.

Cheers,
-Quentin


http://reviews.llvm.org/D12588





More information about the llvm-commits mailing list