projects / platform / upstream / coreclr.git / commit
? search:
summary | shortlog | log | commit | commitdiff | tree
(parent: 3609269) | patch
Don't early-propagate negative array lengths
authorMike Danes <onemihaid@hotmail.com>
Thu, 21 Sep 2017 19:50:16 +0000 (22:50 +0300)
committerMike Danes <onemihaid@hotmail.com>
Thu, 21 Sep 2017 19:50:16 +0000 (22:50 +0300)
commitbc407ced89e9b2a5005ffbdb1756c1fafe93f068
treeb7413a7c0e45027e16bf8aad7ff103b8c7356b63tree | snapshot
parent3609269610ba7df0e52809b989e69a00dfeb089acommit | diff
Don't early-propagate negative array lengths

There's no need for that and if the negative array length is not representable in 32 bit we'll end up producing a GT_CNS_INT node that has TYP_INT and a 64 bit value.

That's because the original type (always TYP_INT) of the GT_ARR_LENGTH is preserved when changing the node to GT_CNS_INT.
src/jit/earlyprop.cpp diff | blob | history
tests/src/JIT/Regression/JitBlue/GitHub_14116/GitHub_14116.il [new file with mode: 0644] blob
tests/src/JIT/Regression/JitBlue/GitHub_14116/GitHub_14116.ilproj [new file with mode: 0644] blob
Domain: Dotnet / Core;