Skip to content
This repository was archived by the owner on Oct 12, 2022. It is now read-only.
/ druntime Public archive

Comments

Win64: set FPU CW to 64-bit, set sprintf output format to 2 digits in ex...#545

Merged
MartinNowak merged 2 commits intodlang:masterfrom
rainers:win64_config_fp
Jul 15, 2013
Merged

Win64: set FPU CW to 64-bit, set sprintf output format to 2 digits in ex...#545
MartinNowak merged 2 commits intodlang:masterfrom
rainers:win64_config_fp

Conversation

@rainers
Copy link
Member

@rainers rainers commented Jul 14, 2013

...ponents

This changes the MS C runtime to better match what is done by the other platforms:

  • set precision of floating point operations to 64-bit mantissa
  • change the number of digits in floating point exponents emitted by sprintf from 3 to 2

There are unittests in phobos that detect the differences, but these are not run under Win64 so far.

src/rt/dmain2.d Outdated
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why the commented-out call?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This was meant to document the desired operation that does not exist for Win64. I guess that's not a really good idea, I'll remove it.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why does it differ from the FreeBSD asm block?

@rainers
Copy link
Member Author

rainers commented Jul 15, 2013

AFAIK the x64 assembler does not properly support accessing stack variables under Win64.

MartinNowak added a commit that referenced this pull request Jul 15, 2013
Win64: set FPU CW to 64-bit, set sprintf output format to 2 digits in ex...
@MartinNowak MartinNowak merged commit d33494c into dlang:master Jul 15, 2013
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants