Thread overview
[Issue 17399] core.checkedint.addu cannot inline function
May 16, 2017
Vladimir Panteleev
May 16, 2017
Vladimir Panteleev
May 19, 2017
Walter Bright
May 19, 2017
Walter Bright
May 19, 2017
Walter Bright
May 23, 2017
safety0ff.bugz
[Issue 17399] [REG2.071] core.checkedint.addu cannot inline function
Jun 10, 2017
Martin Krejcirik
May 16, 2017
https://issues.dlang.org/show_bug.cgi?id=17399

Vladimir Panteleev <thecybershadow@gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|                            |rejects-valid
                 CC|                            |thecybershadow@gmail.com

--- Comment #1 from Vladimir Panteleev <thecybershadow@gmail.com> ---
(In reply to thomas.bockman from comment #0)
> The above program fails to compile in release mode:

(here "in release mode" means with -inline)

--
May 16, 2017
https://issues.dlang.org/show_bug.cgi?id=17399

--- Comment #2 from Vladimir Panteleev <thecybershadow@gmail.com> ---
Introduced in https://github.com/dlang/dmd/pull/5292

--
May 19, 2017
https://issues.dlang.org/show_bug.cgi?id=17399

Walter Bright <bugzilla@digitalmars.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |bugzilla@digitalmars.com

--- Comment #3 from Walter Bright <bugzilla@digitalmars.com> ---
Self-contained test case:
---
pragma(inline, true)
uint addu(uint x, uint y, ref bool overflow) {
    uint r = x + y;
    if (r < x || r < y)
        overflow = true;
    return r;
}

void foo() {
    size_t a, b;
    bool over;
    addu(a, b, over);
}
---

--
May 19, 2017
https://issues.dlang.org/show_bug.cgi?id=17399

--- Comment #4 from Walter Bright <bugzilla@digitalmars.com> ---
https://github.com/dlang/dmd/pull/6815

--
May 19, 2017
https://issues.dlang.org/show_bug.cgi?id=17399

Walter Bright <bugzilla@digitalmars.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           See Also|                            |https://issues.dlang.org/sh
                   |                            |ow_bug.cgi?id=16652

--
May 23, 2017
https://issues.dlang.org/show_bug.cgi?id=17399

safety0ff.bugz <safety0ff.bugz@gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |safety0ff.bugz@gmail.com

--- Comment #5 from safety0ff.bugz <safety0ff.bugz@gmail.com> ---
Possible duplicate: #16350

--
June 08, 2017
https://issues.dlang.org/show_bug.cgi?id=17399

--- Comment #6 from github-bugzilla@puremagic.com ---
Commits pushed to master at https://github.com/dlang/dmd

https://github.com/dlang/dmd/commit/8ff0a0a185e0a7d0c04812cf916fbbe6615f792c fix Issue 17399 - core.checkedint.addu cannot inline function

https://github.com/dlang/dmd/commit/c18a27f5a11912695f0b7a74177bde149427496b Merge pull request #6815 from WalterBright/fix17399

fix Issue 17399 - core.checkedint.addu cannot inline function merged-on-behalf-of: unknown

--
June 08, 2017
https://issues.dlang.org/show_bug.cgi?id=17399

github-bugzilla@puremagic.com changed:

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

--
June 10, 2017
https://issues.dlang.org/show_bug.cgi?id=17399

Martin Krejcirik <mk@krej.cz> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
            Summary|core.checkedint.addu cannot |[REG2.071]
                   |inline function             |core.checkedint.addu cannot
                   |                            |inline function

--
August 07, 2017
https://issues.dlang.org/show_bug.cgi?id=17399

--- Comment #7 from github-bugzilla@puremagic.com ---
Commits pushed to newCTFE at https://github.com/dlang/dmd

https://github.com/dlang/dmd/commit/8ff0a0a185e0a7d0c04812cf916fbbe6615f792c fix Issue 17399 - core.checkedint.addu cannot inline function

https://github.com/dlang/dmd/commit/c18a27f5a11912695f0b7a74177bde149427496b Merge pull request #6815 from WalterBright/fix17399

--