[PATCH] Fix fallout from r219557

Anton Korobeynikov anton at korobeynikov.info
Wed Nov 12 15:20:42 PST 2014


I have to revert this revision - it was breaking some objc stuff.

One of the failed tests looks like:

  printf("%s", (i ? i : i));  // expected-warning{{values of type 'NSInteger' should not be used as format arguments; add an explicit cast to 'long' instead}}

After the change the type of (?:) seems to be int / long here due to promotion happened. Is this intended behavior? I'm not familliar objc that much..

http://reviews.llvm.org/D6217






More information about the cfe-commits mailing list