Merge pull request #4267 from akyrtzi/warn_unused_result-accept-fixit

[FixCode] Accept the fixit for the warning that @warn_unused_result is the default now.

Author: akyrtzi

lib/FrontendTool/FrontendTool.cpp

@@ -636,6 +636,7 @@ class JSONFixitWriter : public DiagnosticConsumer {
         Info.ID == diag::selector_literal_deprecated_suggest.ID ||
         Info.ID == diag::attr_noescape_deprecated.ID ||
         Info.ID == diag::attr_autoclosure_escaping_deprecated.ID ||
+        Info.ID == diag::attr_warn_unused_result_removed.ID ||
         Info.ID == diag::any_as_anyobject_fixit.ID ||
         Info.ID == diag::deprecated_protocol_composition.ID ||
         Info.ID == diag::deprecated_protocol_composition_single.ID ||

test/FixCode/fixits-apply.swift

@@ -233,6 +233,8 @@ func testescape(rec: ()->()) {
   fnWithClosure { rec() }
 }
 
+@warn_unused_result func testDeprecatedAttr() -> Int { return 0 }
+
 protocol Prot1 {}
 protocol Prot2 {
   associatedtype Ty = Prot1

test/FixCode/fixits-apply.swift.result

@@ -236,6 +236,8 @@ func testescape(rec: @escaping ()->()) {
   fnWithClosure { rec() }
 }
 
+func testDeprecatedAttr() -> Int { return 0 }
+
 protocol Prot1 {}
 protocol Prot2 {
   associatedtype Ty = Prot1