Clean up C-style casts from pointers to void

Change-Id: Idff46e5642034cf751d557de9a49c02a7abf27bb

1 files changed, 3 insertions, 8 deletions

diff --git a/compilerplugins/clang/cstylecast.cxx b/compilerplugins/clang/cstylecast.cxx
index 048d2d59cd87..fcc1fc828862 100644
--- a/compilerplugins/clang/cstylecast.cxx
+++ b/compilerplugins/clang/cstylecast.cxx
@@ -114,14 +114,9 @@ bool CStyleCast::VisitCStyleCastExpr(const CStyleCastExpr * expr) {
     std::string incompFrom;
     std::string incompTo;
     if( expr->getCastKind() == CK_BitCast ) {
-        QualType t = resolvePointers(expr->getSubExprAsWritten()->getType());
-        // Ignore "safe" casts for now that involve casting from void* (and can
-        // thus not be interpreted as either a static_cast or a
-        // reinterpret_cast, with potentially different results):
-        if (t->isVoidType()) {
-            return true;
-        }
-        if (t->isIncompleteType()) {
+        if (resolvePointers(expr->getSubExprAsWritten()->getType())
+            ->isIncompleteType())
+        {
             incompFrom = "incomplete ";
         }
         if (resolvePointers(expr->getType())->isIncompleteType()) {