vsapsai accepted this revision. vsapsai added a comment. This revision is now accepted and ready to land.
One small note about `-fdiagnostics-parseable-fixits`. The rest looks good to me. And the test looks readable. ================ Comment at: FixIt/fixit-typedef-instead-of-typename-typo.cpp:1 +// RUN: %clang_cc1 -fsyntax-only -fdiagnostics-parseable-fixits -verify %s + ---------------- I don't think you need `-fdiagnostics-parseable-fixits` here. Though it is required in the second RUN line. https://reviews.llvm.org/D42170 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits