Only import the RoslynPropsFile if it exists. (#22657)

Fix visibility and signed comparison issues for GCC (#22586)

* Fix visibility and signed comparison issues for GCC

* Fix hidden _CLRDataCreateInstance warned by macOS

* Fix indentation in vswprintf/test1

* Change void* to PVOID in implementation files

Avoid pulling in Lazy<T> during startup (#22659)

Environment constructor runs on every startup path. Remove dependency on Lazy<T> from it.

Merge pull request #22528 from mikedn/idx-addr

Fix genCodeForIndexAddr

[master] Update dependencies from dotnet/corefx (#22595)

* Update dependencies from https://github.com/dotnet/corefx build 20190213.23

This change updates the following dependencies
- Microsoft.NETCore.Platforms - 3.0.0-preview4.19113.23
- Microsoft.Private.CoreFx.NETCoreApp - 4.6.0-preview4.19113.23

* Update dependencies from https://github.com/dotnet/corefx build 20190215.1

This change updates the following dependencies
- Microsoft.NETCore.Platforms - 3.0.0-preview4.19115.1
- Microsoft.Private.CoreFx.NETCoreApp - 4.6.0-preview4.19115.1

* Fixing up the tests dependent on the S.R.I.Vector API changes

* Regenerating the tests dependent on the S.R.I.Vector API changes

* Disabling an outdated CoreFX test.

Set the 'PLATFORM_OSX' constant during OSX build (#22639)

* Set the 'PLATFORM_OSX' constant during OSX build

* Remove PLATFORM_OSX define from SPCL

Fix bug in FindDispatchSlot usage

Update BuildTools, IbcData, PgoData to 3.0.0-preview1-03715-01, master-20190208.1, master-20190208.1, respectively (master) (#22647)

Updating some local functions to be 'static' (captureless). (#22635)

JIT: modify how jit determines when to update a type (#22618)

For single-def locals, the type of a reference seen at the assignment to the
local may be a more specific type than the local's declared type. If so the jit
would prefer to use the assignment type to describe the local's value, as this
will lead to better optimization. For instance in
```
    object x = "a string";  // only assignment to x
```
the jit can optimize better if it models the type of `x` as `string`.

Instead of relying on `mergeClasses` plus some jit-side screening to decide if
the assignment type is a more specific type, implement a new jit interface
method `isMoreSpecificType` that tries to answer this question more directly.

Added a test case with type equivalence that hit asserts.

Closes #22583.

Disable arm64 contracts. (#22605)

* Fix ifdef for `g_DbgSuppressAllocationAsserts`.

It needs to be defined under `FEATURE_INTEROP_DEBUGGING`.

* Unify contracts disabling for UNIX/ARM.

It also disables some parts that were disabled for ARM, but not for UNIX.
It fixes the difference that we see between ARM32 and ARM64 Unix Checked.

* Disable contracts on arm64.

FIxes the difference that we see between windows arm32 and arm64 in checked/debug builds.

* Revert CoreFX arm64 timeout change.

That is no longer necessary.

* Reenable arm64 corefx jobs.

They should not fail with timeouts anymore.

* Don't allow `g_DbgSuppressAllocationAsserts` defnition in release builds.

* delete AssertAllocationAllowed

Port profiler howtos from David Broman's blog to the documentation folder (#22363)

Publish version to dotnet/versions

Fix UpdatePublishedVersions.ps1

Widen index to a temporary register

Delete references to System.Memory package that does not exist anymore (#22626)

Disable failing corefx test (#22625)

Related to https://github.com/dotnet/corefx/issues/35351

Manually update the dependencies.props and move S.P.Corelib to use LangVersion=8.0 (#22452)

* Update BuildTools to preview1-03713-01 (master)

* Updating CoreCLR to use LangVersion=8.0

* Moving the Windows scripts to default to `dotnet msbuild` for managed components

* Setting UseSharedCompilation=true

* Changing some additional callsites that were using msbuild to use dotnet msbuild

* Revert packages.builds to use Desktop msbuild on Windows

* Fixing runtest.cmd to always set DotNetCli

Reserve fast dictionary slots for Tier1 code (#22619)

Disable use of fast dictionary slots for R2R images when tiered JITing is enabled.

Fixes #22400

Update dependencies from https://github.com/dotnet/arcade build 20190214.4 (#22621)

This change updates the following dependencies
- Microsoft.DotNet.Arcade.Sdk - 1.0.0-beta.19114.4
- Microsoft.DotNet.Helix.Sdk - 2.0.0-beta.19114.4

Remove CoreLib's src\Interop\Windows directory (#22622)

It contains only one file with an interop signature that's not actually being used. Deleting the dup.

Fix CoreFX build breaks

Signed-off-by: dotnet-bot <dotnet-bot@microsoft.com>

Fixing the number formatting code to only recognize 'R' for double/float. (#22613)

Move Reflection.Emit Tokens to shared partition (#22602)

Missed optimization from #22497

Remove unnecessary ProjectReferences to CoreCLRTestLibrary (#22539)

Remove the ProjectReferences to CoreCLRTestLibrary.csproj in the project files where CoreCLRTestLibrary is not actually being used.

Each ProjectReference item is ended up in PrepareProjectReferences target that creates unnecessary items.

Replace multi-loaderallocator hash implementation in MethodDescBackpatchInfo (#22285)

* GCHeapHash
- Hashtable implementation for runtime use
- Implementation written in C++
- Data storage in managed heap memory
- Based on SHash design, but using managed memory

CrossLoaderAllocatorHash
- Hash for c++ Pointer to C++ pointer where the lifetimes are controlled by different loader allocators
 - Support for add/remove/visit all entries of 1 key/visit all entries/ remove all entries of 1 key
- Supports holding data which is unmanaged, but data items themselves can be of any size (key/value are templated types)

* Swap MethodDescBackpatchInfo to use the CrossLoaderAllocatorHash

* The MethodDescBackpatchCrst needs to be around an allocation
- Adjust the Crst so that it can safely be used around code which allocates
- Required moving its use out from within the EESuspend logic used in rejit

Remove unnecessary fields from WaitHandle (#22593)

* Remove unnecessary fields from WaitHandle

* Drop volatile from _waitHandle

* Address race condition in unmanaged code

* Remove unused hasThreadAffinity parameters

* Fix FCDECL argument counts

Fix check for memory containment safety. (#22563)

This change ensures that if an operand can produce an exception
and any instructions executed after the operand evaluation but before
the operand's parent can also produce an exception, the operand
shouldn't be contained. The reason is that in this case operand
containment may reorder exceptions.

With `strict` set to true the containment is blocked here:
https://github.com/dotnet/coreclr/blob/d27fff3f65193dd71c6197e9876101f496bbd28b/src/jit/sideeffects.cpp#L485-L488

Also, make the check for ordering side-effect interference less
conservative.

Fixes #22556.

Revert signing to fixed version (#22609)

SPC signing got broken when #22544 unpinned the SignTool version. Pin the tool version again until issue gets fixed.

@jashook

Consolidate implementation of Rotate and PopCount (#22584)

* Perf: BitOps.LeadingZeroCount

* Remove redundant MSIL cast, conv.u8

* Use local functions for SoftwareFallback

* Target BIT32/64

Adjust corert for shared SafeHandle and EventRegistrationToken

Signed-off-by: dotnet-bot <dotnet-bot@microsoft.com>

Fix netfx build break in Microsoft.Diagnostics.Tracing.EventSource.Redist

Signed-off-by: dotnet-bot <dotnet-bot@microsoft.com>

Move FrameworkEventSource to shared partition (#22507)

* Move FrameworkEventSource to shared partition

and remove unused code

* Remove more unused code

* Remove CA8001 suppress message

Gnuport cleanup 3 (#22581)

* Reorder extern "C" and visibility statements

In file included from coreclr/src/pal/inc/pal.h:70:0,
coreclr/src/pal/inc/pal_mstypes.h:30:25: error: expected unqualified-id
before string constant
 #define EXTERN_C extern "C"
                          ^
coreclr/src/pal/inc/rt/palrt.h:216:30: note:
  in expansion of macro 'EXTERN_C'
   #define STDAPI_(type)        EXTERN_C type STDAPICALLTYPE

* Need double quotes around single quotes in error statements

fix error statements

Can't have ' character in error statement

* Abstract __declspec(noinline)

Generate compiler independent macros for noinline request.

* Remove unused DBG_NOINLINE_X86__RET_INLINE

* Abstract __declspec(nothrow)

Add Rune creation API from UTF-16 surrogate pair (#22590)

Also brings in some perf improvements to existing char and UnicodeUtility APIs

Update dependencies from https://github.com/dotnet/arcade build 20190213.1 (#22594)

This change updates the following dependencies
- Microsoft.DotNet.Arcade.Sdk - 1.0.0-beta.19113.1
- Microsoft.DotNet.Helix.Sdk - 2.0.0-beta.19113.1

Add coreclr-outerloop build definition jobs (#22578)

Don't use $(_HelixSource) and $(_HelixType) in test-job.yml (#22589)

Annotate PlatformID as a moved type

This should have been annotated along with the rest in dotnet/corert#6928 but I missed it.

[tfs-changeset: 1733685]

Signed-off-by: dotnet-bot <dotnet-bot@microsoft.com>

Improving an assert in fgInsertInlineeBlocks

Use dependency flow for core-setup and corefx dependencies (#22544)

* Remove old use of signing package used to work around SPC signing now that arcade has bubbled up the version.
* Move dependency update for CoreFX and Core-setup dependencies to BAR/Darc/Maestro++ flow
* Update property references in tests.
* Version bump for CoreFX and Core-Setup dependencies.
* Update test Github_20958 to use revised System.Range constructor as surface area changed with #22331.

Merge pull request #22327 from BrianBohe/refactoring_genCodeForBBList

Moving structures initialization before generating code for blocks to…

Remove Marshaling MDA (#22579)

Remove the "marshaling" MDA. In .NET Framework it was off by default and in .NET Core it was commented out in the boilerplate file. Additionally, from what I can tell only half of its functionality was ever implemented in .NET Core.

Contributes to #22538.

Remove some CAS cruft (#22576)

* Update comment on SuppressUnmanagedCodeSecurityAttribute.
* Remove unused declsec.h header.

Perf: Consolidate implementation of LeadingZeroCount (#22497)

* Perf: BitOps.LeadingZeroCount

* CR fix

* CR fixes

* Optimization

* Revert

* Confirmed fix

* Simplify

* Return int

* Fixes

* Simplify

* CR fixes

* CR fixes

* Fix unit failure on Linux-musl x64 Debug

* CR fixes

* CR fixes

* Cleanup

Fix comment placement. (#22580)

Moving structures initialization before generating code for blocks to other function

Replace Win32Native.cs with Interop files (#22540)

* Replace Win32Native.cs with Interop files

- Replace Win32Native usage with existing Interop.* calls where they already existed
- Moved Win32Native.* to their own files otherwise, and changed call sites
- Left a stub Win32Native in place to handle some emitted IL from the runtime.  The current infrastructure doesn't support targeting nested types, like Interop.Kernel32, and fixing that would be more involved.

* Remove Win32Native.cs

Replace emitted calls with ones to Marshal.

* Fix incorrectly changed field

* Fix Kernel32 -> Ole32

Move SafeHandle to managed code and shared (#22564)

* Move SafeHandle to managed code and shared

This moves the implementation of SafeHandle from native code in the runtime to managed.  I used corert's implementation as a base, and reviewed it again against the existing native implementation, making a few tweaks to better match the existing semantics.

This should be a valid move because of the reduced goals around CERs, thread aborts, etc.

However, there are places in the runtime that access SafeHandle functionality via its native counterpart, so I kept the relevant pieces of the native code intact.  Most code will continue to use the managed APIs, but the runtime can continue calling into the native versions when needed.

* Address PR feedback

* Address PR feedback

Update dependencies from https://github.com/dotnet/arcade build 20190212.3 (#22568)

This change updates the following dependencies
- Microsoft.DotNet.Arcade.Sdk - 1.0.0-beta.19112.3
- Microsoft.DotNet.Helix.Sdk - 2.0.0-beta.19112.3

Move EventRegistrationToken to shared (#22565)

* Move EventRegistrationToken to shared

* Address PR feedback

Fixing the double/float formatting code to use a fallback precision for custom-format strings. (#22522)

Fix gen-buildsys-win for vs2019 x86 (#22567)

Improve SpanHelpers.Byte (#22503)

Fix min-opts spill of tree temp large vectors (#22530)

* Fix min-opts spill of tree temp large vectors

Even if we're not enregistering local vars, we may have large vectors live across a call that need to be spilled.

Fix #22200

Refactoring siVarLoc creation (#22543)

* Moving siVarLoc and siVarLocType from compiler.h to CodeGenInterface.h

* Encapsulating siVarLoc construction with siScope and LclVarDsc

* Encapsulating siVarLoc construction from psiScope

* Adding some argument description on genSetScopInfo header

* Changing const siVarLoc& to const siVarLoc* on eeSetLVInfo

* Changing siVarLoc& to siVarLoc* in genSetScopeInfo arguments

* Rename var in genSetScopeInfo header

Fix CoreRT build breaks - Adjust resource name - Workaround C# compiler/analyzer bug

Signed-off-by: dotnet-bot <dotnet-bot@microsoft.com>

Fix exception sets for indirections in value numbering. (#22542)

fgValueNumberAddExceptionSetForIndirection had a bug
where INDIR(ADD(x, <big_offset>)) had NullReferenceException(x)
added instead of NullReferenceException(ADD(x, <big_offset>)).

Fixes #20358.

Enable arm64 linux musl builds (#22495)

* Enable arm64 linux musl builds

Note that -clang5.0 is required to be passed.

* Fix syntax error

* Pass clang arg to build-test.sh

Bug Fix: Calling System.Diagnostics.Tracing.EventPipe.Enable twice asserts #22247 (#22318)

After disabling EventPipe -> SampleProfiler, we were not closing the thread shutdown event, thus asserting on reentrance of the EventPipe.
* Adding regression test, and removing comment.
* Waits until the specified object is in the signaled state.

Renaming RuntimeEventSource to NativeRuntimeEventSource (#22533)

* Renaming RuntimeEventSource to NativeRuntimeEventSource

* missed a couple of comments and XPlatEventLogger

* rename DotNETRuntimeEventSource.cs to NativeRuntimeEventSource.cs

* use condition= instead of ifdef

* removing ifdef from NativeRuntimeEventSource.cs

* Fix case for 'eventing' directory in csproj

* Fix unix builds

* This should fix unix build...

Enable portable fast alloc helpers on ARM64 Windows. (#22541)

Add extensions methods for several of the VectorXXX<T> helper methods (#22336)

* Moving several of the Vector128<T> instance methods to be extension methods.

* Moving several of the Vector256<T> instance methods to be extension methods.

* Moving several of the Vector64<T> instance methods to be extension methods.

Ifdef out code that triggers UTC bug (dotnet/corert#7003)

This results in bad codegen on AMD64. I filed a bug internally; hopefully we can get a fix soon.

Signed-off-by: dotnet-bot <dotnet-bot@microsoft.com>

Update COM Activator class (#22317)

* Update COM Activator to use ALCs

* Update tests to now supply an assembly name rather than an binary "path"
  since the COM activator is now using ALCs.

* Add test for non-rooted assembly path.

* Move the hostpolicy mock project to a Common area
Move hostpolicy mock API to CoreCLR test library

* Add test for COM server isolation

* Move ComActivator class and logic to Internal namespace.

JIT: change how we block gc refs from callee saves for inline pinvokes (#22477)

Add a new marker instruction that we emit once we've enabled preepmtive gc in
the inline pinvoke method prolog. Use that to kill off callee saves registers
with GC references, instead of waiting until the call.

This closes a window of vulnerability we see in GC stress where if a stress
interrupt happens between the point at which we enable preeemptive GC and
the point at which we make the call, we may report callee saves as GC live
when they're actually dead.

Closes #19211.

Add new rounding modes to System.Math, System.MathF (#20815)

* add new rounding modes to MidpointRounding.cs

new modes added to enum
implemented ToZero for double in Math.cs

* ToZero implementation

* implement double and float rounding modes

* updating rounding implementation

now round inline with DecCalc internal round implementation

* small bug fix

also replace var to make things obvious

* update implementation - floor/ceil

code review feedback

* review feedback

add comments, update MathF with floor/ceil

* code review feedback

 - fix comments
 - replace ifelse with switch
 - remove RoundingMode enum from DecCalc

* exclude outdated corefx test

Update dependencies from https://github.com/dotnet/arcade build 20190208.1 (#22506)

This change updates the following dependencies
- Microsoft.DotNet.Arcade.Sdk - 1.0.0-beta.19108.1
- Microsoft.DotNet.Helix.Sdk - 2.0.0-beta.19108.1

Support building with VS2019 Preview (#22525)

* Support building with VS2019 Preview

* Fixing gen-buildsys-win to only set the architecture for the VS generator

* Refactoring Dev11/147911/fpcw.cpp so that it compiles under VS2019

* Removing the remaining traces of VS2015 build support

Fix genCodeForIndexAddr

This does some weird things - treats the array length as 64 bit when it's in fact 32 bit, fails to zero extend TYP_INT indices, creates new GT_IND/GT_LEA nodes out of thin air.

Delete mda.cs and related dead code (#22535)

Move ConditionalWeakTable to shared (#22531)

Leaves DependentHandle as being the runtime PAL

 Make Module partial to allow additional changed for Mono

Annotate EventCounter as a relocated type (dotnet/corert#6990)

This moved to CoreLib and needs to be annotated as such for Project N.

Signed-off-by: dotnet-bot <dotnet-bot@microsoft.com>

Move Volatile to shared (#22523)

* Move Volatile to shared

This largely takes corert's implementation, which uses Unsafe.As and volatile fields.

We could investigate also removing the JIT's replacements subsequently.

* Add [Intrinsic] attributes

Locally define DLLEXPORT in ee_il_dll.cpp

This prevents a desktop build break due to DLLEXPORT being undefined.  If
you know of a better place to put this, I'm happy to try it.  Otherwise,
this should unblock the desktop build.

Move runincontext to src/tools

This SDK-style project is causing problems on some machines with VS
installs that can't build it.  For now, move it to a place where it
doesn't build with the tests.

Move GCHandle to shared (#22499)

* Move GCHandle to shared

* Cache the handle in a local to make it easier for JIT to generate good code

* Rename m_handle to _handle to match the coding conventions

* Delete isPinned argument for GCHandleInternalCompareExchange

* Add fast path to GCHandle.Alloc/Free FCalls

* Disable outdated test

Update master label to preview4

prestub: fix ExternalMethodFixupWorker (#22519)

Use DispatchToken::CreateDispatchToken to get token to resolve
virtual method in case of non interface MT.

Make Nullable partial to allow additional method for in under Mono (#22518)

Cleanup DLL exports (#22500)

Fix casing of interop directories (dotnet/corert#6982)

Match convention introduced by https://github.com/dotnet/corefx/pull/35085

Signed-off-by: dotnet-bot <dotnet-bot@microsoft.com>

Move eventpipe buffer to TLS (#21817)

* start ripping out eventpipe buffer to tls

* can now emit events from gc threads

* cleanup

* more cleanup

* more cleanup

* tested on linux

* Addressing PR comments

* Move things around a bit to build in Linux

* change eventpipe buffer deallocation code

* more cleanup

* this while loop doesnt do anything now

* Fix build

* fixing build

* More cleanup

* more pr comments

* Fix unix build

* more pr comments

* trying to add a message to assertion that seems to be causing CIs to fail

* more pr feedback

* handle non-2-byte aligned string payloads inside payload buffers

* some more cleanup

* Fix off by one error in null index calculation

* Make Get/SetThreadEventBufferList a static member of ThreadEventBufferList

* make only the methods public in ThreadEventBufferList

* Addressing noah's comments

* fix comment and last off by 1 error

set up CoreLib to include EventCounter in build project (#22501)

Add reporting exception from ResolveEHClause (#17947)

* Add reporting exception from ResolveEHClause

When an exception, like EEFileLoadException happens in the
ResolveEHClause, it was not caught by the runtime and so it caused exit
with `terminating with uncaught exception of type EEFileLoadException*`
message without any additional details.

This change adds catching the exception, reporting its details and call
stack and then failing fast.

* Change StackSString to SString

* Ensure the catch clause types are loaded before EH

In crossgen-ed images, ensure the types used in catch clauses are loaded
before the function containing these clauses is executed. That ensures
that a failure to load the EH clause type will occur at that time
instead of during the EH stack walking that searches for the catch
handler.

* Fix EH clause class module check

* Remove the EH clause class module check

It turns out that even if the class was from the current module, it may
depend on types from other modules, so we still need to add a fixup for
it.

move EventCounter to shared CoreLib (dotnet/corefx#35183)

Signed-off-by: dotnet-bot <dotnet-bot@microsoft.com>

Move Array's SZGenericArrayEnumerator to shared partition (#22480)

* Move Array's SZGenericArrayEnumerator to shared partition

* Remove legacy comment

* Move all array enumerators to a new file

Delete EnvironmentAugments (#22485)

Set visibility option to hidden (#21924)

BitOps.TrailingZeroCount has inconsistent software fallback (#22333)

Fix #22326

[Official Build] Remove FreeBSD builds (#22451)

Currently all the FreeBSD machines are offline or in an unclean state
which blocks official builds from finishing.

Revert "Revert "Re-point Jenkins to new Ubuntu ARM64 queue""

This reverts commit d617f96303c4b65b7b77ff5fa0d7aee38ebc19f8.

Move GCSettings to shared (#22483)

Mainly just type definitions and error handling that gets shared.

Pass official build id to build (#22468)

This ensures that coreclr gets a version number based on the build id.

Update CoreRT with shared array implementation

Signed-off-by: dotnet-bot <dotnet-bot@microsoft.com>

Update BuildTools, CoreFx, CoreSetup to preview1-03706-01, preview.19106.8, preview-27406-8, respectively (master) (#22455)

* Update BuildTools, CoreFx, CoreSetup to preview1-03706-01, preview.19106.8, preview-27406-8, respectively

* Disabling some outdated System.ObjectModel CoreFX tests

* Undo BuildTools update

JIT: Suppress emitting same-reg zero extending move (#22454)

Add a peephole optimization to suppress emitting zero extending moves
if the previous instruction has already done a suitable zero extension.

Only implemented for x64 currently.

Closes #21923

JIT: fix ifdef guarding an assert (#22460)

We use `_TARGET_AMD64_`, not `_TARGET_X64_`.