Thread overview
[Issue 21325] Flags not set for ?: evaluation with floating point operands
[Issue 21325] Wrong code with -release -inline -O
Oct 18, 2020
Iain Buclaw
Nov 09, 2020
Walter Bright
Nov 09, 2020
Dlang Bot
Nov 15, 2020
Dlang Bot
October 18, 2020
https://issues.dlang.org/show_bug.cgi?id=21325

thomas.bockman@gmail.com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Severity|critical                    |regression

--- Comment #1 from thomas.bockman@gmail.com ---
My example fails on 2.094.1 beta, but works on 2.093 so I am marking this as a regression.

--
October 18, 2020
https://issues.dlang.org/show_bug.cgi?id=21325

thomas.bockman@gmail.com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|                            |backend, wrong-code

--
October 18, 2020
https://issues.dlang.org/show_bug.cgi?id=21325

Iain Buclaw <ibuclaw@gdcproject.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |ibuclaw@gdcproject.org

--- Comment #2 from Iain Buclaw <ibuclaw@gdcproject.org> ---
-release is irrelevant in the issue, replacing `return 1` with `assert(0)` triggers with just -O -inline.

Regression was caused by this commit https://github.com/dlang/dmd/pull/11682

--
November 09, 2020
https://issues.dlang.org/show_bug.cgi?id=21325

Walter Bright <bugzilla@digitalmars.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |bugzilla@digitalmars.com
            Summary|Wrong code with -release    |Flags not set for ?:
                   |-inline -O                  |evaluation with floating
                   |                            |point operands

--
November 09, 2020
https://issues.dlang.org/show_bug.cgi?id=21325

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

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

--- Comment #3 from Dlang Bot <dlang-bot@dlang.rocks> ---
@WalterBright created dlang/dmd pull request #11939 "fix Issue 21325 - Flags not set for ?: evaluation with floating point…" fixing this issue:

- fix Issue 21325 - Flags not set for ?: evaluation with floating point operands

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

--
November 15, 2020
https://issues.dlang.org/show_bug.cgi?id=21325

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

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

--- Comment #4 from Dlang Bot <dlang-bot@dlang.rocks> ---
dlang/dmd pull request #11939 "fix Issue 21325 - Flags not set for ?: evaluation with floating point…" was merged into master:

- aea1dcb1c2205807456f7cb24d5e1372019e11c1 by Walter Bright:
  fix Issue 21325 - Flags not set for ?: evaluation with floating point
operands

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

--
December 21, 2020
https://issues.dlang.org/show_bug.cgi?id=21325

--- Comment #5 from thomas.bockman@gmail.com ---
I can confirm that this works for me now with DMD Beta 2.095.0-beta.1, including in my original, pre-reduction program. Thanks!

--