Menu
Search

Forums

Forum Index

Thread overview
[dmd-internals] [D-Programming-Language/dmd] 767708: 6984 CTFE generates a torrent of spurious errors, ...
Dec 21, 2011
Don Clugston
Dec 21, 2011
Walter Bright
Dec 22, 2011
Walter Bright
December 21, 2011
Gravatar of noreply at github.comPosted by noreply at github.comReply
noreply at github.com
Gravatar of noreply at github.com


Reply
  Branch: refs/heads/master
  Home:   https://github.com/D-Programming-Language/dmd

  Commit: 767708299a408b4a234ee9ad4f45b0e6e0c21e6d
      https://github.com/D-Programming-Language/dmd/commit/767708299a408b4a234ee9ad4f45b0e6e0c21e6d
  Author: Don Clugston <dclugston at googlemail.com>
  Date:   2011-12-21 (Wed, 21 Dec 2011)

  Changed paths:
    M src/declaration.h
  M src/func.c
  M src/interpret.c

  Log Message:
  -----------
  6984 CTFE generates a torrent of spurious errors, if any previous error

Instead of preventing CTFE when any error occured, only give up if an error occured while running semantic3 on the function being called. This brings us a lot closer to the ultimate goal of one error message per bug.


  Commit: 1042789d8d77c0f1b2c8b48544e60aaffd1e8348
      https://github.com/D-Programming-Language/dmd/commit/1042789d8d77c0f1b2c8b48544e60aaffd1e8348
  Author: Don Clugston <dclugston at googlemail.com>
  Date:   2011-12-21 (Wed, 21 Dec 2011)

  Changed paths:
    M src/interpret.c

  Log Message:
  -----------
  7143 [CTFE] cannot compare class references with "is"

== doesn't work either, but for different reasons.


  Commit: 312a6c95a8400dcda99660d0605b666a31ad3ac5
      https://github.com/D-Programming-Language/dmd/commit/312a6c95a8400dcda99660d0605b666a31ad3ac5
  Author: Don Clugston <dclugston at googlemail.com>
  Date:   2011-12-21 (Wed, 21 Dec 2011)

  Changed paths:
    M test/compilable/interpret3.d

  Log Message:
  -----------
  Test case for 7143


  Commit: 879a20856a957fb7d066dc2ca7dbb84436cedabe
      https://github.com/D-Programming-Language/dmd/commit/879a20856a957fb7d066dc2ca7dbb84436cedabe
  Author: Don Clugston <dclugston at googlemail.com>
  Date:   2011-12-21 (Wed, 21 Dec 2011)

  Changed paths:
    M src/interpret.c

  Log Message:
  -----------
  7144 [CTFE] base class does not call overridden members

We need to do the vtable lookup, even if called directly from 'this'.


  Commit: e59b8806691579f8253993ee62fa0c10f9656552
      https://github.com/D-Programming-Language/dmd/commit/e59b8806691579f8253993ee62fa0c10f9656552
  Author: Don Clugston <dclugston at googlemail.com>
  Date:   2011-12-21 (Wed, 21 Dec 2011)

  Changed paths:
    M test/compilable/interpret3.d

  Log Message:
  -----------
  Test case for 7147


  Commit: feac8ea4c4fcc43b3a370fc90d1e2155c6ce0376
      https://github.com/D-Programming-Language/dmd/commit/feac8ea4c4fcc43b3a370fc90d1e2155c6ce0376
  Author: Walter Bright <walter at walterbright.com>
  Date:   2011-12-21 (Wed, 21 Dec 2011)

  Changed paths:
    M src/declaration.h
  M src/func.c
  M src/interpret.c
  M test/compilable/interpret3.d

  Log Message:
  -----------
  Merge pull request #573 from donc/ctfe6984

CTFE: 6984 7143 7144


Compare: https://github.com/D-Programming-Language/dmd/compare/641c83f...feac8ea
December 21, 2011
Gravatar of Don ClugstonPosted by Don Clugston
in reply to noreply at github.com		Reply
Don Clugston
Gravatar of Don Clugston
Posted in reply to noreply at github.com


Reply
The autotester is segfaulting on 64 bit Linux, in interpret3.d, but
passing everywhere else.
This platform-specific behaviour is strange because that's compiled
with -c, it doesn't reach the backend.

