1) In the Expression Compilers, reference the copies in Concord.
2) Everywhere else, change the signatures to match the Concord versions (PreserveSig everywhere, SymbolToken for tokens).

I've separately checked in an analyzer enforcing consumption of PreserveSig hresults.

TODO: additional cleanup is required - see DevDiv #1098556 and DevDiv #1098557.

CR: greggm (changeset 1389085)

***NO_CI***
 (changeset 1389083)

TextDocumentState.LoadTextAsync previously allowed InvalidDataExceptions to optionally propagate out. It now always catches InvalidDataExceptions and returns an empty SourceText. It also supports an option for reporting the exception as a WorkspaceFailed event. DocumentState reports the exception, while TextDocumentState does not. (changeset 1389074)

This part fixes the production of syntax tree diagnostics, so that no diagnostic is ever produced with a location that is outside the tree. (changeset 1389066)

VB CLSComplianceChecker: Visit declaration in each module separately to avoid dealing with merged namespaces. Merged namespaces do not have containing module, which was causing a NullReferenceException.
***NO_CI***
 (changeset 1389009)

For DevDiv #1092620, we introduced a number of PreserveSig COM calls.  Introduce a diagnostic that will help us catch dropped hresults.

CR: srivatsn; kevinpi; pharring (changeset 1388977)

1. Remove CancellationToken from AsyncQueue<T> constructor. This type is not an async operation, nor does it have the concept of cancellation, hence it doesn't really make sense to have a CancellationToken as parameter. Caller is now responsible for handling cancellation
2. The SetException, Completed and Enqueue methods are inherently racy methods. Added a Try variant of all methods which returns false on failure. The non-Try variants will throw on failure.
3. Separated out the cancellation portion of DequeueAsync. This simplified the rest of the logic in the type and allowed us to remove WaiterTaskArguments as there is no longer a race on completing the TaskCompletionSource. This is an internal implementation detail only.
4. Added a series of unit tests to codify the behavior as I understand it after discussions with Neal.

***NO_CI***
 (changeset 1388936)

The spec for how nameof should work in the face of extension methods is not really worked out, and the implementation is hacky and unlikely to align with an eventual rational specification. So we've decided to make it an error today so we can work out and implement some intentional rational behavior in the future without breaking backward compatibility. Nobody seems to use this particular intersection of features anyway. (changeset 1388917)

***NO_CI***
 (changeset 1388887)

These are being removed because they lead people to incorrectly building CodeFix providers that precompute the fix before registering the CodeAction's, as opposed to have the CodeAction itself do the work on a deferred basis.  (changeset 1388862)

Marks CSharpKind and VBKind extension methods as Obsolete and adds equivalent extension methods Kind for both languages.
Marks CSharpKind and VBKind instance methods on language specific syntax nodes Obsolete as well. (changeset 1388816)

Change the display name of the roslyn analyzers to "Roslyn Analyzers" to avoid conflicting with the "Roslyn Diagnostics" window.

Bonus: move the project into the folder hierarchy.

CR: srivatsn; kevinpi (changeset 1388791)

Allow PE and PDB streams to be arbitrary writable streams. If the specified stream doesn't fulfill implementation specific requirements of the PE/PDB writer use an intermediate in-memory stream and write its content to the user specified one at the end of emit.

Fixes #349.
 (changeset 1388751)

Add a better description for people who encounter the diagnostic about prefixed verbatim crefs. (changeset 1388376)

We also add all of the relevant optimizations requested for current and future
formatting performance improvements. We now use full overload resolution
(using a new binder type specialized for synthesizing invocations) to
select an appropriate string.Format method or format factory method, which
enables it to handle params, generics, arbitrary argument conversions, etc, and we
optimize the special case of no fill-ins by processing the {{ and }} escapes,
producing a literal in the generated code. (changeset 1388224)

Within the compiler layer, we want to access properties like Category, DefaultSeverity, IsEnabledByDefault, etc., off of Diagnostic directly rather than going through Diagnostic.Descriptor. Roslyn diagnostic RS0013 enforces this.

However, it fires within Diagnostic itself, even though this is the one place where accessing Descriptor is legitimate. This change adds suppressions to these occurrences, but the diagnostic should be updated to not fire in Diagnostic itself. A bug has been filed to track that work, and updating the Roslyn solutions to use new toolset & diagnostics packages with the fix.
 (changeset 1388110)

Index expressions in the index initializers should be evaluated exactly once.
In a case of a nested initializer, we were evaluating as many times as we have members in the RHS. That resulted in a number of bugs reported.

Working on this fix revealed some more issues with params and named indices not being supported properly.

Fixes #449
 (changeset 1388102)

Bug 1022828: Buffer overrun in PdbWriter - off-by-one error, we were writing to the SymWriter's stack beyond a reserved buffer.
EnC apply tests can run 32bit.
Resource tests were incorrectly converting a pointer to 32bit integer. (changeset 1387932)

Add a test for 981208. The new cache gets collected when you call Workspace.Dispose() and remove references to the workspace and things that hold onto it, so we just need regression test. (changeset 1387925)

Call WithDocumentText when fed with the same text sometimes doesn't return the same instance of the Document because TextSource, a WeakReference, sometimes loses the reference and the perf optimization to return the same instance of the Document is not possbile. (changeset 1387876)

We shouldn't crash just because a PDB is missing information for a specific method.

It looks like the other callers of GetMethodByVersion handle null appropriately.

CR: ChuckS; KevinH (changeset 1387853)

***NO_CI***
 (changeset 1387627)

We weren't handling a number of the different types of state machine fields, and the Kind returned for hoisted, synthesized local fields ("$S" prefix) looked wrong (returned "HoistedLocalField" instead of "HoistedSynthesizedLocalField") (changeset 1387592)

***NO_CI***
 (changeset 1387571)

This changes EquatableAnalyzer to use a OfType call instead of Cast.  Using Cast here is incorrect and is causing an exception to be thrown in our toolset.

     Bug 1097052
***NO_CI***
 (changeset 1387470)

     The most immediate issue here is cancellation of the CancellationToken passed to AsyncQueue<T> will unconditionally call TaskCompletionSource.SetResult for the main Task value in the queue.  This cancellation can happen long after or in parallel with the completion of AsyncQueue<T>.  When this happens an exception is thrown.  This exception is causing the VBCSCompiler process to crash because it expects cancelling a token to be a safe operation.

     Longer term there are other issues in AsyncQueue<T> that need to be fixed (several race conditions).  Once this is checked in I willmove 1097123 to a Pri 1 and track the remaining fixes with that.   (changeset 1387407)

Changed SymbolEditor to use SyntaxEditor for editting declarations. (changeset 1387404)

NameSyntaxComparer was causing stack overflow due to infinite recursion. (changeset 1387310)

This is consistent with the spec and Dev12.
***NO_CI***
 (changeset 1387266)

***NO_CI***
 (changeset 1387232)

* Add 'when' keyword recommender to the completion list
* Ensure that the completion appears inside exception filters.
* Add 'when' keyword span to keyword highlighting for try..catch statements.
* Ensure that parenthesis simplication works catch filter expressions
* Add tests for 'when' classification (it already was classified correctly) (changeset 1387214)