Merge pull request #2225 from wtgodbe/prTestRun

Mac tests will run automatically on PR

mac tests will run automatically on PR

Merge pull request #2216 from hoyMS/master

Porting CoreCLR JIT SelfHost tests

Merge pull request #2210 from stephentoub/semaphore_in_mscorlib

Port System.Threading.Semaphore from corefx to mscorlib

Merge pull request #2205 from sergiy-k/executablememory

Implement an allocator for executable (JIT) memory in PAL

Porting the folloiwng CoreCLR JIT SelfHost tests:

Jit\Methodical\arrays\dbg\arrres.exe
Jit\Methodical\arrays\rel\arrres.exe
Jit\Methodical\arrays\speed_dbg\arrres.exe
Jit\Methodical\arrays\speed_rel\arrres.exe

jit\methodical\explicit\basic\_dbgrefarg_c.csproj
jit\methodical\explicit\basic\_dbgrefarg_f4.csproj
jit\methodical\explicit\basic\_dbgrefarg_f8.csproj
jit\methodical\explicit\basic\_dbgrefarg_i1.csproj
jit\methodical\explicit\basic\_dbgrefarg_i2.csproj
jit\methodical\explicit\basic\_dbgrefarg_i4.csproj
jit\methodical\explicit\basic\_dbgrefarg_o.csproj
jit\methodical\explicit\basic\_dbgrefarg_s.csproj
jit\methodical\explicit\rotate\_dbgrotarg_double.csproj
jit\methodical\explicit\rotate\_dbgrotarg_float.csproj
jit\methodical\explicit\rotate\_dbgrotarg_objref.csproj
jit\methodical\explicit\rotate\_dbgrotarg_valref.csproj
jit\methodical\explicit\basic\_opt_dbgrefarg_c.csproj
jit\methodical\explicit\basic\_opt_dbgrefarg_f4.csproj
jit\methodical\explicit\basic\_opt_dbgrefarg_f8.csproj
jit\methodical\explicit\basic\_opt_dbgrefarg_i1.csproj
jit\methodical\explicit\basic\_opt_dbgrefarg_i2.csproj
jit\methodical\explicit\basic\_opt_dbgrefarg_i4.csproj
jit\methodical\explicit\basic\_opt_dbgrefarg_o.csproj
jit\methodical\explicit\basic\_opt_dbgrefarg_s.csproj
jit\methodical\explicit\rotate\_opt_dbgrotarg_double.csproj
jit\methodical\explicit\rotate\_opt_dbgrotarg_float.csproj
jit\methodical\explicit\rotate\_opt_dbgrotarg_objref.csproj
jit\methodical\explicit\rotate\_opt_dbgrotarg_valref.csproj
jit\methodical\explicit\basic\_opt_relrefarg_c.csproj
jit\methodical\explicit\basic\_opt_relrefarg_f4.csproj
jit\methodical\explicit\basic\_opt_relrefarg_f8.csproj
jit\methodical\explicit\basic\_opt_relrefarg_i1.csproj
jit\methodical\explicit\basic\_opt_relrefarg_i2.csproj
jit\methodical\explicit\basic\_opt_relrefarg_i4.csproj
jit\methodical\explicit\basic\_opt_relrefarg_o.csproj
jit\methodical\explicit\basic\_opt_relrefarg_s.csproj
jit\methodical\explicit\rotate\_opt_relrotarg_double.csproj
jit\methodical\explicit\rotate\_opt_relrotarg_float.csproj
jit\methodical\explicit\rotate\_opt_relrotarg_objref.csproj
jit\methodical\explicit\rotate\_opt_relrotarg_valref.csproj
jit\methodical\explicit\basic\_relrefarg_c.csproj
jit\methodical\explicit\basic\_relrefarg_f4.csproj
jit\methodical\explicit\basic\_relrefarg_f8.csproj
jit\methodical\explicit\basic\_relrefarg_i1.csproj
jit\methodical\explicit\basic\_relrefarg_i2.csproj
jit\methodical\explicit\basic\_relrefarg_i4.csproj
jit\methodical\explicit\basic\_relrefarg_o.csproj
jit\methodical\explicit\basic\_relrefarg_s.csproj

Porting the follwing test from CoreCLR JIT SelfHost:

