LGTM ================ Comment at: lib/Target/X86/AsmParser/X86AsmParser.cpp:35 @@ -33,1 +34,3 @@ +#include <memory> + ---------------- Looks like it is common to not add an empty line between include groups in LLVM. http://llvm-reviews.chandlerc.com/D2881