Thread overview
[dmd-internals] [D-Programming-Language/dmd] 0fa766: fix Issue 8997 - template instances omit symbol th...
Feb 19, 2013
GitHub
Feb 20, 2013
Rainer Schuetze
Feb 20, 2013
kenji hara
Feb 22, 2013
Rainer Schuetze
February 19, 2013
  Branch: refs/heads/master
  Home:   https://github.com/D-Programming-Language/dmd
  Commit: 0fa766638dec58fb369554e7ce35799acf0dca08
      https://github.com/D-Programming-Language/dmd/commit/0fa766638dec58fb369554e7ce35799acf0dca08
  Author: k-hara <k.hara.pg@gmail.com>
  Date:   2013-02-18 (Mon, 18 Feb 2013)

  Changed paths:
    M src/mtype.c
    A test/runnable/imports/test8997a.d
    A test/runnable/test8997.d

  Log Message:
  -----------
  fix Issue 8997 - template instances omit symbol that may be used in other modules


  Commit: 68766a46499d73f4b7c047869a0be25b4db882e0
      https://github.com/D-Programming-Language/dmd/commit/68766a46499d73f4b7c047869a0be25b4db882e0
  Author: k-hara <k.hara.pg@gmail.com>
  Date:   2013-02-18 (Mon, 18 Feb 2013)

  Changed paths:
    M src/mtype.c

  Log Message:
  -----------
  Use sc->module->importedFrom


  Commit: 9154cf1d34252f8fa1b63cf05cc61e3aec213af0
      https://github.com/D-Programming-Language/dmd/commit/9154cf1d34252f8fa1b63cf05cc61e3aec213af0
  Author: Martin Nowak <code@dawg.eu>
  Date:   2013-02-19 (Tue, 19 Feb 2013)

  Changed paths:
    M src/mtype.c
    A test/runnable/imports/test8997a.d
    A test/runnable/test8997.d

  Log Message:
  -----------
  Merge pull request #1667 from 9rnsr/fix8997

Issue 8997 - template instances omit symbol that may be used in other modules


Compare: https://github.com/D-Programming-Language/dmd/compare/d20ac55fb150...9154cf1d3425


February 20, 2013
On 19.02.2013 16:07, GitHub wrote:
>    Branch: refs/heads/master
>    Home:   https://github.com/D-Programming-Language/dmd
>    Commit: 0fa766638dec58fb369554e7ce35799acf0dca08
>        https://github.com/D-Programming-Language/dmd/commit/0fa766638dec58fb369554e7ce35799acf0dca08
>    Author: k-hara <k.hara.pg@gmail.com>
>    Date:   2013-02-18 (Mon, 18 Feb 2013)
>
>    Changed paths:
>      M src/mtype.c
>      A test/runnable/imports/test8997a.d
>      A test/runnable/test8997.d
>
>    Log Message:
>    -----------
>    fix Issue 8997 - template instances omit symbol that may be used in other modules

Sorry for being a bit late to test this, but I still get undefined symbols when RTInfo is generated (as implemented by the precise GC). Maybe semantics need to drill down a little further?

The pull https://github.com/D-Programming-Language/dmd/pull/1313 does not have this problem.

_______________________________________________
dmd-internals mailing list
dmd-internals@puremagic.com
http://lists.puremagic.com/mailman/listinfo/dmd-internals

February 21, 2013
Piull #1667 did not touch RTInfo generation. It was just for internal
AssociativeArray!(K, V) instantiation.
Could you file a issue about it in bugzilla?

Kenji Hara

2013/2/20 Rainer Schuetze <r.sagitario@gmx.de>

> On 19.02.2013 16:07, GitHub wrote:
>
>>    Branch: refs/heads/master
>>    Home:   https://github.com/D-**Programming-Language/dmd<https://github.com/D-Programming-Language/dmd>
>>    Commit: 0fa766638dec58fb369554e7ce3579**9acf0dca08
>>        https://github.com/D-**Programming-Language/dmd/**commit/**
>> 0fa766638dec58fb369554e7ce3579**9acf0dca08<https://github.com/D-Programming-Language/dmd/commit/0fa766638dec58fb369554e7ce35799acf0dca08>
>>    Author: k-hara <k.hara.pg@gmail.com>
>>    Date:   2013-02-18 (Mon, 18 Feb 2013)
>>
>>    Changed paths:
>>      M src/mtype.c
>>      A test/runnable/imports/**test8997a.d
>>      A test/runnable/test8997.d
>>
>>    Log Message:
>>    -----------
>>    fix Issue 8997 - template instances omit symbol that may be used in
>> other modules
>>
>
> Sorry for being a bit late to test this, but I still get undefined symbols when RTInfo is generated (as implemented by the precise GC). Maybe semantics need to drill down a little further?
>
> The pull https://github.com/D-**Programming-Language/dmd/pull/**1313<https://github.com/D-Programming-Language/dmd/pull/1313>does not have this problem.
>
> ______________________________**_________________
> dmd-internals mailing list
> dmd-internals@puremagic.com
> http://lists.puremagic.com/**mailman/listinfo/dmd-internals<http://lists.puremagic.com/mailman/listinfo/dmd-internals>
>


February 22, 2013
On 20.02.2013 17:33, kenji hara wrote:
> Piull #1667 did not touch RTInfo generation. It was just for internal
> AssociativeArray!(K, V) instantiation.
> Could you file a issue about it in bugzilla?
>

After having dustmite thinking about it for the whole day and the better part of the night, it didn't give a good result (probably a bad condition).

I managed to reduce it manually and it boiled down to happen with debug info only and with https://github.com/D-Programming-Language/dmd/pull/398 applied.

_______________________________________________
dmd-internals mailing list
dmd-internals@puremagic.com
http://lists.puremagic.com/mailman/listinfo/dmd-internals