jit\methodical\explicit\basic\_dbgrefarg_c.csproj
jit\methodical\explicit\basic\_dbgrefarg_f4.csproj
jit\methodical\explicit\basic\_dbgrefarg_f8.csproj
jit\methodical\explicit\basic\_dbgrefarg_i1.csproj
jit\methodical\explicit\basic\_dbgrefarg_i2.csproj
jit\methodical\explicit\basic\_dbgrefarg_i4.csproj
jit\methodical\explicit\basic\_dbgrefarg_o.csproj
jit\methodical\explicit\basic\_dbgrefarg_s.csproj
jit\methodical\explicit\rotate\_dbgrotarg_double.csproj
jit\methodical\explicit\rotate\_dbgrotarg_float.csproj
jit\methodical\explicit\rotate\_dbgrotarg_objref.csproj
jit\methodical\explicit\rotate\_dbgrotarg_valref.csproj
jit\methodical\explicit\basic\_opt_dbgrefarg_c.csproj
jit\methodical\explicit\basic\_opt_dbgrefarg_f4.csproj
jit\methodical\explicit\basic\_opt_dbgrefarg_f8.csproj
jit\methodical\explicit\basic\_opt_dbgrefarg_i1.csproj
jit\methodical\explicit\basic\_opt_dbgrefarg_i2.csproj
jit\methodical\explicit\basic\_opt_dbgrefarg_i4.csproj
jit\methodical\explicit\basic\_opt_dbgrefarg_o.csproj
jit\methodical\explicit\basic\_opt_dbgrefarg_s.csproj
jit\methodical\explicit\rotate\_opt_dbgrotarg_double.csproj
jit\methodical\explicit\rotate\_opt_dbgrotarg_float.csproj
jit\methodical\explicit\rotate\_opt_dbgrotarg_objref.csproj
jit\methodical\explicit\rotate\_opt_dbgrotarg_valref.csproj
jit\methodical\explicit\basic\_opt_relrefarg_c.csproj
jit\methodical\explicit\basic\_opt_relrefarg_f4.csproj
jit\methodical\explicit\basic\_opt_relrefarg_f8.csproj
jit\methodical\explicit\basic\_opt_relrefarg_i1.csproj
jit\methodical\explicit\basic\_opt_relrefarg_i2.csproj
jit\methodical\explicit\basic\_opt_relrefarg_i4.csproj
jit\methodical\explicit\basic\_opt_relrefarg_o.csproj
jit\methodical\explicit\basic\_opt_relrefarg_s.csproj
jit\methodical\explicit\rotate\_opt_relrotarg_double.csproj
jit\methodical\explicit\rotate\_opt_relrotarg_float.csproj
jit\methodical\explicit\rotate\_opt_relrotarg_objref.csproj
jit\methodical\explicit\rotate\_opt_relrotarg_valref.csproj
jit\methodical\explicit\basic\_relrefarg_c.csproj
jit\methodical\explicit\basic\_relrefarg_f4.csproj
jit\methodical\explicit\basic\_relrefarg_f8.csproj
jit\methodical\explicit\basic\_relrefarg_i1.csproj
jit\methodical\explicit\basic\_relrefarg_i2.csproj
jit\methodical\explicit\basic\_relrefarg_i4.csproj
jit\methodical\explicit\basic\_relrefarg_o.csproj
jit\methodical\explicit\basic\_relrefarg_s.csproj

Porting the folloiwng CoreCLR JIT SelfHost tests:

Jit\Methodical\arrays\dbg\arrres.exe
Jit\Methodical\arrays\rel\arrres.exe
Jit\Methodical\arrays\speed_dbg\arrres.exe
Jit\Methodical\arrays\speed_rel\arrres.exe

