Request for Review: DI Generation Improvements

Tyro[17] nospam at home.com
Mon May 14 17:20:10 PDT 2012


On Tuesday, 15 May 2012 at 00:02:23 UTC, Alex Rønne Petersen 
wrote:
> On 15-05-2012 01:50, Adam Wilson wrote:
>> I am formally requesting review of the two pull requests that 
>> make up my
>> DI Generation Improvements patch.
>>
>> The first pull is against the DRuntime and modifies the 
>> makefiles such
>> some files in the DRT are not generated as DI files but copied 
>> from the D
>> source files directly into the DI file. This solves a problem 
>> where
>> certain portions of Phobos the CTFE into the DRT cause a build 
>> failure
>> when using the new DI generation. The files that are copied 
>> instead of
>> generated are all of core and core.stdc, EXCEPT core.atomic, 
>> core.cpuid,
>> core.demangle, and core.thread.
>>
[snip]
>
> I would suggest excluding core.[atomic,cpuid,demangle] from DI 
> generation, too, and just copy them. I don't see any reason to 
> do DI generation for them.

Reread the last line of the paragraph starting with "The first 
pull". Your concern is already addressed.


More information about the Digitalmars-d mailing list