Merge pull request #2446 from jashook/legacy_backend_excludes

Add x86 legacy backend exclusions.

Merge pull request #2506 from mikedn/staticwalk

Make JIT's fgWalk*Rec functions static

Merge pull request #2474 from gkhanna79/FixArm64

Arm64 build fixes

Merge pull request #2513 from taylorjonl/call-to-sqrt-is-ambiguous

Fix "call to 'sqrt' is ambiguous" error when building on SunOS

Fix "call to 'sqrt' is ambiguous" error when building on SunOS

Building on SunOS will fail with the message of "call to 'sqrt' is
ambiguous" because it can't determine the type of the constant value
passed to it. This change adds a cast to a double because every usage
uses this data type

Merge pull request #2461 from LLITCHEV/GetTypeIntrinsic

Fix a bug in morphing GT_COMMA with op1 being the new GetType() intri…

Merge pull request #2505 from mikedn/deadcode

Remove dead code from JIT's earlyprop.cpp

Merge pull request #2504 from mikedn/4044again

Fix LNK4044 again

Merge pull request #2503 from mikedn/4221again

Disable LNK4221 again

Merge pull request #2498 from taylorjonl/smartos

Fixed issue where files in /opt/local/include are not found and LTTng is required on SmartOS

Merge pull request #2496 from kyulee1/round2

Enable ilasm round trip test for xPlatforms

Fix a bug in morphing GT_COMMA with op1 being the new GetType() intrinsic.

The GetType intrinsic is side effect free and the morphing of GT_COMMA is
optimizing out side effect free op1a. The native implementation of GetType
- NativeObject::GetClass - can throw NullReferenceException and such
optimizing the call out is illegal in such case. Also added GTF_EXCEPT
to the intrinsic node to note the other optimizers that the call can
throw an exception.

The coreclr and corefx tests passed. Also minimal, correct asmdifs. The
daily tests passed as well.

Make  JIT's fgWalk*Rec functions static

The compiler object is already available via  fgWalkData and is only needed in debug builds to call gtDispTree. Given that these are recursive functions and have many callsites it seems useful to minimize the number of function parameters.

Remove dead code from JIT's earlyprop.cpp

Fix LNK4044 again

This was fixed in #2310 and the fix was accidentally reverted in #2244 and #2425

Disable LNK4221 again

Linker warning 4221 was disabled in #2320 and was accidentally reenabled in #2244

Fixed issue where CMAKE couldn't identify SunOS processor type

Enable ilasm round trip test for xPlatforms

1. Enable CLRTest.Execute.Bash.targets similar to CLRTest.Execute.Batch.targets.
2. _IlasmSwitches uses '-' instead of '/'
3. Disable optimization on the prebuilt asmparse.c due to https://github.com/dotnet/coreclr/issues/2305

Fixed issue where files in /opt/local/include on SunOS
Modified to not require LTTng on SunOS

Merge pull request #2497 from wtgodbe/nightly2

Priority 1 tests for OSX & Linux x64 Release builds should now run daily (fixed)

Merge pull request #2494 from ramarag/incremenatal_build

Enable Incremental build

message

Merge pull request #2495 from AndyAyersMS/LockJson

Actually lock the project.lock.json

 Enable Incremental build For Generated Files

Disable serialization benchmarks for non-windows

Temporarily disable these until we can sort out what goes wrong on some of the test machines.

Merge pull request #2487 from kyulee1/round

Enable ildasm/ilasm round trip testing

Actually lock the project.lock.json

Hopefully this clears up some test issues on ubuntu with the serialization benchmarks.

Enable ilasm round trip testing

This enables ilasm round-trip testing for Windows.
In order to test il{d}asm on xPlatforms, I injected ilasm round trip before executing the test.
This means this step is a part of the runtest not the buildtest -- unlike runtest, currently buildtest is only on Windows.
1. Test batch file is extended to run ildasm followed by ilasm.
2. BuildTest has now an optional flag "ilasmroundtrip" to enable the round-trip tests. Note under this flag, we run both the original binary and the round-trip binary respectively.
3. Test timeout is increased since the round-trip is part of runtime.
4. Did some clean-up on CLRTest.Execute.Batch.targets
5. Bail out the round-trip tests that are built against mscorlib instead of .Net Core.
6. Added /raweh to ildasm due to https://github.com/dotnet/coreclr/issues/2481
7. ilasmroundtrip.targets (~10 tests) are added to enable them by default in a regular test run.

Tested with enabling the entire tests (>5K tests) with 'ilasmroundtrip' on, which should be on in a daily or weekly stress mode.

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

Merge changes from TFS

Add .gitmirrorall directive

[tfs-changeset: 1560604]

Merge pull request #2490 from wtgodbe/nightly2

Priority 1 tests should run nightly for OSX and Linux

Priority 1 tests should run nightly

Incorporate PR feedback

Merge pull request #2464 from ramarag/event_logging_automation

Automating Event Logging Infrastructure

Merge pull request #2456 from AndyAyersMS/SerializeBenchmark

Simple serialization benchmarks

Merge pull request #2476 from kyulee1/ilasmpermission

Using ilasm for test population

Simple deserialization benchmarks

As with the simple serialization benchmarks, these tests are intended to stress aspects jit code quality and are not meant to compare various deserialization approaches.

Simple serialization benchmarks

Benchmarking a few serialization methods. Not really trying to compare them, just setting things up to see how well the JIT does at each.

Merge pull request #2477 from kyulee1/wrapper

Common Globals for TestWrapper

Merge pull request #2475 from kyulee1/fixilasm

Fix assertion in ilasm

Common Globals for TestWrapper

Individual test in test wrapper initializes and gets the same globals from environment variables.
This increases not only wrapper size but also runtime overhead when running multiple tests.
This refactors this common part out as _Globals so that each test just accesses it.

Automating Event Logging Infrastructure: With this change the infrastructure required for Event Logging
will be generated as part of build

Using ilasm for test population

This enables the use of ilasm from CoreCLR build to populate il tests instead of using a stock ilasm on Windows desktop.
Note currently we populate tests (even for xPlatforms) on Windows only.
But when we migrate this test population on xPlatforms, this can be simply used as well.
All permission related attributes are dropped which CoreCLR ignores anyhow. Or ilasm will fail to parse such attributes.

Fix assertion in ilasm

This is a simple fix for incorrect assertion.
Assertion location is changed and dwErrorIndex is zero-initialized.
ilasm on coreclr does not handle permission set/security attribute which coreclr does not support.
So, such attempt will end up with emitting error on coreclr instead of failing to assert.

Arm64 build fixes

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

Merge changes from TFS

Fix build break in internal arm64 build

[tfs-changeset: 1559946]

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

Merge changes from TFS

Fix build breaks in internal x86 and arm builds

[tfs-changeset: 1559929]

Follow up fixes and cleanup for GC refactoring

[tfs-changeset: 1559927]

Merge pull request #2465 from jkotas/rename

Unify casing of tests\src\JIT\jit64\eh\FinallyExec

Unify casing of tests\src\JIT\jit64\eh\FinallyExec

Merge pull request #2244 from janvorli/gc-interface-refactoring-new

GC OS interface refactoring

GC OS interface refactoring

This change replaces all calls of OS specific functions in the GC by a call to
a platform agnostic interface. Critical sections were abstracted too.
The logging file access was changed to use CRT functions instead of Windows specific APIs.
A "size" member was added to the card_table_info so that we can pass the right
size to the VirtualRelease method when destroying the card table.
I have also fixed a bug in the gc_heap::make_card_table error path where when VirtualCommit
failed, it called VirtualRelease with size that was not the reserved size, but
the committed size.
Other related changes
- All interlocked operations moved to Interlocked class as static methods
- Removed unused function prototypes
- Shuffled stuff in the root CMakeLists.txt to enable building the GC sample using the
  settings inherited from the root CMakeLists.txt and to clean up some things that have
  rotted over time, like the FEATURE_xxx macros not being in one alphabetically ordered
  block
- Fixed the VOLATILE_MEMORY_BARRIER macro in the gcenv.base.h
- Replaced uint32_t thread id by EEThreadId
- Removed thread handles storage (g_gc_thread) from the GC. The thread handle is closed right after the thread is launched. That allowed me to get rid of the GCThreadHandle
- Renamed the methods of the EEThreadId to be easier to understand
- Moved the gcenv.windows.cpp and gcenv.unix.cpp to the sample folder

Merge pull request #1452 from akoeplinger/dictionary-duplicatekey

Dictionary: include the key in the "duplicate key" exception message

Merge pull request #2463 from kyulee1/fixinf

Fix for special number handling in ildasm

Fix for special number handling in ildasm

During the test bring-up, I noticed that _gcvt_s (CRT call) has slightly different output for special number like NAN/INF in between VS2013 and VS2015.
For instance, the former prints "1.#INF" while the latter prints "inf".
Unfortunately ildasm checks "#" to detect such case.
The fix here is to add an explicit check if string contains "inf"/"nan".

