[PATCH] D52613: [ADT] Change the `IntervalMap` alignment assert for x86 MSVC

Aleksandr Urakov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Oct 8 01:35:12 PDT 2018


aleksandr.urakov updated this revision to Diff 168620.
aleksandr.urakov added a comment.

Yes, you are right, it's more simple! I've updated the patch.


https://reviews.llvm.org/D52613

Files:
  include/llvm/ADT/IntervalMap.h


Index: include/llvm/ADT/IntervalMap.h
===================================================================
--- include/llvm/ADT/IntervalMap.h
+++ include/llvm/ADT/IntervalMap.h
@@ -964,7 +964,7 @@
 
 private:
   // The root data is either a RootLeaf or a RootBranchData instance.
-  AlignedCharArrayUnion<RootLeaf, RootBranchData> data;
+  LLVM_ALIGNAS(RootLeaf) AlignedCharArrayUnion<RootLeaf, RootBranchData> data;
 
   // Tree height.
   // 0: Leaves in root.


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D52613.168620.patch
Type: text/x-patch
Size: 462 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20181008/584f2563/attachment.bin>


More information about the llvm-commits mailing list