[Lldb-commits] [PATCH] D65249: [NFC] use C++11 in AlignOf.h

Billy Robert O'Neal III via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Fri Jul 26 17:28:01 PDT 2019


BillyONeal added a comment.

In D65249#1603335 <https://reviews.llvm.org/D65249#1603335>, @jfb wrote:

> @rnk: how about I add a bit of code that wraps `aligned_storage` on all platforms except MSVC (where I'd implement it as Billy suggests). That would mean updating all the uses of `aligned_storage` to this LLVM one, but that's not a big deal.


Why not just always use the alignas version? Or even better, not involve a library for something you can say in the core language at all?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D65249/new/

https://reviews.llvm.org/D65249





More information about the lldb-commits mailing list