[PATCH] D19393: Move Checkers.inc to clang/include/.../Checkers

Stephen Hines via cfe-commits cfe-commits at lists.llvm.org
Sat Apr 23 17:28:55 PDT 2016


srhines added a comment.

Anna, if I scroll over the new include file in phabricator, it shows as a proper file move (in a yellow column at the start of the right diff - hover for it to say this). Every line is the same from the original file, as this is being moved only to fix up the relative paths, which are used across multiple directories.

I know that git and svn both handle renames properly (via mv), but I have no clue whether phabricator + git-svn are going to preserve history in the nicest way possible. If there are suggestions there for minimizing trouble, we're happy to listen.


http://reviews.llvm.org/D19393





More information about the cfe-commits mailing list