On 21 December 2011 23:17,  <noreply at github.com> wrote:
> ?Branch: refs/heads/master
> ?Home: ? https://github.com/D-Programming-Language/dmd
>
> ?Commit: 767708299a408b4a234ee9ad4f45b0e6e0c21e6d
> ? ? ?https://github.com/D-Programming-Language/dmd/commit/767708299a408b4a234ee9ad4f45b0e6e0c21e6d
> ?Author: Don Clugston <dclugston at googlemail.com>
> ?Date: ? 2011-12-21 (Wed, 21 Dec 2011)
>
> ?Changed paths:
> ? ?M src/declaration.h
> ?M src/func.c
> ?M src/interpret.c
>
> ?Log Message:
> ?-----------
> ?6984 CTFE generates a torrent of spurious errors, if any previous error
>
> Instead of preventing CTFE when any error occured, only give up if an error occured while running semantic3 on the function being called. This brings us a lot closer to the ultimate goal of one error message per bug.
>
>
> ?Commit: 1042789d8d77c0f1b2c8b48544e60aaffd1e8348
> ? ? ?https://github.com/D-Programming-Language/dmd/commit/1042789d8d77c0f1b2c8b48544e60aaffd1e8348
> ?Author: Don Clugston <dclugston at googlemail.com>
> ?Date: ? 2011-12-21 (Wed, 21 Dec 2011)
>
> ?Changed paths:
> ? ?M src/interpret.c
>
> ?Log Message:
> ?-----------
> ?7143 [CTFE] cannot compare class references with "is"
>
> == doesn't work either, but for different reasons.
>
>
> ?Commit: 312a6c95a8400dcda99660d0605b666a31ad3ac5
> ? ? ?https://github.com/D-Programming-Language/dmd/commit/312a6c95a8400dcda99660d0605b666a31ad3ac5
> ?Author: Don Clugston <dclugston at googlemail.com>
> ?Date: ? 2011-12-21 (Wed, 21 Dec 2011)
>
> ?Changed paths:
> ? ?M test/compilable/interpret3.d
>
> ?Log Message:
> ?-----------
> ?Test case for 7143
>
>
> ?Commit: 879a20856a957fb7d066dc2ca7dbb84436cedabe
> ? ? ?https://github.com/D-Programming-Language/dmd/commit/879a20856a957fb7d066dc2ca7dbb84436cedabe
> ?Author: Don Clugston <dclugston at googlemail.com>
> ?Date: ? 2011-12-21 (Wed, 21 Dec 2011)
>
> ?Changed paths:
> ? ?M src/interpret.c
>
> ?Log Message:
> ?-----------
> ?7144 [CTFE] base class does not call overridden members
>
> We need to do the vtable lookup, even if called directly from 'this'.
>
>
> ?Commit: e59b8806691579f8253993ee62fa0c10f9656552
> ? ? ?https://github.com/D-Programming-Language/dmd/commit/e59b8806691579f8253993ee62fa0c10f9656552
> ?Author: Don Clugston <dclugston at googlemail.com>
> ?Date: ? 2011-12-21 (Wed, 21 Dec 2011)
>
> ?Changed paths:
> ? ?M test/compilable/interpret3.d
>
> ?Log Message:
> ?-----------
> ?Test case for 7147
>
>
> ?Commit: feac8ea4c4fcc43b3a370fc90d1e2155c6ce0376
> ? ? ?https://github.com/D-Programming-Language/dmd/commit/feac8ea4c4fcc43b3a370fc90d1e2155c6ce0376
> ?Author: Walter Bright <walter at walterbright.com>
> ?Date: ? 2011-12-21 (Wed, 21 Dec 2011)
>
> ?Changed paths:
> ? ?M src/declaration.h
> ?M src/func.c
> ?M src/interpret.c
> ?M test/compilable/interpret3.d
>
> ?Log Message:
> ?-----------
> ?Merge pull request #573 from donc/ctfe6984
>
> CTFE: 6984 7143 7144
>
>
> Compare: https://github.com/D-Programming-Language/dmd/compare/641c83f...feac8ea
> _______________________________________________
> dmd-internals mailing list
> dmd-internals at puremagic.com
> http://lists.puremagic.com/mailman/listinfo/dmd-internals
December 21, 2011
Gravatar of Walter BrightPosted by Walter Bright
in reply to Don Clugston		Reply
Walter Bright
Gravatar of Walter Bright
Posted in reply to Don Clugston


Reply
I'm not seeing a failure on my local 64 bit linux machine.

On 12/21/2011 2:43 PM, Don Clugston wrote:
> The autotester is segfaulting on 64 bit Linux, in interpret3.d, but
> passing everywhere else.
> This platform-specific behaviour is strange because that's compiled
> with -c, it doesn't reach the backend.
>
> On 21 December 2011 23:17,<noreply at github.com>  wrote:
>>   Branch: refs/heads/master
>>   Home:   https://github.com/D-Programming-Language/dmd
>>
>>   Commit: 767708299a408b4a234ee9ad4f45b0e6e0c21e6d
>>       https://github.com/D-Programming-Language/dmd/commit/767708299a408b4a234ee9ad4f45b0e6e0c21e6d
>>   Author: Don Clugston<dclugston at googlemail.com>
>>   Date:   2011-12-21 (Wed, 21 Dec 2011)
>>
>>   Changed paths:
>>     M src/declaration.h
>>   M src/func.c
>>   M src/interpret.c
>>
>>   Log Message:
>>   -----------
>>   6984 CTFE generates a torrent of spurious errors, if any previous error
December 21, 2011
Gravatar of Walter BrightPosted by Walter Bright
in reply to Don Clugston		Reply
Walter Bright
Gravatar of Walter Bright
Posted in reply to Don Clugston


