New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

JIT optimization - Perform additional constant propagation for expressions. #987

Closed
briansull opened this Issue May 12, 2015 · 3 comments

Comments

Projects
None yet
3 participants
@briansull
Contributor

briansull commented May 12, 2015

Thee following two cases should generate the same code:

public static double Run2A()
{
return Math.Sqrt(1) + Math.Sqrt(2);
}

public static double Run2B()
{
double res = Math.Sqrt(1) + Math.Sqrt(2);
return res;
}

@briansull

This comment has been minimized.

Show comment
Hide comment
@briansull

briansull Sep 28, 2017

Contributor

Resolving this old issue as I verified that it is fixed.

Contributor

briansull commented Sep 28, 2017

Resolving this old issue as I verified that it is fixed.

@briansull briansull closed this Sep 28, 2017

@AndreyAkinshin

This comment has been minimized.

Show comment
Hide comment
@AndreyAkinshin

AndreyAkinshin Sep 29, 2017

Member

@briansull, great, thanks!
Could you tell me which version of .NET Core and .NET Framework will contain the fix?

Member

AndreyAkinshin commented Sep 29, 2017

@briansull, great, thanks!
Could you tell me which version of .NET Core and .NET Framework will contain the fix?

@briansull

This comment has been minimized.

Show comment
Hide comment
@briansull

briansull Sep 29, 2017

Contributor

Any current 4.7 version should contain the fix.

filever c:\Windows\Microsoft.NET\Framework64\v4.0.30319\clrjit.dll
--a-- WAMD64 DLL ENU 4.7.2110.0 shp 1,129,112 08-31-2017 clrjit.dll

Contributor

briansull commented Sep 29, 2017

Any current 4.7 version should contain the fix.

filever c:\Windows\Microsoft.NET\Framework64\v4.0.30319\clrjit.dll
--a-- WAMD64 DLL ENU 4.7.2110.0 shp 1,129,112 08-31-2017 clrjit.dll

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment