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

Do not use /OPT:REF /OPT:ICF for static libraries#2310

Merged
jkotas merged 1 commit intodotnet:masterfrom
mikedn:lnk4044
Dec 11, 2015
Merged

Do not use /OPT:REF /OPT:ICF for static libraries#2310
jkotas merged 1 commit intodotnet:masterfrom
mikedn:lnk4044

Conversation

@mikedn
Copy link

@mikedn mikedn commented Dec 11, 2015

These options are only valid on dlls and exes and generate linker warnings when used on static libraries

These options are only valid on dlls and exes and generate linker warnings when used on static libraries
@jkotas
Copy link
Member

jkotas commented Dec 11, 2015

LGTM. Thanks!

jkotas added a commit that referenced this pull request Dec 11, 2015
Do not use /OPT:REF /OPT:ICF for static libraries
@jkotas jkotas merged commit 57f7b99 into dotnet:master Dec 11, 2015
@mikedn mikedn deleted the lnk4044 branch December 14, 2015 17:26
mikedn added a commit to mikedn/coreclr that referenced this pull request Jan 3, 2016
This was fixed in dotnet#2310 and the fix was accidentally reverted in dotnet#2244 and dotnet#2425
@mikedn mikedn mentioned this pull request Jan 3, 2016
picenka21 pushed a commit to picenka21/runtime that referenced this pull request Feb 18, 2022
Do not use /OPT:REF /OPT:ICF for static libraries

Commit migrated from dotnet/coreclr@57f7b99
picenka21 pushed a commit to picenka21/runtime that referenced this pull request Feb 18, 2022
This was fixed in dotnet/coreclr#2310 and the fix was accidentally reverted in dotnet/coreclr#2244 and dotnet/coreclr#2425


Commit migrated from dotnet/coreclr@3179786
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