https://issues.dlang.org/show_bug.cgi?id=18734

Ketmar Dark <ketmar@ketmar.no-ip.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |ketmar@ketmar.no-ip.org

--

https://issues.dlang.org/show_bug.cgi?id=18734

ag0aep6g <ag0aep6g@gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Blocks|                            |18750


Referenced Issues:

https://issues.dlang.org/show_bug.cgi?id=18750
[Issue 18750] [Tracker] everything wrong with code generation for bt
instruction
--

https://issues.dlang.org/show_bug.cgi?id=18734

uplink.coder@googlemail.com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |uplink.coder@googlemail.com

--- Comment #1 from uplink.coder@googlemail.com ---
DMD does not recognize bodies only function signatures.
So change it all you like :) it's only going to be used on systems where the
intrinsic can't work.

--

https://issues.dlang.org/show_bug.cgi?id=18734

--- Comment #2 from ag0aep6g <ag0aep6g@gmail.com> ---
(In reply to uplink.coder from comment #1)
> DMD does not recognize bodies only function signatures.
> So change it all you like :) it's only going to be used on systems where the
> intrinsic can't work.

core.bitop.bt is not an intrinsic. The expression in the body is recognized by the optimizer.

--

https://issues.dlang.org/show_bug.cgi?id=18734

Walter Bright <bugzilla@digitalmars.com> changed:

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

--- Comment #3 from Walter Bright <bugzilla@digitalmars.com> ---
Huh, you are right. This:

https://www.felixcloutier.com/x86/bt

doesn't mention it's signed, but the AMD reference says it is.

--

https://issues.dlang.org/show_bug.cgi?id=18734

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

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

--- Comment #4 from Dlang Bot <dlang-bot@dlang.rocks> ---
@WalterBright created dlang/druntime pull request #3141 "fix Issue 18734 - bitnum parameter of core.bitop.bt should be signed" fixing this issue:

- fix Issue 18734 - bitnum parameter of core.bitop.bt should be signed

https://github.com/dlang/druntime/pull/3141

--

https://issues.dlang.org/show_bug.cgi?id=18734

--- Comment #5 from Dlang Bot <dlang-bot@dlang.rocks> ---
@WalterBright created dlang/dmd pull request #11306 "fix Issue 18734 - bitnum parameter of core.bitop.bt should be signed" fixing this issue:

- fix Issue 18734 - bitnum parameter of core.bitop.bt should be signed

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

--

https://issues.dlang.org/show_bug.cgi?id=18734

Walter Bright <bugzilla@digitalmars.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|                            |backend

--

https://issues.dlang.org/show_bug.cgi?id=18734

Iain Buclaw <ibuclaw@gdcproject.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Priority|P1                          |P3

--