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

Do not pass /O1 except in release/release with debug info modes #1144

Merged
merged 1 commit into from
Sep 15, 2015

Conversation

mmitche
Copy link
Member

@mmitche mmitche commented Jun 16, 2015

No description provided.

@janvorli
Copy link
Member

LGTM

@hadibrais
Copy link

It seems that this test depends on the presence of /O1. Otherwise, a StackOverflowException is thrown.
#1130

@mmitche
Copy link
Member Author

mmitche commented Jun 22, 2015

@hadibrais Yeah...I'm going to look into it.

@mikedn
Copy link

mikedn commented Sep 13, 2015

@mmitche You can avoid the stack overflow if you slightly increase the size of the stack reserve for corerun.exe:

set(CMAKE_EXE_LINKER_FLAGS "${CMAKE_EXE_LINKER_FLAGS} /STACK:1572864")

It would be nice to have this in, especially the /OPT:ICF part which messes up JIT's GenTree debugging option.

@mmitche
Copy link
Member Author

mmitche commented Sep 14, 2015

@mikedn I'll try to get to this today.

@mmitche
Copy link
Member Author

mmitche commented Sep 15, 2015

@mikedn That worked thanks.

mmitche added a commit that referenced this pull request Sep 15, 2015
Do not pass /O1 except in release/release with debug info modes
@mmitche mmitche merged commit 2bc5657 into dotnet:master Sep 15, 2015
@mmitche mmitche deleted the real-debug-build branch September 15, 2015 17:30
picenka21 pushed a commit to picenka21/runtime that referenced this pull request Feb 18, 2022
Do not pass /O1 except in release/release with debug info modes

Commit migrated from dotnet/coreclr@2bc5657
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.

6 participants