…0201.2 (#1792)

[master] Update dependencies from dotnet/runtime

* Only process direct calls and reflection method references.

* Clean test

* Add tests

* Check for null origin

* Warn on DynamicallyAccessedMember kind.

* Update warnings message for Requires Unreferenced Code

* Mark interfaces of types accessed via reflection

* Fix whitespace

* Add includeInterfaceTypes parameter

* Keep set action for methods in MarkEntireType

* Add test for dynamic dependency with RUC.

* Fix DependencyKind in MarkTypeForDynamicallyAccessedMembers
Pass null to origin's parameter when processing optimized overrides
Add DynamicDependency to list of valid dependency kinds in ProcessRequiresUnreferencedCode

* Fix formatting

Linker has intrinsic handling for some methods which are annotated with `RequiresUnreferencedCodeAttribute`. The intrinsic handling will sometimes recognize the pattern and avoid generating a warning, at other times it will generate a better more specific warning. In cases where this happens the generic handler for `RequiresUnreferencedCodeAttribute` should not kick in, as it would still produce a warning.

The change:
* For direct calls, virtual calls and `.ctor` calls rely on `ReflectionMethodBodyScanner` to handle all warnings caused by `RequiresUnreferencedCodeAttribute`. It has knowledge about intrinsic handling and can handle all non-intrisic calls as well.
* Limit the generic handler in `MarkMethod` to all other cases (typically reflection or other indirect references to the method)

Test changes:
* We already have test cases which trigger these conditions, but we don't have a way to validate that extra warnings were not produced
* Added `ExpectedNoWarnings` test attribute which validates no warnings other than those explicitly stated by the test are produced.
* Used it on two test cases which trigger the interesting condition - and fixed the tests to correctly expect all warnings