Skip to content

fix Issue 17605 - [REG2.066.0] __traits(compiles, closure) adds link-…#7897

Merged
WalterBright merged 1 commit intodlang:masterfrom
WalterBright:fix17605
Feb 17, 2018
Merged

fix Issue 17605 - [REG2.066.0] __traits(compiles, closure) adds link-…#7897
WalterBright merged 1 commit intodlang:masterfrom
WalterBright:fix17605

Conversation

@WalterBright
Copy link
Member

…time reference to _d_allocmemory

The offending line was added by #3625 so we'll try removing the line and see what happens.

@dlang-bot
Copy link
Contributor

Thanks for your pull request, @WalterBright!

Bugzilla references

Auto-close Bugzilla Severity Description
17605 regression [REG2.066.0] __traits(compiles, closure) adds link-time reference to _d_allocmemory

⚠️⚠️⚠️ Warnings ⚠️⚠️⚠️

/*******************************************/
// https://issues.dlang.org/show_bug.cgi?id=17605

extern (C) test17605()
Copy link
Member

Choose a reason for hiding this comment

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

Needs return type: extern (C) void test17605().

@WalterBright
Copy link
Member Author

Clearly this is not a simple problem. Will need to work on it more.

@WalterBright WalterBright force-pushed the fix17605 branch 2 times, most recently from d2e5ac0 to 811ad94 Compare February 15, 2018 09:37
@JinShil
Copy link
Contributor

JinShil commented Feb 15, 2018

Easier to review with https://github.com/dlang/dmd/pull/7897/files?w=1

@JinShil
Copy link
Contributor

JinShil commented Feb 15, 2018

@WalterBright Is this ready to go?

@WalterBright
Copy link
Member Author

Yes, it's ready.

@RazvanN7
Copy link
Contributor

RazvanN7 commented Jan 25, 2023

FYI: #14846 fixed the underlying issue, so compileTimeOnly is no longer necessary. It is being removed in: #14847

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants