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

Iain Buclaw <ibuclaw@gdcproject.org> changed:

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

--- Comment #1 from Iain Buclaw <ibuclaw@gdcproject.org> ---
On a mild ranting note, this is the offender in the frontend.

---
// AggregateDeclaration.placeField (aggregate.d)

if ((global.params.is64bit || global.params.isOSX) && memalignsize == 16)
{
}
else if (8 < memalignsize)
    memalignsize = 8;

---

For DMD - you seem to have forgotten that you support 256-bit vectors.

For GDC, you're ignoring the target-specific alignment of fields that we return, instead enforcing that everything must fit your x86 view of ABI.

This piece of code should be moved to Target::fieldalign where it deservedly belongs.

--

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

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

https://github.com/dlang/dmd/commit/10eba509d6a5dfbe509fa1ffe7f4c3962a58a490 fix Issue 17237 - Wrong alignment for 256-bit vectors.

https://github.com/dlang/dmd/commit/25f8bc5bbb2e843fb3ab381d13f4ce919eae0958 Merge pull request #6582 from ibuclaw/issue17237

fix Issue 17237 - Wrong alignment for 256-bit vectors.

--

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

github-bugzilla@puremagic.com changed:

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

--

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

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

https://github.com/dlang/dmd/commit/10eba509d6a5dfbe509fa1ffe7f4c3962a58a490 fix Issue 17237 - Wrong alignment for 256-bit vectors.

https://github.com/dlang/dmd/commit/25f8bc5bbb2e843fb3ab381d13f4ce919eae0958 Merge pull request #6582 from ibuclaw/issue17237

--

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

Vladimir Panteleev <dlang-bugzilla@thecybershadow.net> changed:

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

--

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

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

https://github.com/dlang/dmd/commit/10eba509d6a5dfbe509fa1ffe7f4c3962a58a490 fix Issue 17237 - Wrong alignment for 256-bit vectors.

https://github.com/dlang/dmd/commit/25f8bc5bbb2e843fb3ab381d13f4ce919eae0958 Merge pull request #6582 from ibuclaw/issue17237

--