jit\methodical\explicit\basic\_dbgrefarg_c.csproj
jit\methodical\explicit\basic\_dbgrefarg_f4.csproj
jit\methodical\explicit\basic\_dbgrefarg_f8.csproj
jit\methodical\explicit\basic\_dbgrefarg_i1.csproj
jit\methodical\explicit\basic\_dbgrefarg_i2.csproj
jit\methodical\explicit\basic\_dbgrefarg_i4.csproj
jit\methodical\explicit\basic\_dbgrefarg_o.csproj
jit\methodical\explicit\basic\_dbgrefarg_s.csproj
jit\methodical\explicit\rotate\_dbgrotarg_double.csproj
jit\methodical\explicit\rotate\_dbgrotarg_float.csproj
jit\methodical\explicit\rotate\_dbgrotarg_objref.csproj
jit\methodical\explicit\rotate\_dbgrotarg_valref.csproj
jit\methodical\explicit\basic\_opt_dbgrefarg_c.csproj
jit\methodical\explicit\basic\_opt_dbgrefarg_f4.csproj
jit\methodical\explicit\basic\_opt_dbgrefarg_f8.csproj
jit\methodical\explicit\basic\_opt_dbgrefarg_i1.csproj
jit\methodical\explicit\basic\_opt_dbgrefarg_i2.csproj
jit\methodical\explicit\basic\_opt_dbgrefarg_i4.csproj
jit\methodical\explicit\basic\_opt_dbgrefarg_o.csproj
jit\methodical\explicit\basic\_opt_dbgrefarg_s.csproj
jit\methodical\explicit\rotate\_opt_dbgrotarg_double.csproj
jit\methodical\explicit\rotate\_opt_dbgrotarg_float.csproj
jit\methodical\explicit\rotate\_opt_dbgrotarg_objref.csproj
jit\methodical\explicit\rotate\_opt_dbgrotarg_valref.csproj
jit\methodical\explicit\basic\_opt_relrefarg_c.csproj
jit\methodical\explicit\basic\_opt_relrefarg_f4.csproj
jit\methodical\explicit\basic\_opt_relrefarg_f8.csproj
jit\methodical\explicit\basic\_opt_relrefarg_i1.csproj
jit\methodical\explicit\basic\_opt_relrefarg_i2.csproj
jit\methodical\explicit\basic\_opt_relrefarg_i4.csproj
jit\methodical\explicit\basic\_opt_relrefarg_o.csproj
jit\methodical\explicit\basic\_opt_relrefarg_s.csproj
jit\methodical\explicit\rotate\_opt_relrotarg_double.csproj
jit\methodical\explicit\rotate\_opt_relrotarg_float.csproj
jit\methodical\explicit\rotate\_opt_relrotarg_objref.csproj
jit\methodical\explicit\rotate\_opt_relrotarg_valref.csproj
jit\methodical\explicit\basic\_relrefarg_c.csproj
jit\methodical\explicit\basic\_relrefarg_f4.csproj
jit\methodical\explicit\basic\_relrefarg_f8.csproj
jit\methodical\explicit\basic\_relrefarg_i1.csproj
jit\methodical\explicit\basic\_relrefarg_i2.csproj
jit\methodical\explicit\basic\_relrefarg_i4.csproj
jit\methodical\explicit\basic\_relrefarg_o.csproj
jit\methodical\explicit\basic\_relrefarg_s.csproj

Merge pull request #2202 from mikem8361/sosfixes

Some SOS fixes/cleanup (bpmd, setclrpath, setsostid).

Some SOS fixes/cleanup (bpmd, setclrpath, setsostid).

Enable the module load notification when no module or methods are found.

setclrpath/setsostid print the current path/tid if no arguments. "setsostid -clear"
now clears the tid/os id mapping.

Added better exception hook to sos plugin to catch special CLRN exception. Before
multiple lldb "Exception breakpoints" where set each time bpmd added a new bp. Added
a "DoNotDeleteOrDisable" message to the exception breakpoint created.

Fix bpmd on release builds. RtlpRaiseException was being inlined and the ExceptionRecord
parameter was being optimized away causing the GetLastEventInformation api to failed
and the special CLRN exception to be ignored.

Add sos help support. "soshelp" displays help about a command.

Address code review feedback

Merge pull request #2211 from wtgodbe/prTestRun

Should allow for test suite runs on OSX on each PR

Fixed copyartifacts issue

Port System.Threading.Semaphore from corefx to mscorlib

