Thread overview
[Issue 23347] ImportC: pragma pack causes asm label to set symbol name to be ignored
Sep 24, 2022
Walter Bright
Sep 24, 2022
Dlang Bot
Sep 26, 2022
Dlang Bot
September 19, 2022
https://issues.dlang.org/show_bug.cgi?id=23347

--- Comment #1 from dave287091@gmail.com ---
(In reply to dave287091 from comment #0)
> Most other symbols mangle to the wrong name even without the asm label.

I meant the right name.

--
September 24, 2022
https://issues.dlang.org/show_bug.cgi?id=23347

Walter Bright <bugzilla@digitalmars.com> changed:

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

--- Comment #2 from Walter Bright <bugzilla@digitalmars.com> ---
A simpler repro:

void fork() asm("plugh");

#pragma pack(push, 4)
void spoon() asm("xyzzy");
#pragma pack(pop);

void test()
{
    fork();  // calls plugh
    spoon(); // calls spoon
}

--
September 24, 2022
https://issues.dlang.org/show_bug.cgi?id=23347

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 #14481 "fix Issue 23347 - ImportC: pragma pack causes asm label to set symbol…" fixing this issue:

- fix Issue 23347 - ImportC: pragma pack causes asm label to set symbol name to be ignored

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

--
September 26, 2022
https://issues.dlang.org/show_bug.cgi?id=23347

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 #14481 "fix Issue 23347 - ImportC: pragma pack causes asm label to set symbol…" was merged into master:

- 98ac0a4a142d09fee8db89af9bea7a31b26401af by Walter Bright:
  fix Issue 23347 - ImportC: pragma pack causes asm label to set symbol name to
be ignored

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

--