Merge pull request #261 from stephentoub/fix_semaphore

Implement CreateSemaphoreExA/W via CreateSemaphoreA/W

Implement CreateSemaphoreExA/W

Any managed code that touches the ThreadPool is currently crashing when the ThreadPool tries to initialize. This is due to it creating a semaphore via CreateSemaphoreExA/W in the pal, and those functions not being implemented.

This commit just implements those functions by delegating to their non-Ex counterparts. With this change, ThreadPool.QueueUserWorkItem, Task.Run, etc. are able to successfully schedule work and have it executed.

Merge pull request #262 from AlexGhiondea/produceDevPackage

Update the CoreCLR nuspec to deploy the files under the aspnetcore5 folder

Update the CoreCLR nuspec to deploy the files under the aspnetcore5 folder.

Merge pull request #245 from poizan42/fix-rm-intermediates

Fix wrong flag to rm when deleting intermediates folder.

Merge pull request #251 from shahid-pk/issue-243

Correctly locate Clang [Fixes 250]

Merge pull request #260 from henrybw/henrybw-define-complus

Add definition of "COMPLUS" to the glossary

Add definition of "COMPLUS" to the glossary

In the same vein as "COR" and "URT", the terms "COM+"/"COMPLUS" appear frequently throughout the codebase, most notably in config settings (e.g. "COMPLUS_DefaultVersion"). I couldn't find any other explanation for what this means, and since I remember learning about it myself by asking older members of the team, I thought it would be helpful to have this documented for newcomers to the codebase.

Merge pull request #259 from janvorli/linux-issue177

Modify the windows amd64 unwinder to work as jitted code unwinder on Uni...

Modify the windows amd64 unwinder to work as jitted code unwinder on Unix

Update the windows amd64 unwinder from its source to the latest version
and add context pointers updating that was excluded before.
Change methods of the OOPStackUnwinder and OOPStackUnwinderAMD64 to be
static, since the base class has no data members and the unwinding is
always using the processor specific subclass.
Modify the code so that it is built in two flavors. The first one
is the DAC flavor which is used by the debugger. The second one will be
used for jitted code stack frames unwinding on non-Windows systems.

Merge pull request #258 from sergiy-k/fix-sscanf-parsing

Fix sscanf implementation in PAL

Fix sscanf implementation in PAL

