[PATCH] D48889: [demangler] Ensure proper alignment in BumpPointerAllocator

Erik Pilkington via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jul 4 07:29:54 PDT 2018


erik.pilkington accepted this revision.
erik.pilkington added a comment.
This revision is now accepted and ready to land.

LGTM, thanks again! Please also commit this to the version of this file in libcxxabi/src/cxa_demangle.cpp.



================
Comment at: lib/Demangle/ItaniumDemangle.cpp:1953
 
-  alignas(16) char InitialBuffer[AllocSize];
+  alignas(alignof(long double)) char InitialBuffer[AllocSize];
   BlockMeta* BlockList = nullptr;
----------------
This can be spelt alignas(long double).


Repository:
  rL LLVM

https://reviews.llvm.org/D48889





More information about the llvm-commits mailing list