diff --git a/gcc/stor-layout.c b/gcc/stor-layout.c index 7d6b1b5ce52..784f131ebb8 100644 --- a/gcc/stor-layout.c +++ b/gcc/stor-layout.c @@ -1926,6 +1926,7 @@ finalize_type_size (tree type) However, where strict alignment is not required, avoid over-aligning structures, since most compilers do not do this alignment. */ + bool tua_cleared_p = false; if (TYPE_MODE (type) != BLKmode && TYPE_MODE (type) != VOIDmode && (STRICT_ALIGNMENT || !AGGREGATE_TYPE_P (type))) @@ -1937,7 +1938,9 @@ finalize_type_size (tree type) if (mode_align >= TYPE_ALIGN (type)) { SET_TYPE_ALIGN (type, mode_align); - TYPE_USER_ALIGN (type) = 0; + /* Remember that we're about to reset this flag. */ + tua_cleared_p = TYPE_USER_ALIGN (type); + TYPE_USER_ALIGN (type) = false; } } @@ -1991,14 +1994,21 @@ finalize_type_size (tree type) /* Copy it into all variants. */ for (variant = TYPE_MAIN_VARIANT (type); - variant != 0; + variant != NULL_TREE; variant = TYPE_NEXT_VARIANT (variant)) { TYPE_SIZE (variant) = size; TYPE_SIZE_UNIT (variant) = size_unit; unsigned valign = align; if (TYPE_USER_ALIGN (variant)) - valign = MAX (valign, TYPE_ALIGN (variant)); + { + valign = MAX (valign, TYPE_ALIGN (variant)); + /* If we reset TYPE_USER_ALIGN on the main variant, we might + need to reset it on the variants too. TYPE_MODE will be set + to MODE in this variant, so we can use that. */ + if (tua_cleared_p && GET_MODE_ALIGNMENT (mode) >= valign) + TYPE_USER_ALIGN (variant) = false; + } else TYPE_USER_ALIGN (variant) = user_align; SET_TYPE_ALIGN (variant, valign); diff --git a/gcc/testsuite/g++.dg/cpp0x/alignas19.C b/gcc/testsuite/g++.dg/cpp0x/alignas19.C new file mode 100644 index 00000000000..892125b54df --- /dev/null +++ b/gcc/testsuite/g++.dg/cpp0x/alignas19.C @@ -0,0 +1,8 @@ +// PR c++/94775 +// { dg-do compile { target c++11 } } +// { dg-additional-options "-mstrict-align" { target { aarch64*-*-* powerpc*-*-linux* powerpc*-*-elf* } } } + +struct alignas(8) S { + S *arr[1]; + void fn () const { (void) arr[0]; } +}; diff --git a/gcc/testsuite/g++.dg/warn/Warray-bounds15.C b/gcc/testsuite/g++.dg/warn/Warray-bounds15.C new file mode 100644 index 00000000000..0a18f637e0e --- /dev/null +++ b/gcc/testsuite/g++.dg/warn/Warray-bounds15.C @@ -0,0 +1,40 @@ +// PR c++/94775 +// { dg-do compile { target c++14 } } +// { dg-options "-O2 -Warray-bounds" } + +template using a = int; +template using b = int; +typedef char d; +template using e = int; +template struct h { using i = b>, e>; }; +template using j = typename h::i; +long ab, k, aj; +const d l[]{}; +class m { +public: + m(int); +}; +class n { + void ad() const; + template void o(long) const { + using c __attribute__((aligned(1))) = const ae; + } + long p; + template + auto s(unsigned long, unsigned long, unsigned long, unsigned long) const; + template auto q(unsigned long, unsigned long) const; +}; +template +auto n::s(unsigned long, unsigned long, unsigned long, unsigned long t) const { + o(p); + return t; +} +template auto n::q(unsigned long p1, unsigned long p2) const { + using r = j<4, false>; + using ai = j<4, g>; + return s(ab, k, p1, p2); +} +void n::ad() const { + long f(l[aj]); // { dg-warning "outside array bounds" } + m(q(8, f)); +} diff --git a/gcc/tree.c b/gcc/tree.c index f9d57e6d409..430b76168b2 100644 --- a/gcc/tree.c +++ b/gcc/tree.c @@ -6573,7 +6573,8 @@ check_base_type (const_tree cand, const_tree base) TYPE_ATTRIBUTES (base))) return false; /* Check alignment. */ - if (TYPE_ALIGN (cand) == TYPE_ALIGN (base)) + if (TYPE_ALIGN (cand) == TYPE_ALIGN (base) + && TYPE_USER_ALIGN (cand) == TYPE_USER_ALIGN (base)) return true; /* Atomic types increase minimal alignment. We must to do so as well or we get duplicated canonical types. See PR88686. */ @@ -6608,6 +6609,7 @@ check_aligned_type (const_tree cand, const_tree base, unsigned int align) && TYPE_CONTEXT (cand) == TYPE_CONTEXT (base) /* Check alignment. */ && TYPE_ALIGN (cand) == align + && TYPE_USER_ALIGN (cand) == TYPE_USER_ALIGN (base) && attribute_list_equal (TYPE_ATTRIBUTES (cand), TYPE_ATTRIBUTES (base)) && check_lang_type (cand, base));