Remove non-test changes for now

Author: Sirraide

clang/docs/ReleaseNotes.rst

@@ -36,10 +36,6 @@ Potentially Breaking Changes
 
 - The Objective-C ARC migrator (ARCMigrate) has been removed.
 
-- ``-Wreturn-type`` now defaults to an error in all language modes; like other
-  warnings, it can be reverted to just being a warning by specifying
-  ``-Wno-error=return-type``.
-
 C/C++ Language Potentially Breaking Changes
 -------------------------------------------
 

clang/include/clang/Basic/DiagnosticSemaKinds.td

@@ -717,7 +717,7 @@ def warn_falloff_nonvoid : Warning<
   "non-void "
   "%enum_select<FalloffFunctionKind>{%Function{function}|%Block{block}|%Lambda{lambda}|%Coroutine{coroutine}}0"
   " does not return a value%select{| in all control paths}1">,
-  InGroup<ReturnType>, DefaultError;
+  InGroup<ReturnType>;
 def err_falloff_nonvoid : Error<
   "non-void %select{function|block|lambda|coroutine}0 "
   "does not return a value%select{| in all control paths}1">;