Drop std::max_align_t hack present for GCC 4.8
authorChristopher Dykes <cdykes@fb.com>
Fri, 30 Jun 2017 23:20:37 +0000 (16:20 -0700)
committerFacebook Github Bot <facebook-github-bot@users.noreply.github.com>
Fri, 30 Jun 2017 23:22:49 +0000 (16:22 -0700)
Summary: Folly no longer supports GCC 4.8, so this is no longer needed.

Reviewed By: yfeldblum

Differential Revision: D5354542

fbshipit-source-id: bd9a2732f22f2176f64f92f0af6f7d59dd59a8d9

folly/Portability.h

index fce337606540d975f4d194048f30a62424b185c8..053b5b1632b57004db715d80be899671c24830a4 100644 (file)
@@ -179,13 +179,6 @@ constexpr bool kIsSanitizeThread = false;
 #define FOLLY_GCC_DISABLE_NEW_SHADOW_WARNINGS /* empty */
 #endif
 
-#if defined(__GNUC__) && !defined(__APPLE__) && !__GNUC_PREREQ(4,9)
-// https://gcc.gnu.org/bugzilla/show_bug.cgi?id=56019
-// gcc 4.8.x incorrectly placed max_align_t in the root namespace
-// Alias it into std (where it's found in 4.9 and later)
-namespace std { typedef ::max_align_t max_align_t; }
-#endif
-
 // portable version check for clang
 #ifndef __CLANG_PREREQ
 # if defined __clang__ && defined __clang_major__ && defined __clang_minor__