[libcxxabi] r302764 - Revert "[libcxxabi] Align unwindHeader on a double-word boundary."

[Akira Hatanaka via cfe-commits]

Author: ahatanak
Date: Thu May 11 01:17:49 2017
New Revision: 302764

URL: http://llvm.org/viewvc/llvm-project?rev=302764&view=rev
Log:
Revert "[libcxxabi] Align unwindHeader on a double-word boundary."

This reverts commit r302763.

The commit caused bot failures.

Removed:
    libcxxabi/trunk/test/exception_object_alignment.sh.cpp
Modified:
    libcxxabi/trunk/src/cxa_exception.hpp

Modified: libcxxabi/trunk/src/cxa_exception.hpp
URL: http://llvm.org/viewvc/llvm-project/libcxxabi/trunk/src/cxa_exception.hpp?rev=302764&r1=302763&r2=302764&view=diff
==============================================================================
--- libcxxabi/trunk/src/cxa_exception.hpp (original)
+++ libcxxabi/trunk/src/cxa_exception.hpp Thu May 11 01:17:49 2017
@@ -61,13 +61,7 @@ struct _LIBCXXABI_HIDDEN __cxa_exception
     size_t referenceCount;
 #endif
 
-    // This field is annotated with attribute aligned so that the exception
-    // object following the field is sufficiently aligned and there is no
-    // gap between the field and the exception object. r276215 made a change to
-    // annotate _Unwind_Exception with __attribute__((aligned)), but we cannot
-    // incorporate the fix on Darwin since it is an ABI-breaking change, which
-    // is why we need the attribute on this field.
-    _Unwind_Exception unwindHeader __attribute__((aligned));
+    _Unwind_Exception unwindHeader;
 };
 
 // http://sourcery.mentor.com/archives/cxx-abi-dev/msg01924.html
@@ -102,9 +96,7 @@ struct _LIBCXXABI_HIDDEN __cxa_dependent
     void* primaryException;
 #endif
 
-    // See the comment in __cxa_exception as to why this field has attribute
-    // aligned.
-    _Unwind_Exception unwindHeader __attribute__((aligned));
+    _Unwind_Exception unwindHeader;
 };
 
 struct _LIBCXXABI_HIDDEN __cxa_eh_globals {

Removed: libcxxabi/trunk/test/exception_object_alignment.sh.cpp
URL: http://llvm.org/viewvc/llvm-project/libcxxabi/trunk/test/exception_object_alignment.sh.cpp?rev=302763&view=auto
==============================================================================
--- libcxxabi/trunk/test/exception_object_alignment.sh.cpp (original)
+++ libcxxabi/trunk/test/exception_object_alignment.sh.cpp (removed)
@@ -1,46 +0,0 @@
-//===---------------- exception_object_alignment.sh.cpp -------------------===//
-//
-//                     The LLVM Compiler Infrastructure
-//
-// This file is dual licensed under the MIT and the University of Illinois Open
-// Source Licenses. See LICENSE.TXT for details.
-//
-//===----------------------------------------------------------------------===//
-
-// UNSUPPORTED: libcxxabi-no-exceptions
-
-// RUN: %cxx %flags %compile_flags -O1 %s -o %t.exe
-// RUN: %exec %t.exe
-
-// This test used to segfault on Darwin because field unwindHeader of struct
-// __cxa_exception was not 16B aligned.
-
-namespace {
-
-struct S {
-  int a;
-  int __attribute__((aligned(16))) b;
-};
-
-class base1 {
-protected:
-  virtual ~base1() throw() {}
-};
-
-class derived: public base1 {
-public:
-  derived() : member() {}
-private:
-  S member;
-};
-
-}
-
-int main() {
-  try {
-    throw derived();
-  }
-  catch(...) {
-  }
-  return 0;
-}