Re: [PATCH] D13946: Shrink DynTypedNode by one pointer from 40 to 32 bytes (on x86_64).

[Samuel Benzaquen via cfe-commits]

sbenza accepted this revision.
This revision is now accepted and ready to land.

================
Comment at: include/clang/AST/ASTTypeTraits.h:300
@@ -288,2 +299,3 @@
       if (ASTNodeKind::getFromNodeKind<T>().isBaseOf(NodeKind))
-        return cast<T>(*reinterpret_cast<BaseT *const *>(Storage));
+        return cast<T>(static_cast<const BaseT *>(
+            *reinterpret_cast<const void *const *>(Storage)));
----------------
maybe call getUnchecked() to avoid duplication?
Now it is getting complicated.


http://reviews.llvm.org/D13946



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits