[PATCH] D65991: [llvm-objcopy] Move duplicate tablegen from objcopy and strip into one file

Mike Pozulp via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Aug 14 22:00:52 PDT 2019


mmpozulp added a comment.

In D65991#1624878 <https://reviews.llvm.org/D65991#1624878>, @jhenderson wrote:

> In D65991#1624866 <https://reviews.llvm.org/D65991#1624866>, @MaskRay wrote:
>
> > > Copied from llvm/tools/llvm-objcopy/StripOpts.td
> >
> > Phabricator is smart?
>
>
> I //think// it's to do with how the patch is written. I'm not sure though :/


Git is smart. The top of the patch says "copy from" / "copy to" https://reviews.llvm.org/file/data/ujocforfsc2wd7mdfbdg/PHID-FILE-gbjfb3ce67cfiabcoyar/D65991.id214422.diff. The manpage git-diff(1) explains the patch format but does not describe git's copying detection algorithm. Does anyone know? I'd love to find out more. @MaskRay, you have sparked my interest! :)


Repository:
  rL LLVM

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

https://reviews.llvm.org/D65991





More information about the llvm-commits mailing list