Skip to content

Comments

[REG2.064a] Issue 11251 - Not able to link (Previous Definition Different)#2685

Merged
WalterBright merged 1 commit intodlang:masterfrom
9rnsr:fix11251
Oct 20, 2013
Merged

[REG2.064a] Issue 11251 - Not able to link (Previous Definition Different)#2685
WalterBright merged 1 commit intodlang:masterfrom
9rnsr:fix11251

Conversation

@9rnsr
Copy link
Contributor

@9rnsr 9rnsr commented Oct 20, 2013

http://d.puremagic.com/issues/show_bug.cgi?id=11251

By #2540 , implicitly defined C main has been moved to __entrypoint module. But, the codegen generates ModuleInfo symbol for the module. And vibe.d has its own main function in library. Then, compiling uer progam with vibe.d library will cause the conflict of two ModuleInfo symbols of __entrypoint module.

Due to avoid ModuleInfo symbol generation for __entrypoint, I added a special case in Module::genobjfile function. But, I'm not sure that the change won't cause any other problems...

By dlang#2540 , implicitly defined C main has been moved to __entrypoint module. But, the codegen generates ModuleInfo symbol for the module. And vibe.d has its own main function in library. Then, compiling uer progam with vibe.d library will cause the conflict of two ModuleInfo symbols of __entrypoint module.

Due to avoid ModuleInfo symbol generation for __entrypoint, I added a special case in Module::genobjfile function. But, I'm not sure that the change won't cause any other problems...
WalterBright added a commit that referenced this pull request Oct 20, 2013
[REG2.064a] Issue 11251 - Not able to link (Previous Definition Different)
@WalterBright WalterBright merged commit 01c985a into dlang:master Oct 20, 2013
WalterBright added a commit that referenced this pull request Oct 20, 2013
[REG2.064a] Issue 11251 - Not able to link (Previous Definition Different)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants