[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