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

Conversation

gkhanna79
Copy link
Member

No description provided.

@gkhanna79
Copy link
Member Author

@dotnet-bot test this please

@gkhanna79
Copy link
Member Author

@jkotas PTAL.

jkotas added a commit that referenced this pull request Feb 7, 2016
@jkotas jkotas merged commit 3593fba into dotnet:master Feb 7, 2016
@gkhanna79
Copy link
Member Author

@AndyAyersMS
Copy link
Member

Not part of your change, but related... Why do we link with /LTCG (in release modes) if we never compile with /GL?

@jkotas
Copy link
Member

jkotas commented Feb 7, 2016

https://github.com/dotnet/coreclr/blob/master/CMakeLists.txt#L426 is passing /GL for release modes. I have done a quick check and I see it passed in to cl in the .rsp file. Do you see it not getting passed it to the compiler in some cases?

@AndyAyersMS
Copy link
Member

Sorry for the false alarm, I was confused. I was just randomly looking through the file and didn't see the /GL.

@AndyAyersMS
Copy link
Member

@gkhanna79 I'm still seeing frequent linker crashes. Not a blocker since I can just rebuild, but annoying. Happens when I'm doing development and so am always incrementally rebuilding stuff.

@dotnet/jit-contrib anybody else running into this?

@jashook
Copy link

jashook commented Feb 18, 2016

I see link errors for the arm64 cross build only on certain machines. Unknown if it is related.

@gkhanna79
Copy link
Member Author

CFG is not enabled for Arm64

@LLITCHEV
Copy link

I used to see this every other time for Debug builds for Windows. Fr some reason the installation of My VS got corrupted yesterday (the mspdb14.dll) was failing version check, so I reinstalled VS last night. This morning I have seen the crash only twice.

@pgavlin
Copy link

pgavlin commented Feb 18, 2016

I've also run into this, specifically when incrementally linking crossgen.exe.

@BruceForstall
Copy link

I hit it almost every build. Succeeds on rebuild.

@gkhanna79
Copy link
Member Author

Do you see this on debug or release build? Also, does the failing build command line having incremental linking disabled (which is what this change enabled)?

@BruceForstall
Copy link

I mostly do plain vanilla debug x64 builds. Don't know what flags are used.

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.

9 participants