aaron.ballman accepted this revision. aaron.ballman added a comment. This revision is now accepted and ready to land.
Aside from a few minor nits, LGTM! ================ Comment at: test/FixIt/fixit-format-ios.m:2 +// RUN: cp %s %t +// RUN: %clang_cc1 -triple thumbv7-apple-ios8.0.0 -fsyntax-only -fblocks -Wformat -fixit %t +// RUN: grep -v CHECK %t | FileCheck %s ---------------- I don't think you need to enable `-fblocks` for this test. ================ Comment at: test/FixIt/fixit-format-ios.m:12-14 + // For thumbv7-apple-ios8.0.0 + // the underlying type of size_t is unsigned long + // and for ssize_t it is long. ---------------- Reflow these comments. Repository: rL LLVM https://reviews.llvm.org/D38159 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
