RKSimon added a comment. Any idea why phabricator is showing so many unchanged lines from X86InstrInfo.cpp? Have you changed the line endings or something? They aren't appearing in the the downloaded diff FWIW. Repository: rL LLVM https://reviews.llvm.org/D33169