<html>
<head>
<style><!--
.hmmessage P
{
margin:0px;
padding:0px
}
body.hmmessage
{
font-size: 12pt;
font-family:Calibri
}
--></style></head>
<body class='hmmessage'><div dir='ltr'><font color="#000000" face="Calibri,sans-serif">I just noticed this commit by @spouliot: <a href="https://github.com/mono/mono/commit/298962b7ddd5e3af33c3177e8523cc36da4de553">https://github.com/mono/mono/commit/298962b7ddd5e3af33c3177e8523cc36da4de553</a></font><BR><font color="#000000" face="Calibri,sans-serif"></font> <BR><font color="#000000" face="Calibri,sans-serif">In my opinion, this isn't the right approach, we should rather fix the cases where a cast would overflow in MS referencesource code rather than changing the tests.</font><BR> <BR><font color="#000000" face="Calibri,sans-serif">I sent a PR a week or so ago that fixes the particular DateTime tests on ARM by explictly checking if the value fits into long, which I think is better: </font><a href="https://github.com/mono/referencesource/pull/8">https://github.com/mono/referencesource/pull/8</a><BR> <BR>There are a couple more of these overflows in MS code that make tests fail and I think we should discuss what the best approach is. What are your thoughts?<BR> <BR>-- Alex<BR>                                      </div></body>
</html>