← Back to index | Original Bugzilla link

Bug 21378 – `pragma(inline)` should allow not only a bool literal but any CTFE expression that converts to bool

Status: RESOLVED
Resolution: FIXED
Severity: enhancement
Priority: P1
Component: dmd
Product: D
Version: D2
Platform: All
OS: All
Creation time: 2020-11-12T09:02:44Z
Last change time: 2020-11-17T05:59:45Z
Assigned to: No Owner
Creator: Basile-z

Comments

Comment #0 by b2.temp — 2020-11-12T09:02:44Z

details in NG discussion : https://forum.dlang.org/post/[email protected]

Comment #1 by dlang-bot — 2020-11-13T00:31:43Z

@thewilsonator created dlang/dlang.org pull request #2879 "spec change for issue 21378 - Allow `pragma(inline, knownAtCTFE);`" mentioning this issue: - spec change for issue 21378 - Allow `pragma(inline, knownAtCTFE);` https://github.com/dlang/dlang.org/pull/2879

Comment #2 by dlang-bot — 2020-11-14T05:03:07Z

@BorisCarvajal created dlang/dmd pull request #11967 "Allow pragma(inline, knownAtCTFE); version 2" mentioning this issue: - Issue 21378 - `pragma(inline)` should allow not only a bool literal but any CTFE expression that converts to bool https://github.com/dlang/dmd/pull/11967

Comment #3 by dlang-bot — 2020-11-15T13:52:55Z

dlang/dmd pull request #11967 "Allow pragma(inline, knownAtCTFE); version 2" was merged into master: - 3589629ef5502f26b0af972e71f48e8867b28329 by Boris Carvajal: Issue 21378 - `pragma(inline)` should allow not only a bool literal but any CTFE expression that converts to bool https://github.com/dlang/dmd/pull/11967

Comment #4 by petar.p.kirov — 2020-11-17T05:56:30Z

Marking as resolved/fixed, as pull request dmd#11967 is now merged in master.

Comment #5 by dlang-bot — 2020-11-17T05:59:45Z

dlang/dlang.org pull request #2879 "spec change for issue 21378 - Allow `pragma(inline, knownAtCTFE);`" was merged into master: - a40e5e7f40dd00dd5f3ef05432ccb580c4333fcc by Nicholas Lindsay Wilson: spec change for issue 21378 - Allow `pragma(inline, knownAtCTFE);` https://github.com/dlang/dlang.org/pull/2879