Thread overview
[Issue 24153] Inliner breaks -betterC by requiring TypeInfo
Sep 23, 2023
ryuukk_
Sep 23, 2023
ryuukk_
Sep 23, 2023
ryuukk_
Nov 23, 2023
RazvanN
Dec 31, 2023
ryuukk_
Nov 02
Dlang Bot
6 days ago
Dlang Bot
September 23, 2023
https://issues.dlang.org/show_bug.cgi?id=24153

ryuukk_ <ryuukk.dev@gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|                            |betterC

--
September 23, 2023
https://issues.dlang.org/show_bug.cgi?id=24153

--- Comment #1 from ryuukk_ <ryuukk.dev@gmail.com> ---
I sublimted a PR that fixes it, but apparently is not valid? https://github.com/dlang/dmd/pull/15627

--
September 23, 2023
https://issues.dlang.org/show_bug.cgi?id=24153

--- Comment #2 from ryuukk_ <ryuukk.dev@gmail.com> ---
Found the cause:

Here reduced test:

```D
struct InvBoneBindInfo
{
}


struct Test(Value)
{
    void test()
    {
        auto t = Value.init;
    }
}

extern(C) void main()
{
    Test!(InvBoneBindInfo[32]) test;
    test.test();
}

```

The problem is ``Value.init``, because value is a static array

--
November 23, 2023
https://issues.dlang.org/show_bug.cgi?id=24153

RazvanN <razvan.nitu1305@gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
                 CC|                            |razvan.nitu1305@gmail.com
         Resolution|---                         |FIXED

--- Comment #3 from RazvanN <razvan.nitu1305@gmail.com> ---
Closing this as the submitter cannot reproduce the issue anymore: https://github.com/dlang/dmd/pull/15627#issuecomment-1805526316

--
December 31, 2023
https://issues.dlang.org/show_bug.cgi?id=24153

ryuukk_ <ryuukk.dev@gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|RESOLVED                    |REOPENED
         Resolution|FIXED                       |---

--- Comment #4 from ryuukk_ <ryuukk.dev@gmail.com> ---
The issue still happen with the latest master, i probably forgot the ``-inline`` when i said that i couldn't reproduce it


Reduced test above still trigger the issue

--
November 02
https://issues.dlang.org/show_bug.cgi?id=24153

Dlang Bot <dlang-bot@dlang.rocks> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|                            |pull

--- Comment #5 from Dlang Bot <dlang-bot@dlang.rocks> ---
@bangbangsheshotmedown created dlang/dmd pull request #17049 "Fix bugzilla issue 24153 - Inliner breaks -betterC by requiring TypeInfo" fixing this issue:

- Fix bugzilla issue 24153 - Inliner breaks -betterC by requiring TypeInfo

https://github.com/dlang/dmd/pull/17049

--
6 days ago
https://issues.dlang.org/show_bug.cgi?id=24153

Dlang Bot <dlang-bot@dlang.rocks> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|REOPENED                    |RESOLVED
         Resolution|---                         |FIXED

--- Comment #6 from Dlang Bot <dlang-bot@dlang.rocks> ---
dlang/dmd pull request #17049 "Fix bugzilla issue 24153 - Inliner breaks -betterC by requiring TypeInfo" was merged into master:

- 3013c15815a2ba606d511b3678fdd9ea1fdb66b5 by bangbangsheshotmedown:
  Fix bugzilla issue 24153 - Inliner breaks -betterC by requiring TypeInfo

https://github.com/dlang/dmd/pull/17049

--