dexonsmith added a subscriber: arphaman.
dexonsmith added inline comments.
================
Comment at: clang/test/FixIt/format.m:195-208
- NSLog(@"%C", 0x260300); // expected-warning{{format specifies type
'unichar' (aka 'unsigned short') but the argument has type 'int'}}
- // CHECK-NOT: fix-it:"{{.*}}":{[[@LINE-1]]:11-[[@LINE-1]]:13}:"%d"
- // CHECK: fix-it:"{{.*}}":{[[@LINE-2]]:16-[[@LINE-2]]:16}:"(unsigned short)"
typedef unsigned short unichar;
-
- NSLog(@"%C", 0x260300); // expected-warning{{format specifies type
'unichar' (aka 'unsigned short') but the argument has type 'int'}}
----------------
aaron.ballman wrote:
> Hmmm, did we intend to change the behavior here? `%C` is an extension, so
> it's not clear to me if we wanted to modify this behavior or not. CC
> @rjmccall and @dexonsmith for some ObjC opinions.
I’m not sure, but I agree it’s suspicious. Besides John, Mike Ash might be
another person to check with (`@`-completion isn’t working for me on my phone
right now, but perhaps someone else will have more luck?). Or, @arphaman, any
other suggestions?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D132568/new/
https://reviews.llvm.org/D132568
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
