This revision was automatically updated to reflect the committed changes.
Closed by commit rL310874: [Sema] Improve some -Wunguarded-availability
diagnostics (authored by epilk).
Changed prior to commit:
https://reviews.llvm.org/D36200?vs=109252=111051#toc
Repository:
rL LLVM
arphaman accepted this revision.
arphaman added a comment.
This revision is now accepted and ready to land.
Sorry for the delay, LGTM
https://reviews.llvm.org/D36200
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
erik.pilkington added a comment.
Ping!
https://reviews.llvm.org/D36200
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
arphaman added a comment.
Right, sorry, I didn't realize that code was moved.
https://reviews.llvm.org/D36200
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
erik.pilkington added a comment.
In https://reviews.llvm.org/D36200#829104, @arphaman wrote:
> This needs a test for the fixits as well, see test/FixIt/fixit-availability*
Why? This patch doesn't change the behavior of the fixits, so there isn't any
new behavior to test.
arphaman added a comment.
This needs a test for the fixits as well, see test/FixIt/fixit-availability*
https://reviews.llvm.org/D36200
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
erik.pilkington created this revision.
This patch improves the decl-level unguarded availability warnings to use the
same messages as the function-level ones. This makes the diagnostic have
different parameters than deprecated/unavailable, so I moved some things around
in