Reply
Ok, I'm getting the same failure. Only for the 64 bit build of dmd.

On 12/21/2011 2:43 PM, Don Clugston wrote:
> The autotester is segfaulting on 64 bit Linux, in interpret3.d, but
> passing everywhere else.
> This platform-specific behaviour is strange because that's compiled
> with -c, it doesn't reach the backend.
>
> On 21 December 2011 23:17,<noreply at github.com>  wrote:
>>   Branch: refs/heads/master
>>   Home:   https://github.com/D-Programming-Language/dmd
>>
>>   Commit: 767708299a408b4a234ee9ad4f45b0e6e0c21e6d
>>       https://github.com/D-Programming-Language/dmd/commit/767708299a408b4a234ee9ad4f45b0e6e0c21e6d
>>   Author: Don Clugston<dclugston at googlemail.com>
>>   Date:   2011-12-21 (Wed, 21 Dec 2011)
>>
>>   Changed paths:
>>     M src/declaration.h
>>   M src/func.c
>>   M src/interpret.c
>>
>>   Log Message:
>>   -----------
>>   6984 CTFE generates a torrent of spurious errors, if any previous error
>>
>> Instead of preventing CTFE when any error occured, only give up if an error occured while running semantic3 on the function being called. This brings us a lot closer to the ultimate goal of one error message per bug.
>>
>>
>>   Commit: 1042789d8d77c0f1b2c8b48544e60aaffd1e8348
>>       https://github.com/D-Programming-Language/dmd/commit/1042789d8d77c0f1b2c8b48544e60aaffd1e8348
>>   Author: Don Clugston<dclugston at googlemail.com>
>>   Date:   2011-12-21 (Wed, 21 Dec 2011)
>>
>>   Changed paths:
>>     M src/interpret.c
>>
>>   Log Message:
>>   -----------
>>   7143 [CTFE] cannot compare class references with "is"
>>
>> == doesn't work either, but for different reasons.
>>
>>
>>   Commit: 312a6c95a8400dcda99660d0605b666a31ad3ac5
>>       https://github.com/D-Programming-Language/dmd/commit/312a6c95a8400dcda99660d0605b666a31ad3ac5
>>   Author: Don Clugston<dclugston at googlemail.com>
>>   Date:   2011-12-21 (Wed, 21 Dec 2011)
>>
>>   Changed paths:
>>     M test/compilable/interpret3.d
>>
>>   Log Message:
>>   -----------
>>   Test case for 7143
>>
>>
>>   Commit: 879a20856a957fb7d066dc2ca7dbb84436cedabe
>>       https://github.com/D-Programming-Language/dmd/commit/879a20856a957fb7d066dc2ca7dbb84436cedabe
>>   Author: Don Clugston<dclugston at googlemail.com>
>>   Date:   2011-12-21 (Wed, 21 Dec 2011)
>>
>>   Changed paths:
>>     M src/interpret.c
>>
>>   Log Message:
>>   -----------
>>   7144 [CTFE] base class does not call overridden members
>>
>> We need to do the vtable lookup, even if called directly from 'this'.
>>
>>
>>   Commit: e59b8806691579f8253993ee62fa0c10f9656552
>>       https://github.com/D-Programming-Language/dmd/commit/e59b8806691579f8253993ee62fa0c10f9656552
>>   Author: Don Clugston<dclugston at googlemail.com>
>>   Date:   2011-12-21 (Wed, 21 Dec 2011)
>>
>>   Changed paths:
>>     M test/compilable/interpret3.d
>>
>>   Log Message:
>>   -----------
>>   Test case for 7147
>>
>>
>>   Commit: feac8ea4c4fcc43b3a370fc90d1e2155c6ce0376
>>       https://github.com/D-Programming-Language/dmd/commit/feac8ea4c4fcc43b3a370fc90d1e2155c6ce0376
>>   Author: Walter Bright<walter at walterbright.com>
>>   Date:   2011-12-21 (Wed, 21 Dec 2011)
>>
>>   Changed paths:
>>     M src/declaration.h
>>   M src/func.c
>>   M src/interpret.c
>>   M test/compilable/interpret3.d
>>
>>   Log Message:
>>   -----------
>>   Merge pull request #573 from donc/ctfe6984
>>
>> CTFE: 6984 7143 7144
>>
>>
>> Compare: https://github.com/D-Programming-Language/dmd/compare/641c83f...feac8ea
>> _______________________________________________
>> dmd-internals mailing list
>> dmd-internals at puremagic.com
>> http://lists.puremagic.com/mailman/listinfo/dmd-internals
> _______________________________________________
> dmd-internals mailing list
> dmd-internals at puremagic.com
> http://lists.puremagic.com/mailman/listinfo/dmd-internals
>
>