[dmd-beta] dmd 1.070 and 2.055 beta

[Brad Roberts]

On Tuesday, September 06, 2011 11:35:00 PM, Rainer Schuetze wrote:
> On 07.09.2011 02:21, Walter Bright wrote:
>>
>>
>> On 9/6/2011 3:07 PM, Brad Roberts wrote:
>>>
>>> Damn.. that one would be my changes then.  If the choice is to revert then
>>> the fix to iasm.d that depends on the changes to backend_init will also
>>> need to be reverted (also re-opening the closed bug associated with it).
>>>
>>> I'm not in a good position to spend much time digging into this right now,
>>> but on quick inspection of the code, in mars.c, try moving the call of
>>> backend_init to after the module creation loop, before the try.
>>
>> I've checked in a patch for that. Rainer, want to give it a try?
>>
>> https://github.com/D-Programming-Language/dmd/commit/b60b44fda4be65df75149d4c772a84698a69a544
>> _______________________________________________
> 
> Sorry, I could not answer while I was asleep. The new beta works fine.
> 
> I'm not sure about Brads concerns, but the new position of the c
all to backend_init() is between
> parsing+importing+expanding version/debug conditionals and semantic analysis. This seems fine, but maybe there is
> something depending on backend initialization? backend_init could also move before the parsing...

It's ok where it is I'm pretty sure.  The iasm dependency doesn't kick 
in until semantic analysis.  If there's anything in lexing and parsing 
that interfaces with the backend, I'd love to know about it.