]> git.ipfire.org Git - thirdparty/gcc.git/commit
vec: Fix bootstrap on i686-linux, 32-bit darwin and AIX
authorJakub Jelinek <jakub@redhat.com>
Fri, 14 Aug 2020 07:29:47 +0000 (09:29 +0200)
committerGiuliano Belinassi <giuliano.belinassi@usp.br>
Mon, 17 Aug 2020 18:08:12 +0000 (15:08 -0300)
commit70a973ad305aea4c67a2222a95604431541ea079
treea1f9ced8d3d206eb309172cf5b62d0350cba54c2
parentc05027e5c96fb719639e8ef67b0b57907c136070
vec: Fix bootstrap on i686-linux, 32-bit darwin and AIX

As mentioned earlier, embedded_size is broken on vecs of long long, double
etc. on some platforms, which breaks bootstrap.
E.g. on i686-linux, the problem is mostly with older GCC versions being used
as stage1 compiler (GCC 4.8 to 7.* in particular), because alignas (long long)
makes U 64-bit aligned, while when long long m_vecdata[1]; is in vec, it is
only 32-bit aligned.  We've tried various ways and the following one seems
to work, use the old way (offsetof (vec, m_vecdata)) for non-class types
as well as standard layout class types, i.e. whenever offsetof is guaranteed
to work, and for others use the new day (in that case we don't run into
problems with long long or other scalar types and for the structure layout
there is just a struct with a given alignment.

2020-08-14  Jakub Jelinek  <jakub@redhat.com>
    Jonathan Wakely  <jwakely@redhat.com>

* system.h: Include type_traits.
* vec.h (vec<T, A, vl_embed>::embedded_size): Use offsetof and asserts
on vec_stdlayout, which is conditionally a vec (for standard layout T)
and otherwise vec_embedded.

Co-Authored-By: Jonathan Wakely <jwakely@redhat.com>
gcc/system.h
gcc/vec.h