Implementation of sscanf was incorrectly handling cases where the input format was in the form of \93%2c\94, \93%2s\94, or \93%[\93 which could result in an access violation. The problem was that sscanf was calling sscanf_s with incorrect set of arguments. Implementation of sscanf_s expects to find the size of the target buffer in the list of its parameters but that parameter was not present in some cases. This change fixes this issue and re-enables previously disabled sscanf tests.

Merge pull request #254 from ellismg/remove-locale-cpp

Remove locale.cpp from the PAL on OSX

Merge pull request #246 from akoeplinger/fix-botr

Improvements to docs

Correctly locate Clang [Fixes 243]

Addressing comments

Remove locale.cpp from the PAL on OSX

The globalization plans no longer require these methods to be
implemented in the PAL, since we no longer call into the VM for
globalization support except on Windows.

It's possible that more stuff can be cleaned up here, and not just for
OSX, but I need to take a careful look at the use of codepages across the
runtime to understand if it's still needed.

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

Merge changes from TFS

Workaround for an ARM specific MS C++ compiler bug

Due to a bug #1123874 in arm codegen, critical code path in mscordbi.dll became unusable making any managed debugging impossible on ARM.

[tfs-changeset: 1414914]

Merge pull request #249 from libengu/msilchookup_1

Expand the availability of MSILC JIT to all tests

Merge pull request #244 from benpye/file-size-ex

Add GetFileSizeEx implementation to pal

Merge pull request #248 from mmitche/xunit-xml-output

Xunit xml output

Expand the availability of MSILC JIT to all tests

Spit out Xunit xml result files

Spit out the xunit result files, as well as make the directory for the xunit html result files consistent with the result of the logs.  We should also be passing the include and exclude attributes to the xunit task

Cleanup GetFileSize test to reduce duplicated code

Pal tests should pass on windows

Error if the filesize pointer is null, improve GetFileSizeEx test.

[docs] Replace smart quotes and ellipses with their ASCII equivalent

Merge pull request #242 from libengu/msilchookup

Hook up initial JIT tests for MSILC JIT

Fix wrong flag to rm when deleting intermediates folder. Fixes #243

Merge pull request #241 from pgavlin/AltJitFixes

Small fixes to enable AltJit support on UNIX.

Add GetFileSizeEx implementation to pal

Merge pull request #228 from shahid-pk/issue-225

Disable the 'trim_trailing_whitespace' fixes issue 225

Merge pull request #188 from stephentoub/disable_etw_unix

Update FEATURE_MANAGED_ETW and disable it on Unix

Hook up initial JIT tests for MSILC JIT

Small fixes to enable AltJit support on UNIX.

- Load libcoreclr.so with the RTLD_GLOBAL flag set in unixxorerun s.t.
  its exports are available for subsequently loaded libraries.
- Demangle PAL_TryExcept
- Replace a non-portable path separator character with the corresponding
  portable definition from the PAL.

Disable the 'trim_trailing_whitespace' fixes issue 225

Merge pull request #136 from poizan42/cmake-locate

Make build.cmd lookup the install directory of cmake in the registry if ...

Merge pull request #220 from mmitche/issue-208

Fix rebuild issue with development package

Merge pull request #238 from poizan42/fix-spaces-linux-build

Fix build on paths with spaces and with llvm path with spaces.

Make build.cmd lookup the install directory of cmake in the registry if it's not on the path.

Merge pull request #239 from kangaroo/master

[issue #232] Rename dispatchexceptionwrapper on the filesystem as well

Merge pull request #183 from poizan42/build-space-fix

Correct handling of paths with spaces and ampersands when building on windows

Add missing license header

[issue #232] Rename dispatchexceptionwrapper on the filesystem as well

Fix build on paths with spaces and with llvm path with spaces. Fixes #216

Merge pull request #237 from richlander/master

Add Rudi's GH handle to stackwalking doc

Add Rudi's GH handle

Merge pull request #236 from richlander/master

Update document links

Merge branch 'master' of https://github.com/dotnet/coreclr

Fix documentation links

Merge pull request #235 from richlander/master

Update documents

Make gittr link more visible

Update readme links to index page

Add documents index

Attribute stack walking doc to Rudi

Merge pull request #223 from kangaroo/warning-clean

Clean up compiler warnings on the OSX build

Merge pull request #233 from lioncash/leak

ceesectionstring: Fix a potential memory leak

Merge pull request #231 from poizan42/fix-abs-64bit

Fix copy paste errors in llabs test

Merge pull request #234 from StephenCleary/patch-1

Replaced TODO with doc links.

Replaced TODO with doc links.

Added links to the CLR Code Guide.

ceesectionstring: Fix a potential memory leak

[docs] Remove trailing spaces

See markdownlint rule MD009: https://github.com/mivok/markdownlint/blob/master/docs/RULES.md#md009---trailing-spaces

[docs] Add space after hash on atx style header

See markdownlint rule MD018: https://github.com/mivok/markdownlint/blob/master/docs/RULES.md#md018---no-space-after-hash-on-atx-style-header

[docs] Headers should be surrounded by blank lines

See markdownlint rule MD-022: https://github.com/mivok/markdownlint/blob/master/docs/RULES.md#md022---headers-should-be-surrounded-by-blank-lines

[docs] Get rid of multiple consecutive blank lines

See markdownlint rule MD-012: https://github.com/mivok/markdownlint/blob/master/docs/RULES.md#md012---multiple-consecutive-blank-lines

[docs] Make formatting of code blocks/lists consistent

Code block is denoted by one tab, indentation in code blocks is four spaces.
List starts at the beginning of a line, nested list items are indented by two spaces.

[docs] Fix typos in recently added BotR chapters

Merge pull request #230 from AndreyAkinshin/master

Typo fix in objecthandle.cpp

fix copy paste errors in llabs test

Typo fix in objecthandle.cpp

Merge pull request #226 from richlander/master

Add new BOTR docs to readme

Add new BOTR docs to readme

Respond to PR feedback on DAC filenames

Add CLR coding guide

Add stack walking document

Update glossary based on issue #209

Clean up compiler warnings on the OSX build

Invert ETW condition in clr.coreclr.props

Change a condition to be == Windows_NT rather than != Unix.

Merge pull request #211 from kangaroo/issue-199

Implement JIT_MemSet and JIT_MemCpy for clang

Merge pull request #207 from poizan42/fix-abs-64bit

Add abs(__int64) overload for 64-bit targets. Fixes #143

Merge pull request #221 from ellismg/palsuite-line-endings

Force PAL test lists to have Unix line endings

Ensure that we call memcpy from the PLT on linux

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

Merge changes from TFS

Force PAL test lists to have Unix line endings

A common workflow for some folks on our team is to check out the Git
repository, then share that folder out over to Linux.  In this case,
we need the pal test files to have Unix line endings, even in the case
where core.autocrlf = true, which is the case on Windows.

My hunch is that we would also need this even if we didn't do this,
since otherwise changes to this file will be mirrored back to TFS with
DOS line endings, which would break building the TFS version when
checked out, so it's likely a good idea to do this anyway, even if the
leading factor is a somewhat odd workflow.

Address PR feedback on ETW disablement on Unix

Fix rebuild issue with development package

If the development package is deleted and then a rebuild is attempted we fail because mscorlib is not where we think it should be.  We should be looking for mscorlib.pdb in the PDBs dir and also building the packages after the PDBs are moved.

Merge pull request #215 from mmitche/sos-in-dev-package

Add sos to the development package

Fix ARM Ready To Run: A dynamic helper generator was not allocating enough memory.

[tfs-changeset: 1414126]

Merge pull request #217 from sergiy-k/disable-pal-sscanf-tests

Disable a few PAL sscanf test cases

Disable a few PAL sscanf test cases

These tests need to be temporary disabled untill the issue https://github.com/dotnet/coreclr/issues/161 is fixed. This needs to be done in order to enable running PAL tests in the lab. Currently these test cases fail on Mac and Linux release builds.

Add abs(__int64) overload for 64-bit targets. Fixes #143

Add sos to the development package

Merge pull request #189 from akoeplinger/fix-filename-casing

Correct casing of file paths in mscorlib

Merge pull request #212 from poizan42/fix-hardcoded-tools-resgen-path

Use $(SDK40ToolsPath) to get path to ResGen instead of hardcoding "NETFX 4.5.1 Tools".

Remove the CPP stubs and ensure to mark the leaf end

Use $(SDK40ToolsPath) to get path to ResGen instead of hardcoding "NETFX 4.5.1 Tools".
Fixes #210

Implement JIT_MemSet and JIT_MemCpy for clang

The windows build has a hand written version of these functions, but
modern systems should have an equivalent performant version of this.
We do need to wrap them to get the trap for a null reference exception,
but after that we can tail call directly.

Correct casing of file paths in mscorlib

Some of the files had a different path/filename casing in the project reference than on disk.

Correct handling of paths with spaces and ampersands when building on windows.
Fixes #178

Merge pull request #179 from maksqwe/hresult_return_fix

AcquireNoThrow(): return type must be HRESULT