[UBSan] Wrap Location variants in anonymous union

[BStott6]

(Addresses the FIXME)

[llvmbot]

@llvm/pr-subscribers-compiler-rt-sanitizer

Author: Benjamin Stott (BStott6)

Changes

(Addresses the FIXME)

---

Full diff: https://github.com/llvm/llvm-project/pull/168866.diff

1 Files Affected:

• (modified) compiler-rt/lib/ubsan/ubsan_diag.h (+5-4)

diff --git a/compiler-rt/lib/ubsan/ubsan_diag.h b/compiler-rt/lib/ubsan/ubsan_diag.h
index c836647c98f3c..d37bc309fdd0a 100644
--- a/compiler-rt/lib/ubsan/ubsan_diag.h
+++ b/compiler-rt/lib/ubsan/ubsan_diag.h
@@ -37,10 +37,11 @@ class Location {
 
 private:
   LocationKind Kind;
-  // FIXME: In C++11, wrap these in an anonymous union.
-  SourceLocation SourceLoc;
-  MemoryLocation MemoryLoc;
-  const SymbolizedStack *SymbolizedLoc;  // Not owned.
+  union {
+    SourceLocation SourceLoc;
+    MemoryLocation MemoryLoc;
+    const SymbolizedStack *SymbolizedLoc; // Not owned.
+  };
 
 public:
   Location() : Kind(LK_Null) {}

[github-actions]

🐧 Linux x64 Test Results

• 5824 tests passed
• 1319 tests skipped