[PATCH] D64077: [ELF] Assert sizeof(SymbolUnion) <= 80

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jul 2 06:42:07 PDT 2019


MaskRay created this revision.
MaskRay added reviewers: grimar, peter.smith, ruiu.
Herald added subscribers: llvm-commits, arichardson, emaste.
Herald added a reviewer: espindola.
Herald added a project: LLVM.

Repository:
  rLLD LLVM Linker

https://reviews.llvm.org/D64077

Files:
  ELF/Symbols.h


Index: ELF/Symbols.h
===================================================================
--- ELF/Symbols.h
+++ ELF/Symbols.h
@@ -466,6 +466,7 @@
   static_assert(sizeof(T) <= sizeof(SymbolUnion), "SymbolUnion too small");
   static_assert(alignof(T) <= alignof(SymbolUnion),
                 "SymbolUnion not aligned enough");
+  static_assert(sizeof(SymbolUnion) <= 80, "SymbolUnion too large");
 };
 
 static inline void assertSymbols() {


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D64077.207549.patch
Type: text/x-patch
Size: 441 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20190702/af137905/attachment.bin>


More information about the llvm-commits mailing list