Skip to content
This repository has been archived by the owner on Jan 23, 2023. It is now read-only.

Changes for building using VS2015 #949

Merged
merged 1 commit into from
May 7, 2015
Merged

Changes for building using VS2015 #949

merged 1 commit into from
May 7, 2015

Conversation

jkotas
Copy link
Member

@jkotas jkotas commented May 7, 2015

Assorted set of changes to make build produce coreclr.dll using VS2015 successfully. Fixes #824.

More fixes are needed in the managed part of the build to make build work end to end on VS2015.

generation_allocation_pointer (consing_gen), generation_allocation_limit (consing_gen)));
generation_plan_allocation_start (consing_gen),
generation_allocation_pointer (consing_gen),
generation_allocation_limit (consing_gen)));
}

Choose a reason for hiding this comment

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

Looks like mixed tab/spaces

Copy link
Member Author

Choose a reason for hiding this comment

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

Fixed.

@jkotas
Copy link
Member Author

jkotas commented May 7, 2015

fmodf - fixed. I have also cleanup how fabsf is done in the PAL while I was on it.

@jkotas
Copy link
Member Author

jkotas commented May 7, 2015

@janvorli Could you please take a look?

@kangaroo
Copy link

kangaroo commented May 7, 2015

@jkotas fmodf changes broke the Linux build. Once that's resolve this LGTM

@janvorli
Copy link
Member

janvorli commented May 7, 2015

LGTM

@jkotas jkotas force-pushed the warnings branch 2 times, most recently from 632ae07 to 8c4e64e Compare May 7, 2015 17:30
jkotas added a commit that referenced this pull request May 7, 2015
Changes for building using VS2015
@jkotas jkotas merged commit 5f0f1c5 into dotnet:master May 7, 2015
@jkotas jkotas deleted the warnings branch June 8, 2015 12:47
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.

Fix warnings reported by VS 2015
6 participants