[llvm] r284910 - Now that VS2013 is gone, make a memoryssa structure an anonymous union again

Daniel Berlin via llvm-commits llvm-commits at lists.llvm.org
Fri Oct 21 21:15:41 PDT 2016


Author: dannyb
Date: Fri Oct 21 23:15:41 2016
New Revision: 284910

URL: http://llvm.org/viewvc/llvm-project?rev=284910&view=rev
Log:
Now that VS2013 is gone, make a memoryssa structure an anonymous union again

Modified:
    llvm/trunk/lib/Transforms/Utils/MemorySSA.cpp

Modified: llvm/trunk/lib/Transforms/Utils/MemorySSA.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Transforms/Utils/MemorySSA.cpp?rev=284910&r1=284909&r2=284910&view=diff
==============================================================================
--- llvm/trunk/lib/Transforms/Utils/MemorySSA.cpp (original)
+++ llvm/trunk/lib/Transforms/Utils/MemorySSA.cpp Fri Oct 21 23:15:41 2016
@@ -144,10 +144,10 @@ public:
   }
 
 private:
-  // FIXME: MSVC 2013 does not properly implement C++11 union rules, once we
-  // require newer versions, this should be made an anonymous union again.
-  ImmutableCallSite CS;
-  MemoryLocation Loc;
+  union {
+      ImmutableCallSite CS;
+      MemoryLocation Loc;
+  };
 };
 }
 




More information about the llvm-commits mailing list