D issues are now tracked on GitHub. This Bugzilla instance remains as a read-only archive.
Issue 16646 - Header generation on druntime
Summary: Header generation on druntime
Status: RESOLVED WORKSFORME
Alias: None
Product: D
Classification: Unclassified
Component: dmd (show other issues)
Version: D2
Hardware: All All
: P1 blocker
Assignee: No Owner
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2016-10-29 12:48 UTC by Temtaime
Modified: 2022-03-24 13:19 UTC (History)
1 user (show)

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this issue.
Description Temtaime 2016-10-29 12:48:55 UTC 
druntime\core\atomic.di(238): Error: asm statement is assumed to be @system - mark it with '@trusted' if it is not

But compiling this file is OK.
Comment 1 Temtaime 2016-10-29 12:56:25 UTC 
Also it generated strange statements.

ref rename(names...)() return
        if (names.length == 0 || allSatisfy!(isSomeString, typeof(names)))

to

ref return rename(names...)() if (names.length == 0 || allSatisfy!(isSomeString, typeof(names)))

which leads to errors.
Comment 2 RazvanN 2022-03-24 13:19:13 UTC 
I cannot reproduce this. I just used the header generator on the latest version of druntime's core.atomic and the resulting .di file compiles successfully. There were numerous fixes for the header generator since 2016 so I assume this was fixed.