platform/upstream/dotnet/runtime.git
7 years agoGeneric Dictionary - add constructor that accepts IEnumerable
Dan Moseley [Sat, 19 Nov 2016 18:24:21 +0000 (10:24 -0800)]
Generic Dictionary - add constructor that accepts IEnumerable

Generic Dictionary - add constructor that accepts IEnumerable

Commit migrated from https://github.com/dotnet/coreclr/commit/198bdf53c0590bda08b9b8e1ede1270fc9a8afc1

7 years agoAdd System.Collections.Generic.Dictionary constructor that accepts
Robert Matusewicz [Thu, 17 Nov 2016 09:37:43 +0000 (09:37 +0000)]
Add System.Collections.Generic.Dictionary constructor that accepts
IEnumerable<KeyValyePair<TKey, TValue>> to initialize dictionary

Commit migrated from https://github.com/dotnet/coreclr/commit/e08c2d4bdaf5e211a31303f271a4f11b2223ad7a

7 years agoMerge pull request dotnet/coreclr#8180 from jashook/arm64_lst_update
Jarret Shook [Fri, 18 Nov 2016 21:28:31 +0000 (13:28 -0800)]
Merge pull request dotnet/coreclr#8180 from jashook/arm64_lst_update

Update the lst file to exclude failing tests.

Commit migrated from https://github.com/dotnet/coreclr/commit/f4b527dfb2ed02e268e4f1d5318fa79053ede064

7 years agoMerge pull request dotnet/coreclr#8097 from pgavlin/VSO279396
Pat Gavlin [Fri, 18 Nov 2016 21:19:29 +0000 (13:19 -0800)]
Merge pull request dotnet/coreclr#8097 from pgavlin/VSO279396

Do not fold (ind (addr (lclVar))) in certain cases.

Commit migrated from https://github.com/dotnet/coreclr/commit/2c53985424e578b2f5b011d354f6ccb971e81c73