Merge pull request #2313 from richardlford/portilasm1211

Port more CoreClr and Desktop JIT Selfhost ILASM tests.

Merge pull request #2462 from mmitche/add-arm64-users

Add explicit arm64 permissions

Add explicit arm64 permissions

Merge pull request #2459 from jkotas/bogus-test

Delete bogus test

Delete bogus test

Merge pull request #2431 from mikedn/notestwarn

Cleanup warnings in native test projects

Port additional desktop tests to CoreClr

Add exclusions for tests that are failing but that
we expect to be able to get working.

Merge pull request #2455 from AndyAyersMS/FixSqMtx

Reduce iterations in SqMtx

Merge pull request #2438 from mmitche/update-readme

Update readme with arm64 badge

Reduce iterations in SqMtx

This test had two nested loops using `Iterations` and so ran for quite a long time. Remove the outer nest and then bump up the `Iterations` value so the test runs for about 1 second from command line and 10 seconds under xunit-performance.

Merge pull request #2440 from mmitche/add-private-pr-trigger

Add private PR trigger for arm64 windows builds

Merge pull request #2453 from mmitche/fix-triggers

Re-add missing FreeBSD PR trigger and reenable PR tests on NT

Re-add missing FreeBSD PR trigger and reenable PR tests on NT

Merge pull request #2425 from erozenfeld/CheckedBuild

Enable checked builds of CoreCLR.

Merge pull request #2424 from erozenfeld/TailRecursionToLoopFix

Fix for a bug in recursive tail call to loop transformation.

Add x86 legacy backend exclusions.

Pass this file to runtest.cmd using exclude0 or exclude.

Enable checked builds of CoreCLR.

In checked builds coreclr, mscorlib, and the test are built optimized
but assertion checking is on. This adds additional coverage (the jit is
optimizing and assertion checking is on), speeds up testing compared to debug,
and allows testing JIT stress modes.

This doesn't affect CoreFX.

Several tests are currently failing in checked configuration due to newly
discovered  bugs (JIT asserts). We didn't see these asserts in debug mode
because by default JIT is in minopt mode; we didn't see these bugs in release
mode because assertion checking is off. I will file the bugs once checked build
changes are in.

Fix for a bug in recursive tail call to loop transformation.

The bug affects recursive tail calls where there is both a this arg and
an argument passed in outgoing arg space.

I brought over RecursiveTailCall tests and added TestStackParam test
that was a repro for the bug.

fixup

Add private PR trigger for arm64 windows builds

Trigger is @dotnet-bot test arm64 windows_nt

Merge pull request #2439 from mmitche/debug-to-release

Run release not debug

Run release not debug

Update readme with arm64 badge

Merge pull request #2436 from richlander/docs

Update doc links

Merge pull request #2437 from mmitche/disable-coreclr-debug

Disable debug coreclr PR testing on linux/osx to keep job times down

Disable debug coreclr PR testing on linux/osx to keep job times down

Merge pull request #2428 from taylorjonl/smartos

Initial support for building on SmartOS

Update doc links

Cleanup warnings in native test projects

Most of the warnings (>10000) are generated by /Wall. It doesn't make much sense to use /Wall but since there's only a small number of warning types that cause issues I left /Wall in and disabled those particular warning types.

A few other warnings were caused by actual code issues (signed/unsigned mismatch, pointer cast to int, unused variables) which I fixed.

Merge pull request #2395 from ellismg/wait-for-single-object-recursion-fix

Fix infinite recursion in hostimpl.cpp

Initial support for building on SmartOS

Fix infinite recursion in hostimpl.cpp

WaitForSingleObject was calling itself instead of delegating to
WaitForSingleObjectEx.

Merge pull request #2394 from jashook/path_change

Remove the quotes on include for arm64 build.

Remove the quotes on include for arm64 build.

Merge pull request #2393 from mmitche/fix-native-test-comp

Archive native test bits too for non-windows

Archive native test bits too for non-windows

Merge pull request #2392 from mmitche/use-release

Switch to using release corefx

Switch to using release corefx

Merge pull request #2378 from ellismg/revert-29ce4f6

Revert 29ce4f6

Merge pull request #2376 from jashook/path_setup

Set up path for toolset.

Merge pull request #2391 from mmitche/fix-netci

globalParams no longer defined

globalParams no longer defined

Merge pull request #2377 from mmitche/arm64-win

Arm64 CI support and general refactoring of the CI scripting