In the full framework, System.Threading.Semaphore is defined in System.dll, which no longer exists in coreclr/corefx.  Thus Semaphore needs to be defined in either mscorlib in coreclr or in System.Threading.dll in corefx (either way it's exposed from the System.Threading contract).  Today it's implemented in System.Threading.dll in corefx, but this results in a very unnatural dependency on Unix from System.Threading.dll to libcoreclr's PAL layer.  This is because on Unix libcoreclr needs to emulate wait handles, and thus all of the implementation for a Win32 semaphore (for which Semaphore is a thin wrapper) is in libcoreclr.

To eliminate that unnatural dependency, this commit ports System.Threading.Semaphore to mscorlib.  It'll be type-forwarded as part of the System.Threading.dll partial facade.  A corresponding commit in corefx will remove all of the implementation from there.

Merge pull request #2209 from wtgodbe/prTestRun

Modifications to OS X PR test run

Added release

Merge pull request #2208 from wtgodbe/prTestRun

resolved naming conflict

resolved naming conflict

Merge pull request #2199 from wtgodbe/prTestRun

OS X tests should now be run with each PR

First crack

Implement an allocator for executable (JIT) memory in PAL

This change improves performance of CoreCLR with Server GC enabled by
about 30% according to ASP.NET benchmarks on Linux. The table below shows
number of requests per second that an ASP.NET benchmark could handle on my
machine before and after the change.

Pipeline        Before          After           Improvement
16              230K Req/sec    305K Req/sec    33%
256             240K Req/sec    340K Req/sec    42%

The problem was that with Server GC enabled, the GC initialization code was
reserving a large chunck (about 18GB on my machine) of virtual address
space during runtime initialization. Unfortunately, due to implementation
details of MM on Linux, GC memory was located next to the location of
libcoreclr. As a result, the runtime could not allocate memory for JIT'ed
code close to the coreclr library. Because of that the JIT'ed code had to
use jump stubs to call functions from the runtime (which can become very
expensive, for example, for write barriers).

This change fixes this issue by implementing a simple allocator that tries
to reserve (during process startup) a chuck of virtual memory that is
located near the coreclr library (within 2GB range) that can be later used
for JIT'ed code.

Merge pull request #2203 from dotnet-bot/from-tfs

Merge changes from TFS

Integrate GC fix to compensate for OS bug

[tfs-changeset: 1553463]

Merge pull request #2201 from dotnet-bot/from-tfs

Merge changes from TFS

Fix regression on desktop introduce by #2164

[tfs-changeset: 1553409]

Merge pull request #2125 from wtgodbe/macRunTest

runtest.sh now works on Mac as well as Linux

runtest.sh now works on Mac as well as Linux

Merge pull request #2194 from juergenhoetzel/pal-build-enable-ms-extensions

Enable Microsoft extensions for PAL compilation

Enable Microsoft extensions for PAL compilation

Clang 3.7 now requires passing the -fms-extensions flag to use the
__declspec language extension for declaration attributes.

Refs #2135

Merge pull request #1671 from wtgodbe/QueryThreadCycleTime

Implemented QueryThreadCycleTime (time.cpp) & GetThreadTimes (thread.…

Merge pull request #2168 from pgavlin/EnvFileChange

Clean up the CoreCLR test wrappers.

Merge pull request #2180 from justinvp/charenumerator

Remove duplicate code in CharEnumerator

Merge pull request #2177 from juergenhoetzel/mismatched-new-delete

Fix mismatched use of new and delete

Remove duplicate code in CharEnumerator

Change IEnumerator.Current to simply return Current, instead of
having duplicate code in both properties.

Fix mismatched use of new and delete

Clang > 2.7.0 emits warnings for mismatched uses of new and
delete (-Wmismatched-new-delete).

Refs #2135

Merge pull request #2171 from manu-silicon/patch-1

Update cross compilation instructions

Update cross compilation instructions

Provide exact instructions for installing the required packages to do
the cross compilation.

Add section about the manual change to the urcu header files as the fix
is only in later revisions of Ubuntu.

Merge pull request #2169 from dotnet-bot/from-tfs

Merge changes from TFS

Merge pull request #2162 from manu-silicon/patch-1

Use hkp protocol on port 80 to get mono

Integrate bug fixes from desktop

[tfs-changeset: 1552157]

Merge pull request #2167 from steveharter/StdArg

Fix unicode input on the command line

Merge pull request #2138 from LLITCHEV/SysVTests

Fixes for Issues #s 2105 and 2106.

Merge pull request #2166 from kyulee1/ildasm2

Enable ILDASM for Windows

Fix unicode input on the command line

Enable ILDASM for Windows

This enables building ILDASM with Cmake for Windows.
This ILDASM now depends on CoreCLR targeting cross-platform and thus I dropped some features like GUI/PDB -- default output is console.
Metadata related APIs in CoreCLR are directly exported, and used in ILDASM:
 MetaDataGetDispenser
 GetMetaDataInternalInterface
 GetMetaDataInternalInterfaceFromPublic
 GetMetaDataPublicInterfaceFromInternal
The code path is diverged by a definition FEATURE_CORECLR.
There are still Window specific components. Among others, resource file/dll generation is the one that should be ported.

Merge pull request #2161 from jkotas/xunit-update

Update xunit version to 2.1.0

Merge pull request #2160 from alexandrnikitin/gentree-remove-duplication

Remove duplicate assignment in Compiler::gtNodeHasSideEffects

Clean up the CoreCLR test wrappers.

- Various simplifications to the Xunit wrapper and the test helper
- Allow the specification of the test env script and the CORE_ROOT
  directory at the command line on Windows.

Merge pull request #2164 from janvorli/fix-wait-for-secondary-threads

Make app shutdown wait for secondary threads

Make app shutdown wait for secondary threads

This change makes the CoreCLR behavior the same as the desktop CLR
w.r.t. the app shutdown and secondary threads. Now the application
waits for secondary threads to exit before exiting.

Use hkp protocol on port 80 to get mono

Minor formatting to be consistent with the rest of the document.

Remove duplicate assignment in Compiler::gtNodeHasSideEffects

Update auto-generated project.lock.json

Update xunit version to 2.1.0

Merge pull request #2156 from sergiy-k/disableconcurrentgc

Don't enable Concurrent GC on Unix by default

Fixes for Issues #s 2105 and 2106.

For issue 2105 after more analysis there might be a very rare situation
(since this is not hit by the tests we have in coreclr and corefx) where
(even though there are not XMM callee saved register on System V) a
circular dependency might occur when homing params passed in XMM
registers. Used the right instruction for doing copying of XMM reg to XMM
reg

For issue # 2106 make sure for 2 register passed struct to use the
already available scOtherReg in struct psiScope.

Merge pull request #2152 from CarolEidt/ryujit-porting-doc

Document porting RyuJIT to different platforms

Don't enable Concurrent GC on Unix by default

This change temporary disables the code in Unix hosting APIs that switches Concurrent GC on by default on Unix because this feature is not supported yet and having this flag set causes a performance regression (this problem is tracked by issue https://github.com/dotnet/coreclr/issues/2155).

Merge pull request #2154 from jashook/master

Small architecture specific changes to fix the build on linux arm64.

Small architecture specific changes to fix the build on linux arm64.

Merge pull request #2111 from davmason/master

Porting EventSource from the nuget package to coreclr

Document porting RyuJIT to different platforms

This document outlines the various RyuJIT components that must be modified
to port to a new platform.

Restoring the fix to WriteEvent with null/empty arrays that was over written by my port.

Merge remote-tracking branch 'upstream/master'

Merge pull request #2149 from alexandrnikitin/optimizer-remove-unsed-var

Remove unused variable from Optimizer Compiler::optHoistLoopExprsForTree method

Merge pull request #2132 from AndyAyersMS/FixLinuxTestScript

Fix generated test script to properly check for debugger

Merge pull request #2142 from steveharter/IcuPerf

Improve DateTime.Parse performance by reducing string comparisons

Merge pull request #2151 from dotnet-bot/from-tfs

Merge changes from TFS

Improve DateTime.Parse performance by reducing string comparisons

Open sourcing ILDASM

[tfs-changeset: 1551732]

Merge pull request #2148 from manu-silicon/build

Build script now use current platform to compile

Merge pull request #2147 from manu-silicon/minor

Consistency fix for shell variables

Remove unused variable from Optimizer Compiler::optHoistLoopExprsForTree method

Build script now use current platform to compile

No need to specify the architecture to compile on Linux, we use
"uname -p" to infer it.
Removed __MSBuildBuildArch which was not really used, we can use
__BuildArch instead.
Added warning for x86.

Consistency fix for shell variables

Use 0 instead of false since later we compare it with 1.

Merge pull request #2136 from gsfreema/docs

Cleaning up Contributing Workflow documentation

Merge pull request #2134 from jkotas/NO_CRT

Delete code under !NO_CRT define

Delete code under !NO_CRT define

It is always defined - even in internal builds

Cleaning up Contributing Workflow documentation

The line 'Please following our commit messages guidance, below.' is
incorrect since the commit messages section is above.  To future proof
this line, I'm updating the line to provide a link to the Commit
Messages section.  Also, correcting 'Please following' to say 'Please
follow'.

Merge pull request #2129 from maksqwe/GenTreeBoundsChk_fix

Fix GenTreeBoundsChk::gtThrowKind check in GenTree::Compare(). It was comparing the same field of op1 instead of op1 and op2.

Merge pull request #2018 from AndyAyersMS/LessLogging

Reduce volume of MSBuild logging when running tests via xunit

Merge pull request #2133 from gsfreema/master

Fixing a grammar mistake in Documentation\README.md

Fixing a grammar mistake in Documentation\README.md

Simply using the correct conjugation of the verb 'support'.

Fix generated test script to properly check for debugger

Also tried fixing the generated help to show that you need `=`. This should be conditional based on `HasParam` but since there is only one switch it works for now.

Fix GenTreeBoundsChk:: gtThrowKind check in GenTree::Compare()

Merge pull request #2112 from ellismg/cache-ucollators

Cache UCollators in a Locale

Merge pull request #2127 from dotnet-bot/from-tfs

Merge changes from TFS

Merge pull request #2117 from hickford/patch-1

Add link from readme to CoreFX

Implemented QueryThreadCycleTime (time.cpp) & GetThreadTimes (thread.cpp) for Unix and OS X, plus tests. Also added helper methods IsCurrentThread and GetThreadTimesInternal to time.cpp. Currently, GetThreadTimes returns all time as user time on Unix, since I don't yet know of a way to get user time and kernel time separately there.

Fix build breaks in desktop build

[tfs-changeset: 1550852]

Merge pull request #2123 from stephentoub/cts_perf

Separate CancellationTokenSource linking into a derived class

Merge pull request #2122 from adityamandaleeka/testArgsQuoteFix

Replace double quotes around test arguments with single quotes.

Merge pull request #2096 from LLITCHEV/SysVTests

Fix for issue 2058.

Merge pull request #2120 from kyulee1/osx

Package spec for RyuJit on OSX

Merge pull request #2121 from dotnet-bot/from-tfs

Merge changes from TFS

Cache UCollators in CompareInfo

Creating a UCollator is an expensive operation and we are presently
doing it on ever collation operation. We can improve this by caching
the UCollators we use for collation on the CompareInfo object itself.

This change introduces a new method GetSortHandle which gives back an
opaque wrapper which can be used in collation operations instead of a
culture name.

Internally we represent this is a struct holding the two types of
UCollators we care about (if we add additional collators per locale with
different options to handle other types of CompareOption flags, we can
cache these as well).  Collation methods can get a `const UCollator*`
reference from the sort handle which is safe to share across
threads (per the ICU Design Guidelines[1]).

Unfortunately, tracking the lifetime of the SortHandle itself is not as
straightfoward as I would like. Right now, we use a SafeHandle to wrap
the internal handle and rely on the finalizer of the class to clean up
the native resources. However this means that the following code sample
will create two finalizable objects:

```csharp
    var c1 = new CultureInfo("en-US").CompareInfo;
    var c2 = new CultureInfo("en-US").CompareInfo;
```

If this ends up being an issue, we could explore an approach where we
keep a cahce of SortHandles in managed code and pass out references to
that SortHandle which would let us share a single SortHandle for a given
locale across more than one CompareInfo object.

Wins are seeing in places where we previously did lots of string
comparisions in a tight loop (for example: dotnet/corefx#3811) moving
these operations down to ~6ms per iteration vs ~330ms on my local machine.

[1]: http://userguide.icu-project.org/design

Merge pull request #2107 from pgavlin/SysVCleanup

Small cleanups in SysV classification.

Separate CancellationTokenSource linking into a derived class

The m_linkingRegistrations field is only used on CancellationTokenSource instances created by CreateLinkedTokenSource.  This commit creates a private LinkedCancellationTokenSource to contain that field and related logic, so that the field is pay for play.  This shaves off ~11% of normal CancellationTokenSource's memory consumption.

Replace double quotes around test arguments with single quotes.

Fix build breaks in legacy build

[tfs-changeset: 1550765]

Added missing URL to .NET Foundation forums

Grammar fix-up