diff options
author | Marek Polacek <polacek@redhat.com> | 2020-05-04 10:26:43 -0400 |
---|---|---|
committer | Marek Polacek <polacek@redhat.com> | 2020-05-04 10:35:24 -0400 |
commit | 05be85b649173b10d0bf10255eb15275c2dcf509 (patch) | |
tree | 62c616ef6bf58a7e256f7ea98a0107f0b54d9b91 /gcc | |
parent | 367766f40a031ff064857681dc4da3309f0ce57d (diff) | |
download | gcc-05be85b649173b10d0bf10255eb15275c2dcf509.zip gcc-05be85b649173b10d0bf10255eb15275c2dcf509.tar.gz gcc-05be85b649173b10d0bf10255eb15275c2dcf509.tar.bz2 |
Revert "tree: Don't reuse types if TYPE_USER_ALIGN differ [PR94775]"
This reverts commit 6318fe77395fc0dd59419bc4d69cd06ac0158e54.
Revert:
2020-04-30 Marek Polacek <polacek@redhat.com>
PR c++/94775
* tree.c (check_base_type): Return true only if TYPE_USER_ALIGN match.
(check_aligned_type): Check if TYPE_USER_ALIGN match.
Diffstat (limited to 'gcc')
-rw-r--r-- | gcc/ChangeLog | 9 | ||||
-rw-r--r-- | gcc/testsuite/g++.dg/warn/Warray-bounds-10.C | 40 | ||||
-rw-r--r-- | gcc/tree.c | 4 |
3 files changed, 10 insertions, 43 deletions
diff --git a/gcc/ChangeLog b/gcc/ChangeLog index 88a32ac..e6d102b 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,3 +1,12 @@ +2020-05-04 Marek Polacek <polacek@redhat.com> + + Revert: + 2020-04-30 Marek Polacek <polacek@redhat.com> + + PR c++/94775 + * tree.c (check_base_type): Return true only if TYPE_USER_ALIGN match. + (check_aligned_type): Check if TYPE_USER_ALIGN match. + 2020-05-04 Richard Biener <rguenther@suse.de> PR tree-optimization/93891 diff --git a/gcc/testsuite/g++.dg/warn/Warray-bounds-10.C b/gcc/testsuite/g++.dg/warn/Warray-bounds-10.C deleted file mode 100644 index 0a18f63..0000000 --- a/gcc/testsuite/g++.dg/warn/Warray-bounds-10.C +++ /dev/null @@ -1,40 +0,0 @@ -// PR c++/94775 -// { dg-do compile { target c++14 } } -// { dg-options "-O2 -Warray-bounds" } - -template <typename> using a = int; -template <bool, typename, typename> using b = int; -typedef char d; -template <long> using e = int; -template <int f, int q> struct h { using i = b<q, a<e<f>>, e<f>>; }; -template <long f, bool g> using j = typename h<f, g>::i; -long ab, k, aj; -const d l[]{}; -class m { -public: - m(int); -}; -class n { - void ad() const; - template <class ae> void o(long) const { - using c __attribute__((aligned(1))) = const ae; - } - long p; - template <class, class> - auto s(unsigned long, unsigned long, unsigned long, unsigned long) const; - template <bool = false> auto q(unsigned long, unsigned long) const; -}; -template <class, class> -auto n::s(unsigned long, unsigned long, unsigned long, unsigned long t) const { - o<d>(p); - return t; -} -template <bool g> auto n::q(unsigned long p1, unsigned long p2) const { - using r = j<4, false>; - using ai = j<4, g>; - return s<ai, r>(ab, k, p1, p2); -} -void n::ad() const { - long f(l[aj]); // { dg-warning "outside array bounds" } - m(q(8, f)); -} @@ -6493,8 +6493,7 @@ check_base_type (const_tree cand, const_tree base) TYPE_ATTRIBUTES (base))) return false; /* Check alignment. */ - if (TYPE_ALIGN (cand) == TYPE_ALIGN (base) - && TYPE_USER_ALIGN (cand) == TYPE_USER_ALIGN (base)) + if (TYPE_ALIGN (cand) == TYPE_ALIGN (base)) return true; /* Atomic types increase minimal alignment. We must to do so as well or we get duplicated canonical types. See PR88686. */ @@ -6529,7 +6528,6 @@ 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)); |