[PATCH] D12247: [libc++] remove possible trailing padding from aligned_storage

Eric Fiselier via cfe-commits cfe-commits at lists.llvm.org
Wed Aug 26 23:18:51 PDT 2015


EricWF added a comment.

I think I've misunderstood this patch the whole time. I didn't understand that it was undefined to access the trailing padding of the storage type. Am I correct in saying that this patch won't change the actual values of `sizeof(aligned_storage<N>::type)` and `sizeof(aligned_storage<N>::type)` for any `N`? I thought this was changing the type sizes but it's really just making trailing padding part of the actual type.

We probably still need part of my change for `std::function` but I also think we need this patch.


http://reviews.llvm.org/D12247





More information about the cfe-commits mailing list