7 years agoDelete dead files (dotnet/coreclr#8202)
Jan Kotas [Fri, 18 Nov 2016 21:03:01 +0000 (13:03 -0800)]
Delete dead files (dotnet/coreclr#8202)

Commit migrated from https://github.com/dotnet/coreclr/commit/13ee34fc2c5cdff0bf8effb04fe870ac82c46ff1

7 years agoAdd CI job and build.cmd command for building with FEATURE_STANDALONE_GC (dotnet...
Sean Gillespie [Fri, 18 Nov 2016 20:57:38 +0000 (12:57 -0800)]
Add CI job and build.cmd command for building with FEATURE_STANDALONE_GC (dotnet/coreclr#8183)

* Add CI job and build.cmd command for building with FEATURE_STANDALONE_GC

* Add missed case

* Correct a condition

* Accidental fallthrough in switch

* Only run on x64

Commit migrated from https://github.com/dotnet/coreclr/commit/ecc468cf683513a2c3e1635f7abcdb06101eec38

7 years agoUpdate the lst file to exclude failing tests.
jashook [Thu, 17 Nov 2016 18:18:24 +0000 (10:18 -0800)]
Update the lst file to exclude failing tests.

These are new regressions that were introduced during CI downtime.

Commit migrated from https://github.com/dotnet/coreclr/commit/aaead2060f01165a29c92b1f5d30e7ead2d05cd6

7 years agoOrganizing the perflab tests under a common namespace. (dotnet/coreclr#8184)
José Rivero [Fri, 18 Nov 2016 19:24:07 +0000 (11:24 -0800)]
Organizing the perflab tests under a common namespace. (dotnet/coreclr#8184)

- These changes adds a hierarchy to the perflab tests under BenchView.

Commit migrated from https://github.com/dotnet/coreclr/commit/c48d0320f6bccc9da93b5aefb576f0ac2692c446

7 years agoMerge pull request dotnet/coreclr#8198 from pgavlin/GH8170
Pat Gavlin [Fri, 18 Nov 2016 18:46:46 +0000 (10:46 -0800)]
Merge pull request dotnet/coreclr#8198 from pgavlin/GH8170

Fix codegen for `(umod (gt_long) (const int))`.

Commit migrated from https://github.com/dotnet/coreclr/commit/4807a27422964b3ae7fca2deca2afb3d1b383594

7 years agoFix codegen for `(umod (gt_long) (const int))`.
Pat Gavlin [Fri, 18 Nov 2016 06:37:17 +0000 (22:37 -0800)]
Fix codegen for `(umod (gt_long) (const int))`.

When targeting x86, we opt not to use a helper call to implement the
64-bit unsigned modulus operator if we have a tree of either of the
following forms:
- `(umod ulong (x ulong) (cast ulong (const int const_val)))`
- `(umod ulong (x ulong) (const ulong const_val))`
where `const_val` falls in the range `[2, 0x3fffffff]`.

We eventually decompose this into `(umod uint (gt_long lo hi) (const int const_val))`.
This was then emitted as:

      mov eax, lo
      mov edx, hi
      mov reg, const_val

      div edx:eax reg

Unfortunately, this generated code does not take into account the fact
that the result of the modulus may not fit into 32 bits. This can cause
an overflow exception at runtime despite the fact that the modulus
originally produced a 64-bit result.

This change fixes the code generation to check for overflow before
performing the modulus and perform a 64-bit division if the result would
overflow. This results in generated code like the following:

      mov eax, lo
      mov edx, hi
      mov reg, const_val

      cmp edx, reg
      jb noOverflow
      mov temp, eax
      mov edx, eax
      xor edx, edx
      div edx:eax, const_val
      mov eax, temp

    noOverflow:
      div edx:eax, const_val

Which is identical to the code produced by JIT32 and the legacy backend.

Commit migrated from https://github.com/dotnet/coreclr/commit/7e00ee6be9590670d060117fd3932808bd7a86e6

7 years agoMerge pull request dotnet/coreclr#8194 from AndyAyersMS/FixMismatchTests
Pat Gavlin [Fri, 18 Nov 2016 16:02:12 +0000 (08:02 -0800)]
Merge pull request dotnet/coreclr#8194 from AndyAyersMS/FixMismatchTests

Revise mismatch tests so they don't rely on undefined behavior

Commit migrated from https://github.com/dotnet/coreclr/commit/4f8ba1e05e53e40a52fd66d7660efa8a992da9a9

7 years agoAdd missing Arg_BadDecimal resource string (dotnet/coreclr#8197)
mikedn [Fri, 18 Nov 2016 14:19:12 +0000 (18:19 +0400)]
Add missing Arg_BadDecimal resource string (dotnet/coreclr#8197)

Commit migrated from https://github.com/dotnet/coreclr/commit/85b73ed0ea01d487b9458548082ae613c95f7ddd

7 years agoMerge pull request dotnet/coreclr#8178 from adiaaida/excludeHeapVerify
Russ Keldorph [Fri, 18 Nov 2016 11:11:09 +0000 (03:11 -0800)]
Merge pull request dotnet/coreclr#8178 from adiaaida/excludeHeapVerify

Exclude tests that timeout with HeapVerify

Commit migrated from https://github.com/dotnet/coreclr/commit/c173ac0bbb0d7c3518541334d72ecd42619915a9

7 years agoMerge pull request dotnet/coreclr#8187 from pgavlin/gh8150
Pat Gavlin [Fri, 18 Nov 2016 04:37:50 +0000 (20:37 -0800)]
Merge pull request dotnet/coreclr#8187 from pgavlin/gh8150

Fix the transformation of `(call ARRADDR_ST x, y, null)`.

Commit migrated from https://github.com/dotnet/coreclr/commit/f261bb509b615a80215f59d26277ed50b877c8fc

7 years agoType sigs (dotnet/coreclr#8191)
Dan Moseley [Fri, 18 Nov 2016 03:39:43 +0000 (19:39 -0800)]
Type sigs (dotnet/coreclr#8191)

Commit migrated from https://github.com/dotnet/coreclr/commit/51ccb89389d0fd314e9090ec1d9108670041494a

7 years agoMerge pull request dotnet/coreclr#8182 from mikedn/x86-cmp-long-linqmax
Sivarv [Fri, 18 Nov 2016 03:24:12 +0000 (19:24 -0800)]
Merge pull request dotnet/coreclr#8182 from mikedn/x86-cmp-long-linqmax

Fix x86 long compares that produce a result

Commit migrated from https://github.com/dotnet/coreclr/commit/eb4920aeb75f40ee5029c5ad1c30545a02006ee3

7 years agoFormat code.
Pat Gavlin [Fri, 18 Nov 2016 02:58:06 +0000 (18:58 -0800)]
Format code.

Commit migrated from https://github.com/dotnet/coreclr/commit/2ae5c001a1c3bd99e7c81796da1e63f76d4163cd

7 years agoRevise mismatch tests so they don't rely on undefined behavior
Andy Ayers [Fri, 18 Nov 2016 00:06:46 +0000 (16:06 -0800)]
Revise mismatch tests so they don't rely on undefined behavior

The mismatch32 and mismatch64 tests ensure that the jit will avoid
inlining in cases where there are bad type mismatches at call sites.

This change modifies the tests so that by default the call with the
bad type mismatch is not hit at runtime, since execution of this code
exposes undefined behavior that can be arch or jit specific (eg failing
on x86 and passing on other architectures).

The problematic call can still be hit at runtime by passing at least one
command line argument.

Remove the associated test exclusions for x86.

Fixes dotnet/coreclr#8152.

Commit migrated from https://github.com/dotnet/coreclr/commit/021f775b3a7aae7557304dae80496519bb50fb55

7 years agoFix the transformation of `(call ARRADDR_ST x, y, null)`.
Pat Gavlin [Thu, 17 Nov 2016 23:49:49 +0000 (15:49 -0800)]
Fix the transformation of `(call ARRADDR_ST x, y, null)`.

`fgMorphCall` transforms `(call ARRADDR_ST x, y, null)` into
`(assign (arrindex x y) null)`. The transformation was not taking into
account the fact that the operands to the call might have been spilled
to temps, and was therefore silently dropping code in the case that they
had been.

This change collects all argument setup nodes in `gtCallArgs` in a
sequence of comma nodes if any exist and attaches the assignment
expression as the ultimate LHS of the comma tree.

Fixes dotnet/coreclr#8150.

Commit migrated from https://github.com/dotnet/coreclr/commit/f997866be19aa7086e75def295392cc69a76fd4d

7 years agoExclude tests that timeout with HeapVerify
Michelle McDaniel [Thu, 17 Nov 2016 17:16:28 +0000 (09:16 -0800)]
Exclude tests that timeout with HeapVerify

Commit migrated from https://github.com/dotnet/coreclr/commit/4193615add8777e351893ae81e321af93dee2035

7 years agoUse CoreFX Path code (dotnet/coreclr#8132)
Jeremy Kuhne [Thu, 17 Nov 2016 21:51:43 +0000 (13:51 -0800)]
Use CoreFX Path code (dotnet/coreclr#8132)

* Use CoreFX Path code

This is to address consistency and perf issues by moving the Path
code down to CoreCLR. It is more of an issue with Windows as
normalization requires complicated logic and costly platform calls.

* Delete unused files and avoid Unix crypto lib

* Address feedback, fix bug

Remove Unix crypto files and ifdef out usages completely.
Fix issue with static initialization of PathInternal.

* Use TriState to initialize case sensitivity

Commit migrated from https://github.com/dotnet/coreclr/commit/204f6e8859e3676114c85f49b8d2c311458ac715

7 years agoFix x86 long compares that produce a result
Mike Danes [Thu, 17 Nov 2016 21:15:26 +0000 (23:15 +0200)]
Fix x86 long compares that produce a result

Remove the duplicate genProduceReg call. Also remove an unnecessary movzx.

Commit migrated from https://github.com/dotnet/coreclr/commit/fca7e09e50bd20ed1917fd88dff390ad8c8acafc

7 years agoInliner: updates to RandomPolicy (dotnet/coreclr#8128)
Andy Ayers [Thu, 17 Nov 2016 18:52:06 +0000 (10:52 -0800)]
Inliner: updates to RandomPolicy (dotnet/coreclr#8128)

Fix issues with RandomPolicy setup in release builds with -DINLINE_DATA.

Reparent RandomPolicy on top of DiscretionaryPolicy to enable inline
data dumps from random inline runs. Remove some now-unneeded overrides.

Add a full dump mode to JitInlineDumpData that reports the inliner-visible
data for all inlines, as opposed to inliner-visible and post-inline
data for just the most recent inline. Small mods to the dumper code to
adjust comma placement for this new mode.

Have the RandomPolicy make the full set of profitability observations
for each accepted inline so they can be dumped.

Commit migrated from https://github.com/dotnet/coreclr/commit/db81622ed195624591e2151c4f2a009ddffef29e

7 years agoMerge pull request dotnet/coreclr#8138 from sivarv/rmwFix
Sivarv [Thu, 17 Nov 2016 18:19:11 +0000 (10:19 -0800)]
Merge pull request dotnet/coreclr#8138 from sivarv/rmwFix

Do not generate "inc/dec [mem]" when asked to generate debuggable code.

Commit migrated from https://github.com/dotnet/coreclr/commit/ae29fa880afb6a0043941800c2b29b2a1024d137

7 years agoTweak params of Console
Dan Moseley [Thu, 17 Nov 2016 17:27:09 +0000 (09:27 -0800)]
Tweak params of Console

This isn't the implementation of Console we ship, apparently it's only here for CLR tests, or maybe shouldn't be at all. Just tweaking the param names to match what we ship so it doesn't show up in the diffs.

Commit migrated from https://github.com/dotnet/coreclr/commit/d6e198c266d68dce6296572e51a20289f8164c84

7 years agoAddress PR feedback.
Pat Gavlin [Thu, 17 Nov 2016 16:47:27 +0000 (08:47 -0800)]
Address PR feedback.

Commit migrated from https://github.com/dotnet/coreclr/commit/c30cd2b84608cc855cf0603c268fb126665b42a4

7 years agoMerge pull request dotnet/coreclr#8151 from adiaaida/enableTests
Michelle McDaniel [Thu, 17 Nov 2016 16:36:21 +0000 (08:36 -0800)]
Merge pull request dotnet/coreclr#8151 from adiaaida/enableTests

Remove test exclusions for tests that now pass

Commit migrated from https://github.com/dotnet/coreclr/commit/13f1598437a42a2abec1be6bc4e0776ef4d00147

7 years agoAdd RuntimeHelpers.Equals
Dan Moseley [Thu, 17 Nov 2016 04:21:12 +0000 (20:21 -0800)]
Add RuntimeHelpers.Equals

Fix dotnet/corefxdotnet/coreclr#13692

We added this to the ref in 19fa781 but it was not present in the impl. The tools didn't flag this perhaps because Equals is available on Object, although since this is a new it really ought to know it wasn't referring to the Object one.

Commit migrated from https://github.com/dotnet/coreclr/commit/24ff0527361b6aa5026e6b7f8d4a189d3f595c65

7 years agoAdd RuntimeHelpers.Equals
danmosemsft [Thu, 17 Nov 2016 01:36:03 +0000 (17:36 -0800)]
Add RuntimeHelpers.Equals

Commit migrated from https://github.com/dotnet/coreclr/commit/42e882cf4fcf2cf2689b142cccf5b91bdb95502e

7 years agoTweak params
danmosemsft [Thu, 17 Nov 2016 01:08:56 +0000 (17:08 -0800)]
Tweak params

Commit migrated from https://github.com/dotnet/coreclr/commit/21002f6cc2d8ddec21797b71e73f72ebec7c783a

7 years agoAdd the missing implementation of IdnMapping.GetAscii (dotnet/coreclr#8149)
Tarek Mahmoud Sayed [Thu, 17 Nov 2016 01:08:45 +0000 (17:08 -0800)]
Add the missing implementation of IdnMapping.GetAscii (dotnet/coreclr#8149)

* Add the missing implementation of IdnMapping.GetAsccii

we should have this implementation in place but no idea why got removed in the first place

* Use nameof

Commit migrated from https://github.com/dotnet/coreclr/commit/529a9d25ba4f0869ac2e3a5a4259623af12f57de

7 years agoRemove test exclusions for tests that now pass
Michelle McDaniel [Thu, 17 Nov 2016 00:04:40 +0000 (16:04 -0800)]
Remove test exclusions for tests that now pass

This change removes tests that have been fixed for x86 RyuJIT. It also
updates the Issues for the remaining tests with additional details.

Commit migrated from https://github.com/dotnet/coreclr/commit/7d88885b72f4f632b289d472c2bcd659452eb478

7 years agoMerge pull request dotnet/coreclr#8146 from jashook/test_update_arm64
Jarret Shook [Wed, 16 Nov 2016 23:59:57 +0000 (15:59 -0800)]
Merge pull request dotnet/coreclr#8146 from jashook/test_update_arm64

Update arm64 tests used by the CI

Commit migrated from https://github.com/dotnet/coreclr/commit/7f46e5ddd7d8803f533efb3e62c246bb0003cd91

7 years agoMerge pull request dotnet/coreclr#8147 from rahku/reflection
Rahul Kumar [Wed, 16 Nov 2016 23:42:56 +0000 (15:42 -0800)]
Merge pull request dotnet/coreclr#8147 from rahku/reflection

Expose MethodBase.IsSecuritySafeCritical

Commit migrated from https://github.com/dotnet/coreclr/commit/e1a6a4de6c484bfaf443d9e5a6f40107c3f44b36

7 years agoDon't generate inc/dec [mem] while generating debuggable code.
sivarv [Wed, 16 Nov 2016 22:20:10 +0000 (14:20 -0800)]
Don't generate inc/dec [mem] while generating debuggable code.

Commit migrated from https://github.com/dotnet/coreclr/commit/d7dc0295e2fd7f0bb178972340a3b27465396579

7 years agoExpose MethodBase.IsSecuritySafeCritical
Rahul Kumar [Wed, 16 Nov 2016 21:38:38 +0000 (13:38 -0800)]
Expose MethodBase.IsSecuritySafeCritical

Commit migrated from https://github.com/dotnet/coreclr/commit/e40a56feb2b2fb0e14bf57c9d249e361cb80811d

7 years agoFormat code.
Pat Gavlin [Wed, 16 Nov 2016 21:35:06 +0000 (13:35 -0800)]
Format code.

Commit migrated from https://github.com/dotnet/coreclr/commit/0ccd73a224b9d474986bd02feaa45db1d95280d7

7 years agoDo not fold (ind (addr (lclVar))) in certain cases.
Pat Gavlin [Fri, 11 Nov 2016 21:41:09 +0000 (13:41 -0800)]
Do not fold (ind (addr (lclVar))) in certain cases.

Specifically, do not fold these trees into (lclFld) or (lclVar) if the
size of the access is larger than the size of the lclVar. Furthermore,
mark the lclVar as unenregisterable, as the indirection requires the
lclVar to be in memory.

Commit migrated from https://github.com/dotnet/coreclr/commit/52670d32714a2b1f09422d9ca08aab653158a1d6

7 years agoUpdate IssuesFeedbackEngagement.md
Dan Moseley [Wed, 16 Nov 2016 21:13:22 +0000 (13:13 -0800)]
Update IssuesFeedbackEngagement.md

Commit migrated from https://github.com/dotnet/coreclr/commit/124754f2f5382f2781db8f132e3c89d21b56600b

7 years agoExpose a member of StrongNameKeyPair, fix bug in SynchronizationContext (dotnet/corec...
Koundinya Veluri [Wed, 16 Nov 2016 20:55:53 +0000 (12:55 -0800)]
Expose a member of StrongNameKeyPair, fix bug in SynchronizationContext (dotnet/coreclr#8142)

Related to dotnet/corefxdotnet/coreclr#11636
Related to dotnet/corefxdotnet/coreclr#11808

Commit migrated from https://github.com/dotnet/coreclr/commit/3baf3a71dfa8a1504efd54973fe7ae5aedef0e13

7 years agoFixed string Join method
Dan Moseley [Wed, 16 Nov 2016 20:55:45 +0000 (12:55 -0800)]
Fixed string Join method

Fixed string Join(string separator, params object[] values) method.
Calling string.Join(",", null, 1, 2, 3); return empty string but should ",1,2,3".

Commit migrated from https://github.com/dotnet/coreclr/commit/b6e5a7fd2deaf73cb45ad82474b9e4beee7883a4

7 years agoUpdate arm64 tests used by the CI.
jashook [Wed, 16 Nov 2016 19:00:30 +0000 (11:00 -0800)]
Update arm64 tests used by the CI.

Commit migrated from https://github.com/dotnet/coreclr/commit/0546743b83506e5d048a4996014df3c8fdccc2c0

7 years agoMerge pull request dotnet/coreclr#8144 from adiaaida/formatArtifacts
Michelle McDaniel [Wed, 16 Nov 2016 17:48:23 +0000 (09:48 -0800)]
Merge pull request dotnet/coreclr#8144 from adiaaida/formatArtifacts

Fix the path to build artifacts for formatting

Commit migrated from https://github.com/dotnet/coreclr/commit/60254333e47e432691734ff8f14a8799842d5357

7 years agoFix the path to build artifacts for formatting
Michelle McDaniel [Wed, 16 Nov 2016 17:45:49 +0000 (09:45 -0800)]
Fix the path to build artifacts for formatting

Commit migrated from https://github.com/dotnet/coreclr/commit/967f4598f1ba31a4208253ce93f1d700abd950b0

7 years agoEnable GCToOSInterface to be defined on the GC side of the GC interface (dotnet/corec...
Sean Gillespie [Wed, 16 Nov 2016 17:34:57 +0000 (09:34 -0800)]
Enable GCToOSInterface to be defined on the GC side of the GC interface (dotnet/coreclr#8121)

* Re-introduce changes lost in a merge conflict

* Enable GCToOSInterface to be defined behind the GC interface
when building the GC in standalone mode. Provide a skeleton Windows
implementation and the framework for a Unix implementation.

* Address code review feedback

Commit migrated from https://github.com/dotnet/coreclr/commit/bb9f30e4658342f83bb3b5639ae24ec92e0a1d2a

7 years agoFull support of idn mapping (dotnet/coreclr#8134)
Tarek Mahmoud Sayed [Wed, 16 Nov 2016 01:35:36 +0000 (17:35 -0800)]
Full support of idn mapping (dotnet/coreclr#8134)

* Enable full implementation of IdnMapping in corelib

* Add missing resources

Commit migrated from https://github.com/dotnet/coreclr/commit/2c0a2c05ba82460a8d8a4b1e2d98e908e59d5d54

7 years agoMerge pull request dotnet/coreclr#8135 from pgavlin/fixExtendPtrVN
Pat Gavlin [Wed, 16 Nov 2016 01:00:10 +0000 (17:00 -0800)]
Merge pull request dotnet/coreclr#8135 from pgavlin/fixExtendPtrVN

Fix value numbering for FieldSeqStore::NotAField.

Commit migrated from https://github.com/dotnet/coreclr/commit/c917228c41b812e10ae7c9610570c205e17e13cf

7 years agoFormat code.
Pat Gavlin [Wed, 16 Nov 2016 00:19:57 +0000 (16:19 -0800)]
Format code.

Commit migrated from https://github.com/dotnet/coreclr/commit/b4d13d5f8b2676c8bc07a80bc422ef9c30d747a3

7 years agoChanging JIT perf test names to fix how they are organized on BenchView (dotnet/corec...
jorive [Tue, 15 Nov 2016 23:25:52 +0000 (15:25 -0800)]
Changing JIT perf test names to fix how they are organized on BenchView (dotnet/coreclr#8083)

* Changing JIT performance tests' names to fix how tests are organized on BenchView.

* Adding changes based on PR suggestions.
- Add double quotes on cmd script
- Keep name casing consistent for the Benchstone benchmarks.

* Updating datacontractxml value to match new class name.

* Removing an extra / on the xmlns string.

* Making sure we are getting latest BenchView tools.
- Changed where BenchView tools are installed to (from C:\tools to %WORKSPACE%)
- Check if the tools already exist, if so, delete them and download them again.

Commit migrated from https://github.com/dotnet/coreclr/commit/164b3f568209bdbe3ae40477decfb8c2e0e22382

7 years agoMerge pull request dotnet/coreclr#8110 from BruceForstall/FixLIRCallArgDumping
Bruce Forstall [Tue, 15 Nov 2016 23:25:05 +0000 (15:25 -0800)]
Merge pull request dotnet/coreclr#8110 from BruceForstall/FixLIRCallArgDumping

Fix the arg name/number dumping for call args in LIR

Commit migrated from https://github.com/dotnet/coreclr/commit/fd16499452a87cf75dbc1d64f3cc746a7ddcad9a

7 years agoFix value numbering for FieldSeqStore::NotAField.
Pat Gavlin [Tue, 15 Nov 2016 22:28:30 +0000 (14:28 -0800)]
Fix value numbering for FieldSeqStore::NotAField.

Before this change, value numbering produced the same value number for
each occurrence of `FieldSeqStore::NotAField`. This semantics is not
logically consistent with the semantics of `NotAField`, however:
`NotAField` indicates that we cannot reason about the address expression
that it annotates, therefore we cannot reason about the value stored at
the corresponding location. A recent change to admit the VN for
`NotAField` in `ExtendPtrVN` exposed SBCG due to this mismatch in
semantics: distinct array elements were assigned the same value number
because each had the same base coupled with the `NotAField` VN.

This change revises the representation of `NotAField` during value
numbering: instead of reusing the same value number for each
occurrence of `NotAField`, each occurrence is instead assigned a new,
unique value number. These value numbers are represented using a new
chunk attribute to retain the ability to decide whether or not a value
number represents `NotAField` without the use of a more heavyweight map
(e.g. a `VNMap`).

Fixes dotnet/coreclr#8133.

Commit migrated from https://github.com/dotnet/coreclr/commit/cfc122c62f1f00dafb2950d103275839f0dcb3a1

7 years agoMerge pull request dotnet/coreclr#8125 from stephentoub/divrem_perf
Stephen Toub [Tue, 15 Nov 2016 22:22:53 +0000 (17:22 -0500)]
Merge pull request dotnet/coreclr#8125 from stephentoub/divrem_perf

Improve Math.DivRem performance

Commit migrated from https://github.com/dotnet/coreclr/commit/68e3e94374906bfef85de4881967aa313e9c6a6c

7 years agoExpose members of ExecutionContext and AsyncFlowControl (dotnet/coreclr#8107)
Koundinya Veluri [Tue, 15 Nov 2016 21:08:41 +0000 (13:08 -0800)]
Expose members of ExecutionContext and AsyncFlowControl (dotnet/coreclr#8107)

Expose members of ExecutionContext and AsyncFlowControl

- Fix ExecutionContext.Capture and AsyncFlowControl.Undo when flow is suppressed
- Expose deserialization constructor for ExecutionContext

Commit migrated from https://github.com/dotnet/coreclr/commit/2246c1ff94cbd84df83e5308c81a1d1018c3f154

7 years agoMerge pull request dotnet/coreclr#8131 from JohnChen0/xmm
John Chen [Tue, 15 Nov 2016 20:44:10 +0000 (12:44 -0800)]
Merge pull request dotnet/coreclr#8131 from JohnChen0/xmm

Fix XMM register count in CONTEXT_SetThreadContextOnPort in PAL

Commit migrated from https://github.com/dotnet/coreclr/commit/f4c1915a833fadbf4e47c2bb31b425049d048343

7 years agoMerge pull request dotnet/coreclr#8124 from martijnhoekstra/doc-aggex
Dan Moseley [Tue, 15 Nov 2016 20:00:13 +0000 (12:00 -0800)]
Merge pull request dotnet/coreclr#8124 from martijnhoekstra/doc-aggex

fix grammar for AggregateException.Flatten

Commit migrated from https://github.com/dotnet/coreclr/commit/59c9e61eb49aad0f5a1306d13e08731b7669fcbe

7 years agoFix XMM register count in CONTEXT_SetThreadContextOnPort in PAL
John Chen (CLR) [Tue, 15 Nov 2016 19:09:20 +0000 (11:09 -0800)]
Fix XMM register count in CONTEXT_SetThreadContextOnPort in PAL

Commit migrated from https://github.com/dotnet/coreclr/commit/e88fc98866433eb053160f50684ca26208953875

7 years agoMerge pull request dotnet/coreclr#8127 from jashook/arm64_post_build_script
Jarret Shook [Tue, 15 Nov 2016 18:10:11 +0000 (10:10 -0800)]
Merge pull request dotnet/coreclr#8127 from jashook/arm64_post_build_script

Change over from the old arm64 scripting.

Commit migrated from https://github.com/dotnet/coreclr/commit/ecc88a6a4d8fe7cba1905f8f548c5a1eef603db0

7 years agoChange over from the old ar64 scripting.
jashook [Tue, 15 Nov 2016 01:48:56 +0000 (17:48 -0800)]
Change over from the old ar64 scripting.

This change moves away from hosting all of scripts on the Z:\ drive.
The share that used to be shared between the x64 machines and the
arm64 devices. It removes all the old scripting for one python script
arm64_post_build.py, which is responsible for setup of dotnet and
launching
the x64_client.

Commit migrated from https://github.com/dotnet/coreclr/commit/896858eb63b455588976d8d35ee8b4e750db420b

7 years agoMerge pull request dotnet/coreclr#8120 from adiaaida/fixDebuggerVarargs
Michelle McDaniel [Tue, 15 Nov 2016 15:59:56 +0000 (07:59 -0800)]
Merge pull request dotnet/coreclr#8120 from adiaaida/fixDebuggerVarargs

Add special handling of varargs for x86 debugger

Commit migrated from https://github.com/dotnet/coreclr/commit/294bb94d18cc8f9d13ae69f51e002f31224d4bdd

7 years agoFix crash in FunctionMemberPtrArrayHolder destructor (dotnet/coreclr#8113)
Igor Kulaychuk [Tue, 15 Nov 2016 14:55:53 +0000 (17:55 +0300)]
Fix crash in FunctionMemberPtrArrayHolder destructor (dotnet/coreclr#8113)

Some elements in FunctionMember array might not be initialized
and contain garbage. This leads to crash when elements are being deleted.

Move array allocation inside FunctionMemberPtrArrayHolder class
and initialize all array elements to nullptr.

Commit migrated from https://github.com/dotnet/coreclr/commit/de9e159ce0b1abad260100d447071c3377d2f34c

7 years agoclean build-roofs and add lldb option (dotnet/coreclr#8123)
SaeHie Park [Tue, 15 Nov 2016 14:49:02 +0000 (23:49 +0900)]
clean build-roofs and add lldb option (dotnet/coreclr#8123)

- clean unused __MachineTriple
- add indents to case
- add lld 3.6 or 3.8 as option

Commit migrated from https://github.com/dotnet/coreclr/commit/301ec423beaf8f31fe6172a846ec552c2cb9a14e

7 years agoUse Math.DivRem in another place in mscorlib
Stephen Toub [Tue, 15 Nov 2016 14:36:38 +0000 (09:36 -0500)]
Use Math.DivRem in another place in mscorlib

Went through all uses of % looking for places DivRem could be used.  This looks like it's the only one of note.

Commit migrated from https://github.com/dotnet/coreclr/commit/64edfc9b668d2f4af8c32d7dda2e64e776b40af1

7 years agoImprove Math.DivRem performance
Stephen Toub [Tue, 15 Nov 2016 12:28:20 +0000 (07:28 -0500)]
Improve Math.DivRem performance

Until the JIT is able to eliminate one of the two idiv operations, using a multiplication and subtraction is measurably faster than an extra division.

Commit migrated from https://github.com/dotnet/coreclr/commit/bd1f21d17c439481d7bc2a5219779e2474419a96

7 years agofix grammar for AggregateException.Flatten
Martijn Hoekstra [Tue, 15 Nov 2016 12:24:05 +0000 (13:24 +0100)]
fix grammar for AggregateException.Flatten

The old phrasing wasn't gramatically correct. It's somewhat
difficult to find a concise summary

Commit migrated from https://github.com/dotnet/coreclr/commit/463aca61eb527ac86f111d6e2e6c5519c84db850

7 years agoReduce variable scope (dotnet/coreclr#8021)
Dan Moseley [Mon, 14 Nov 2016 23:47:55 +0000 (15:47 -0800)]
Reduce variable scope (dotnet/coreclr#8021)

Commit migrated from https://github.com/dotnet/coreclr/commit/7bd7335b34b3e4ee650b5db4991d922e3fbc56ea

7 years agoMerge pull request dotnet/coreclr#8073 from dotnet/enablebinclash
chcosta [Mon, 14 Nov 2016 23:17:54 +0000 (15:17 -0800)]
Merge pull request dotnet/coreclr#8073 from dotnet/enablebinclash

Enable binclash logger

Commit migrated from https://github.com/dotnet/coreclr/commit/41f042f71c6d70d08813a9d5068b0a1918176a2d

7 years agoMerge pull request dotnet/coreclr#8086 from pgavlin/VSO278523
Pat Gavlin [Mon, 14 Nov 2016 23:15:34 +0000 (15:15 -0800)]
Merge pull request dotnet/coreclr#8086 from pgavlin/VSO278523

Admit more dest addresses in block assign value-numbering.

Commit migrated from https://github.com/dotnet/coreclr/commit/a5638a742043e2698a050e7970d4917b5de5bff3

7 years agoAdmit more dest addresses in block assign value-numbering.
Pat Gavlin [Fri, 11 Nov 2016 18:20:59 +0000 (10:20 -0800)]
Admit more dest addresses in block assign value-numbering.

Internal testing revealed an assertion when computing the value number
for block assignments: the VN framework expected any block assignments
to a tracked lclVar to have a destination address of the form
`(addr (lclVar))` or `(addr (lclFld))`. However, the check that it
uses to determine whether or not a block assignment targets a lclVar
also admits addresses formed by some combination of adds of constants
to these patterns (e.g. `(add (const 4) (add (addr lclVar) (const 4)))`.

This fix admits these additional patterns in value numbering by allowing
`ExtendPtrVN(GenTree*, GenTree*)` to supply `NotAField` to
`ExtendPtrVN(GenTree*, FieldSeqNode*)`.

Commit migrated from https://github.com/dotnet/coreclr/commit/f6eded26022145c80bf6309e59313a85d8a09f8d

7 years agoAdd special handling of varargs for x86 debugger
Michelle McDaniel [Mon, 14 Nov 2016 21:44:06 +0000 (13:44 -0800)]
Add special handling of varargs for x86 debugger

On x86, stack arguments must be accessed via the varargs cookie for
varargs. Since the functions would be identical, this change moves the
code from codegenlinear to codegencommon to fix vararg argument
handling in the debugger.

Fixes VSO 284774.

Commit migrated from https://github.com/dotnet/coreclr/commit/3941d170a53c5d63c5da35c123f9d22cc62f4616

7 years agoFix the arg name/number dumping for call args in LIR
Bruce Forstall [Mon, 14 Nov 2016 03:17:36 +0000 (19:17 -0800)]
Fix the arg name/number dumping for call args in LIR

Use the arg number in the arg info table. Before this, I saw cases
where the args were named: arg0, arg1, arg1, arg3, instead of:
arg0, arg2, arg1, arg3, for example.

Commit migrated from https://github.com/dotnet/coreclr/commit/4535df67481edeeefbed21efcaceaf4a504ebcc9

7 years agoMerge pull request dotnet/coreclr#8111 from BruceForstall/FixTestNewlines
Bruce Forstall [Mon, 14 Nov 2016 21:44:06 +0000 (13:44 -0800)]
Merge pull request dotnet/coreclr#8111 from BruceForstall/FixTestNewlines

Don't generate explicit newlines in SIMD tests

Commit migrated from https://github.com/dotnet/coreclr/commit/152550bd09144a061dcb1d0675a9859923032aa5

7 years agoMerge pull request dotnet/coreclr#8115 from chcosta/updatebuildtools
chcosta [Mon, 14 Nov 2016 21:40:51 +0000 (13:40 -0800)]
Merge pull request dotnet/coreclr#8115 from chcosta/updatebuildtools

Update buildtools to version 01008-01

Commit migrated from https://github.com/dotnet/coreclr/commit/e4a36d27e4ea7ce177af7400b5727f68fcbcb90d

7 years agoMerge pull request dotnet/coreclr#8031 from pgavlin/VSO283795
Pat Gavlin [Mon, 14 Nov 2016 21:10:52 +0000 (13:10 -0800)]
Merge pull request dotnet/coreclr#8031 from pgavlin/VSO283795

Do not fold `(x + null)` in certain cases.

Commit migrated from https://github.com/dotnet/coreclr/commit/8914cb0fb1b512e77343c3006a95dd1b7eef4197

7 years agoMerge pull request dotnet/coreclr#8101 from sivarv/rmwOpt
Sivarv [Mon, 14 Nov 2016 19:07:36 +0000 (11:07 -0800)]
Merge pull request dotnet/coreclr#8101 from sivarv/rmwOpt

Generate "inc/dec [mem]" instead of "add/sub [mem], 1" for Read-Modify-Write (RMW) operations.

Commit migrated from https://github.com/dotnet/coreclr/commit/1f227ce8f53882520fba9ad383f5740e5daecc0a

7 years agoFix test exclusions.
Pat Gavlin [Mon, 14 Nov 2016 19:02:12 +0000 (11:02 -0800)]
Fix test exclusions.

The new regression test should only be disabled for builds that use the
legacy backend.

Commit migrated from https://github.com/dotnet/coreclr/commit/62f116de0a4752046ab951312b5d242aed16f12d

7 years agoMerge pull request dotnet/coreclr#8100 from rahku/reflection
Rahul Kumar [Mon, 14 Nov 2016 18:17:28 +0000 (10:17 -0800)]
Merge pull request dotnet/coreclr#8100 from rahku/reflection

Expose Assembly::GetFile(s) apis

Commit migrated from https://github.com/dotnet/coreclr/commit/d67e1627158ff5c973b21faf9fd03ebbb49f4679

7 years agoUpdate buildtools to version 01008-01
Christopher Costa [Mon, 14 Nov 2016 16:40:16 +0000 (08:40 -0800)]
Update buildtools to version 01008-01

Commit migrated from https://github.com/dotnet/coreclr/commit/b30be6971ce58f87fd15478fc62b0f581cf4cae4

7 years agoFixed string Join method
Alexander Radchenko [Mon, 14 Nov 2016 15:58:58 +0000 (22:58 +0700)]
Fixed string Join method

Fixed string Join(string separator, params object[] values) method.
Calling string.Join(",", null, 1, 2, 3); return empty string but should ",1,2,3".

Commit migrated from https://github.com/dotnet/coreclr/commit/976519d243e28ced430efddbfc932390f3b37528

7 years agoMerge pull request dotnet/coreclr#8108 from gkhanna79/Arm32Test
Gaurav Khanna [Mon, 14 Nov 2016 05:21:07 +0000 (21:21 -0800)]
Merge pull request dotnet/coreclr#8108 from gkhanna79/Arm32Test

Enable WinArm32 P1 Test Build

Commit migrated from https://github.com/dotnet/coreclr/commit/32fa4911074bf95a30f7fa1828701fb920af91c6

7 years agoDon't generate explicit newlines in SIMD tests
Bruce Forstall [Mon, 14 Nov 2016 03:20:55 +0000 (19:20 -0800)]
Don't generate explicit newlines in SIMD tests

Use Console.WriteLine() instead.

Commit migrated from https://github.com/dotnet/coreclr/commit/ba0ed89958b2c1185b4dc6aba75598d9b542f90e

7 years agoEnable WinArm32 P1 Test Build
Gaurav Khanna [Sun, 13 Nov 2016 23:47:11 +0000 (15:47 -0800)]
Enable WinArm32 P1 Test Build

Commit migrated from https://github.com/dotnet/coreclr/commit/ec01f8b6cc44b3fb89933dc1008bfb6a0217727c

7 years agoInliner: ensure all statements have an inline context (dotnet/coreclr#8064)
Andy Ayers [Sun, 13 Nov 2016 22:15:14 +0000 (14:15 -0800)]
Inliner: ensure all statements have an inline context (dotnet/coreclr#8064)

* JIT: Retype fgMorphStmt and related variables

There are quite a few variables typed as `GenTreePtr` that must point
at statements -- for instance `fgMorphStmt`. Retype these as `GenTreeStmt*`.
Cleanup dereferences which no longer require a `gtStmt`.

* Inliner: ensure all statements have an inline context

We would previously allow some statements inserted by the inliner
"between" the caller and callee to have a null inline context. This
lack of context could lead to incomplete or misleading inline trees
if the inserted code included calls. In particular some failed inlines
might not show up in the trees.

Set the context for these inserted statements to be the context for the
call being inlined (thus they logically "belong" to the caller).

Commit migrated from https://github.com/dotnet/coreclr/commit/445405d84034564a6433074e0f920f74105ba499

7 years agoAdd implementation for AssemblyName::EscapedCodeBase (dotnet/coreclr#8072)
Rahul Kumar [Sun, 13 Nov 2016 17:34:58 +0000 (09:34 -0800)]
Add implementation for AssemblyName::EscapedCodeBase (dotnet/coreclr#8072)

Commit migrated from https://github.com/dotnet/coreclr/commit/b28f557ee79d70a83e299d6c83a2c803139edba2

7 years agoRemove conflicting proj files (dotnet/coreclr#8090)
John Chen [Sun, 13 Nov 2016 00:20:14 +0000 (16:20 -0800)]
Remove conflicting proj files (dotnet/coreclr#8090)

Some test cases have both .ilproj and .csproj files, resulting
in a race condition during build. Removing the .csproj files,
as the MSIL versions appear to be the intended ones.

Commit migrated from https://github.com/dotnet/coreclr/commit/dfd0e4b4e20d00a02fdd412251f6146a639ae33c

7 years agoMerge pull request dotnet/coreclr#8085 from stephentoub/daylight_alloc
Stephen Toub [Sat, 12 Nov 2016 14:19:33 +0000 (09:19 -0500)]
Merge pull request dotnet/coreclr#8085 from stephentoub/daylight_alloc

Avoid allocations in DateTime.Now

Commit migrated from https://github.com/dotnet/coreclr/commit/ca6ca4746754fa87aff344c32ed4b004ce8746de

7 years agoAvoid allocations in DateTime.Now
Stephen Toub [Sat, 12 Nov 2016 12:51:08 +0000 (07:51 -0500)]
Avoid allocations in DateTime.Now

Every call to DateTime.Now ends up allocating a DaylightTime as it computes the offset from UtcNow.  DaylightTime is just a plain old object that stores two DateTimes and a Timespan.  Ideally we could just change it to be a struct, but it's public.  Instead, this commit adds an internal DaylightTimeStruct, which is the same as DaylightTime except as a value type, and changes some of the internal uses of DaylightTime to instead use DaylightTime.  As a result, DateTime.Now is allocation-free.

Commit migrated from https://github.com/dotnet/coreclr/commit/2457e2b3831d73ff5178961a62f7a6fcc332eceb

7 years agoInliner: updates to random inlining (dotnet/coreclr#8088)
Andy Ayers [Sat, 12 Nov 2016 03:52:24 +0000 (19:52 -0800)]
Inliner: updates to random inlining (dotnet/coreclr#8088)

Cache the random state on the InlineStrategy instead of on the compiler
instance so that the state is reinitialized and private to each jit request.
That way the random policy evaluations made compiling one method won't
alter the evaluations for subsequent methods. This should make it somewhat
easier to do A/B comparisons under jit stress when changing the jit.

Make it possible to enable the RandomPolicy outside of stress. This may
prove useful in various randomized inline performance studies or as
a simple stress mode on its own.

Random state seed is built from an external seed value (via JitStress or
JitInlinePolicyRandom) and an internal seed value (method hash), so that
random sequences potentially differ for each method but are deterministic
across runs and changes to the jit.

Commit migrated from https://github.com/dotnet/coreclr/commit/7697dec7db3e34239ffa33d8867de12f7176e415

7 years agoRemove redundant statement (dotnet/coreclr#7900)
Dmitry-Me [Sat, 12 Nov 2016 00:54:37 +0000 (03:54 +0300)]
Remove redundant statement (dotnet/coreclr#7900)

Commit migrated from https://github.com/dotnet/coreclr/commit/6d93a827a2417662e64d55fea252acd49b5b9ebb

7 years agoGenerate inc/dec [mem] instead of add/sub [mem], 1 for read-modify-write operations.
sivarv [Sat, 12 Nov 2016 00:41:16 +0000 (16:41 -0800)]
Generate inc/dec [mem] instead of add/sub [mem], 1 for read-modify-write operations.

Commit migrated from https://github.com/dotnet/coreclr/commit/8753709778f4282b16647946dfe61cfd8f8414e5

7 years agoEnable the rest if TimeZoneInfo APIs (dotnet/coreclr#8096)
Tarek Mahmoud Sayed [Fri, 11 Nov 2016 23:46:47 +0000 (15:46 -0800)]
Enable the rest if TimeZoneInfo APIs (dotnet/coreclr#8096)

Commit migrated from https://github.com/dotnet/coreclr/commit/38b1f672caeb8f01aa2970c02c06b4ae7fcf06c4

7 years agoExpose Assembly::GetFile(s) apis
Rahul Kumar [Fri, 11 Nov 2016 20:10:16 +0000 (12:10 -0800)]
Expose Assembly::GetFile(s) apis

Commit migrated from https://github.com/dotnet/coreclr/commit/b654a3dd2a8dc0190257dd326d651846cd6ce3b6

7 years agoRename gitmirrorfile
Matt Ellis [Fri, 11 Nov 2016 22:20:40 +0000 (14:20 -0800)]
Rename gitmirrorfile

Commit migrated from https://github.com/dotnet/coreclr/commit/17b820f470b22f0d8b8d5cb91257bac2b0bee5b4

7 years agoMerge pull request dotnet/coreclr#8087 from ellismg/selective-mirror-sketch
Matt Ellis [Fri, 11 Nov 2016 21:52:11 +0000 (13:52 -0800)]
Merge pull request dotnet/coreclr#8087 from ellismg/selective-mirror-sketch

Update .gitmirror metadata

Commit migrated from https://github.com/dotnet/coreclr/commit/be1710cde6fe0ad16716560f9c8c6c880a248285

7 years agoRefactor some logic in GT_IND morphing.
Pat Gavlin [Fri, 11 Nov 2016 20:34:28 +0000 (12:34 -0800)]
Refactor some logic in GT_IND morphing.

This change separates the logic that morphs child trees that reference
lclVars in preparation to use them as the result of a GT_IND fold from
the logic that sets the flags on the result of the fold. This also
standardizes GT_IND morphing on using the `typ` local to refer to the
type of the tree being morphed (rather than a mix of `typ` and
`tree->TypeGet()`).

Commit migrated from https://github.com/dotnet/coreclr/commit/39cb6bb082430bcec17ba061e0cc9f62972b8029

7 years agoMerge pull request dotnet/coreclr#8001 from JosephTremoulet/UnrollSimd
Joseph Tremoulet [Fri, 11 Nov 2016 19:37:45 +0000 (14:37 -0500)]
Merge pull request dotnet/coreclr#8001 from JosephTremoulet/UnrollSimd

Enable unrolling of SIMD_LIMIT loops

Commit migrated from https://github.com/dotnet/coreclr/commit/5ae721ccb6b3746bb42fbc91e8349fa7f4b064d2

7 years agoEnable unrolling of SIMD_LIMIT loops
Joseph Tremoulet [Fri, 14 Oct 2016 18:53:52 +0000 (14:53 -0400)]
Enable unrolling of SIMD_LIMIT loops

Since the Vector<T> abstraction has a `Count` that is not a
C#-compile-time constant, it encourages use of iteration to
search/aggregate individual elements using symbolic indexing, which in
turn leads to codegen that spills the vector to memory for each element
access, and performs bounds checks for each access.  These loops will have
low trip counts that are jit-compile-time constant, and constant indexing
into Vector<T> allows more efficient register-to-register sequences and
bounds-check elision.  This change enables RyuJit's loop unroller when
such a loop is discovered, and increases the size threshold to target
optimizing such loops much more aggressively than the unroller's previous
incarnation.

Add a test with a motivating loop to the Performance/CodeQuality/SIMD
suite.

Closes dotnet/coreclr#7843.

Commit migrated from https://github.com/dotnet/coreclr/commit/e3e7d1ac2e2949479ff64c3de4b6aebe271d1ed4

7 years agoDetect overflow in unroller cost computation
Joseph Tremoulet [Thu, 10 Nov 2016 21:37:05 +0000 (16:37 -0500)]
Detect overflow in unroller cost computation

Make sure to avoid trying to unroll cases so large as to overlow the cost.

Commit migrated from https://github.com/dotnet/coreclr/commit/634e3391953123acd61aa3846b25546b97fe1665

7 years agoAllow unrolling loops with multiple branches
Joseph Tremoulet [Fri, 14 Oct 2016 18:47:20 +0000 (14:47 -0400)]
Allow unrolling loops with multiple branches

Lift both the single-exit restriction and the no-internal-branching
restriction.  Share some utilities with the loop cloner to facilitate
this (particularly `CloneBlockState` and `fgUpdateChangedFlowGraph`).

Commit migrated from https://github.com/dotnet/coreclr/commit/705c3375224b7c6d5ed3a195edcec8e2b7a0e9c9

7 years agoUnroll loops in inner-to-outer order
Joseph Tremoulet [Wed, 19 Oct 2016 18:21:07 +0000 (14:21 -0400)]
Unroll loops in inner-to-outer order

There's no need for fixpoint iteration; the loop indices are a pre-order,
so walking them in reverse order will visit inner loops before outer ones.

Commit migrated from https://github.com/dotnet/coreclr/commit/56dac6e09a732cf371baae2b0d696efb93861824

7 years agoStop expecting ASG_ operators in loop unroller
Joseph Tremoulet [Fri, 14 Oct 2016 18:31:45 +0000 (14:31 -0400)]
Stop expecting ASG_ operators in loop unroller

Expect instead to see arithmetic nodes that are arguments of separate
assign nodes.

Commit migrated from https://github.com/dotnet/coreclr/commit/4251467f9512baededbdbe86d3fcf6d147833727