Don't try to emit nullability fix-its within/around macros.
authorJordan Rose <jordan_rose@apple.com>
Mon, 19 Dec 2016 22:35:24 +0000 (22:35 +0000)
committerJordan Rose <jordan_rose@apple.com>
Mon, 19 Dec 2016 22:35:24 +0000 (22:35 +0000)
commit281139c159cc26a6e7aadcab25e607ea43aaae65
treef85ad0e47382f4ce37569215a667bfe1191d1c77
parent5b1ee2fad97ceec55c6b22154b425954649a95b8
Don't try to emit nullability fix-its within/around macros.

The newly-added notes from r290132 are too noisy even when the fix-it
is valid. For the existing warning from r286521, it's probably the
right decision 95% of the time to put the change outside the macro if
the array is outside the macro and inside otherwise, but I don't want
to overthink it right now.

Caught by the ASan bot!

More rdar://problem/29524992

llvm-svn: 290141
clang/lib/Sema/SemaType.cpp
clang/test/FixIt/Inputs/nullability.h
clang/test/FixIt/nullability.mm