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

Serge Pavlov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jul 4 07:17:48 PDT 2018


sepavloff added a comment.

> We don't need 16-byte alignment on MSVC, we only need 8 because alignof(max_align_t) is 8 on that platform, and malloc is required to provide memory that's aligned to that boundary.

If allignment provided by malloc is sufficient, there is no need in runtime alignment. and `alignas(alignof(long double))` should be OK.


Repository:
  rL LLVM

https://reviews.llvm.org/D48889





More information about the llvm-commits mailing list