[PATCH] D38159: [clang] Fix printf fixit for objc specific types
    Aaron Ballman via Phabricator via cfe-commits 
    cfe-commits at lists.llvm.org
       
    Fri Sep 22 06:07:17 PDT 2017
    
    
  
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
    
    
More information about the cfe-commits
mailing list