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



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to