Bug 21648 – Speculative recursive reference to struct with semantic error prints "error: unknown": Part 2

Status
REOPENED
Severity
minor
Priority
P3
Component
dmd
Product
D
Version
D2
Platform
x86_64
OS
Linux
Creation time
2021-02-19T10:17:31Z
Last change time
2024-12-13T19:14:48Z
Keywords
pull
Assigned to
No Owner
Creator
FeepingCreature
Moved to GitHub: dmd#19875 →

Comments

Comment #0 by default_357-line — 2021-02-19T10:17:31Z
Very similar to https://issues.dlang.org/show_bug.cgi?id=21646 template bla() { semantics error; } struct Type { enum e = __traits(compiles, Type.init); static if (bla!()) { } }
Comment #1 by dlang-bot — 2021-02-19T10:23:09Z
@FeepingCreature created dlang/dmd pull request #12212 "Fix issue 21648: remove two lines from dsymbolsem.d." fixing this issue: - Fix issue 21648: remove two lines from dsymbolsem.d. I don't know what they do but it fixes the bug. And doesn't break the testsuite, so ¯\_(ツ)_/¯ https://github.com/dlang/dmd/pull/12212
Comment #2 by dlang-bot — 2021-02-19T14:46:40Z
dlang/dmd pull request #12212 "Fix issue 21648: remove two lines from dsymbolsem.d." was merged into master: - 198ff5f2ea3b92ffba99175db6c69dfe038a8aaf by Mathis Beer: Fix issue 21648: remove two lines from dsymbolsem.d. I don't know what they do but it fixes the bug. And doesn't break the testsuite, so ¯\_(ツ)_/¯ https://github.com/dlang/dmd/pull/12212
Comment #3 by bugzilla — 2021-02-20T05:05:42Z
Reopened because randomly deleting compiler code till it works is not a solution.
Comment #4 by default_357-line — 2021-02-20T06:21:08Z
I mean, I'd be secretly wondering once I saw that the testsuite passes if I should just set up a really really longrunning dustmite session between DMD and the testsuite :)
Comment #5 by dlang-bot — 2021-02-22T01:57:22Z
dlang/dmd pull request #12213 "Revert "Fix issue 21648: remove two lines from dsymbolsem.d."" was merged into master: - bed1af60a9e3bc7ba089239b52dcfe8a7b70e8e1 by Walter Bright: Revert "Fix issue 21648: remove two lines from dsymbolsem.d. I don't know what they do but it fixes the bug. And doesn't break the testsuite, so ¯\_(ツ)_/¯" This reverts commit 1ed39162b0e82663d35c731d7437eeb8eadc9431. https://github.com/dlang/dmd/pull/12213
Comment #6 by robert.schadek — 2024-12-13T19:14:48Z
THIS ISSUE HAS BEEN MOVED TO GITHUB https://github.com/dlang/dmd/issues/19875 DO NOT COMMENT HERE ANYMORE, NOBODY WILL SEE IT, THIS ISSUE HAS BEEN MOVED TO GITHUB