Fixes #21515

* IDE Changes

* tests

* PR comments

Quote variable/array expansions in shell scripts

Merging on behalf of @alrz. Thanks

Do not emit PDB path in PE file if not generating PDB

Property Reference Underlying Instance Type

Ref ternary should be shielded form optimization in the containing expression that may expose it to mutations

Prevent crash when no capitalization is specified (invalid naming style)

Ref ternary shoudl be shielded form optimization in the containing expression that may expose it to mutations

Just need to add it to the list of other possible writeable lvalues checked by WouldBeAssignableIfUsedAsMethodReceiver helper.

Fixes:#18220

Escape analysis for all expressions

Improve method type argument inference around tuples and address backwards compatibility break. (#22295)

Fixes #20583.

Fixes #21297 and #22004

Add an option to disable name suggestions

- makes val escape switch comprehensive (unhandled/unexpected nodes will cause an exception)
- tests for added expression kinds
- handle varargs correctly in the escape analysis + tests

* dotnet/master:
  Fix unit test failures from merge
  Add some more unit tests
  Add more unit tests as requested in previous review feedback
  Remove IArrayCreationExpression.ElementType and fix unit tests
  Fix unit tests after merge resolution
  Fix incorrect merge conflict resolution
  Add unit tests for error cases
  Missed checking in some changes in prior commit - adding them now
  Add unit tests for IArrayCreationExpression and IArrayInitializer

Add unit tests for IArrayCreationExpression and IArrayInitializer

* dotnet/master:
  Fix BOM

* dotnet/master: (351 commits)
  Avoid scheduling an unnecessary dispose task for EmptyAsyncToken
  Support for ref-readonly locals (#22269)
  ported tests from https://github.com/dotnet/roslyn/pull/21263 (#22296)
  NormalizeWhitespace should not add a space in nullable type (#22234)
  Fix the work item number associated with the unit test
  Add unit test
  Terminate a sentence
  Add instructions for upgrading PowerShell
  Unsafe local should require /unsafe flag on compilation (#22268)
  CR feedback
  Not poisoning restricted types known to be stack only.
  Typo
  Error messsage should mention C# 7.0 not C# 7 (#22255)
  Package descriptions for CSharp and VB should include summary blurb (#22166)
  Fix a trivial issue in how IsLifted is calculated for C# compound assignment operator
  Do not zero out "With" statement target expression locals referenced within a lambda. (#22223)
  Fix unit test failure from merge
  Mark "private protected", "ref readonly" and other C# 7.2 features as merged (#22209)
  Add unit tests for IAwaitExpression and make the API public again
  more test fixes due to merge conflicts
  ...

Fix BOM

Fix build parallelism issue with documentation file generation

Avoid scheduling an unnecessary dispose task for EmptyAsyncToken

Similar to BUILD_ARGS in 3cead2d5, PUBLISH_ARGS needs to undergo word
expansion.

Similar to BUILD_ARGS in 3cead2d5, RESTORE_ARGS needs to undergo word
expansion.

Lack of quoting causes issues when files have spaces or are empty names.
Additionally, some arrays (though single elements) were treated like
simple variables.

The careful reader will note that I have left BUILD_ARGS

dotnet build "${ROOT_PATH}"/CrossPlatform.sln ${BUILD_ARGS}

untouched, because it contains spaces meant to separate arguments. Word
expansion is somewhat desired there (I believe) to ensure the arguments
are properly parsed. Unfortunately, BUILD_ARGS may contains paths with
spaces in them, causing this to still fail.

Fix #22259

Fix a trivial issue in how IsLifted is calculated for C# compound ass…

Fixes #22272

Add unit tests for IAwaitExpression and make the API public again

Ref-readonly locals are mostly identical to ref-readonly parameters. The
most important difference is that when possibly mutating methods are
called on ref-readonly locals of struct type, a proper temporary is
created before calling the method.