Thread overview
[Issue 15235] inline asm: silent ICE (segfault) in asm_add_exp()
Oct 31, 2015
Cauterite
Aug 19, 2016
Cauterite
Feb 28, 2019
Basile-z
Mar 21
Basile-z
May 19
Dlang Bot
May 23
Dlang Bot
October 22, 2015
https://issues.dlang.org/show_bug.cgi?id=15235

anoneuron@gmail.com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|                            |iasm, ice
                URL|                            |http://dpaste.dzfl.pl/701f2
                   |                            |6f64385

--
October 28, 2015
https://issues.dlang.org/show_bug.cgi?id=15235

--- Comment #1 from anoneuron@gmail.com ---
After some more thorough testing:

void main() {
    asm {
        mov [+], EAX; // syntax error
        mov [-], EAX; // syntax error
        mov [*], EAX; // segfault
        mov [****], EAX; // segfault
        mov [/], EAX; // syntax error
        mov [EBX+], EAX; // compiles (?!)
        mov [EBX-], EAX; // segfault
        mov [EBX+*], EAX; // segfault
        mov [EBX*], EAX; // segfault
        mov [EBX*EBX*], EAX; // segfault
        mov [*EBX], EAX; // segfault
        mov [/EBX], EAX; // syntax error

        mov [EBX+EBX+EBX], EAX; // prints the same error message 20 times
    };
};

--
October 28, 2015
https://issues.dlang.org/show_bug.cgi?id=15235

anoneuron@gmail.com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Assignee|nobody@puremagic.com        |anoneuron@gmail.com

--
October 31, 2015
https://issues.dlang.org/show_bug.cgi?id=15235

Cauterite <cauterite@gmail.com> changed:

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

--- Comment #2 from Cauterite <cauterite@gmail.com> ---
https://github.com/D-Programming-Language/dmd/pull/5248

--
August 19, 2016
https://issues.dlang.org/show_bug.cgi?id=15235

--- Comment #3 from Cauterite <cauterite@gmail.com> ---
Previous pull request closed; this will now be fixed by the patch for issue
15257:
https://github.com/dlang/dmd/pull/6068

--
February 28, 2019
https://issues.dlang.org/show_bug.cgi?id=15235

Basile-z <b2.temp@gmx.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|ice, pull                   |diagnostic
                 CC|                            |b2.temp@gmx.com

--- Comment #4 from Basile-z <b2.temp@gmx.com> ---
Downgraded to diagnostic. Nowadays just

void main()
{
    asm {
        mov [EBX+EBX+EBX], EAX; // prints the same error message 20 times
    }
}

is still there and all the segfaults are not happening anymore

--
March 21
https://issues.dlang.org/show_bug.cgi?id=15235

Basile-z <b2.temp@gmx.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|b2.temp@gmx.com             |

--
May 19
https://issues.dlang.org/show_bug.cgi?id=15235

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

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

--- Comment #5 from Dlang Bot <dlang-bot@dlang.rocks> ---
@benjones created dlang/dmd pull request #11166 "Fix issue 15235" fixing this issue:

- fix issue 15235.  Return after spotting an asm syntax error rather than goto-ing it 10 times

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

--
May 23
https://issues.dlang.org/show_bug.cgi?id=15235

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

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

--- Comment #6 from Dlang Bot <dlang-bot@dlang.rocks> ---
dlang/dmd pull request #11166 "Fix issue 15235: inline asm: silent ICE
(segfault) in asm_add_exp()" was merged into master:

- 185fd7155ea1dc7b22ed4b995ae12208d86f62e1 by Ben Jones:
  fix issue 15235.  Return after spotting an asm syntax error rather than
goto-ing it 10 times

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

--