[PATCH] D38366: Fix a warning about casting a const pointer to void*

[Nikolai Bozhenov via Phabricator via cfe-commits]

n.bozhenov abandoned this revision.
n.bozhenov added a comment.

Seems that it has been just fixed by https://reviews.llvm.org/rL314424 :-)
Abandoning this review.


https://reviews.llvm.org/D38366



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

[PATCH] D38366: Fix a warning about casting a const pointer to void*

[Nikolai Bozhenov via Phabricator via cfe-commits]

n.bozhenov created this revision.

https://reviews.llvm.org/rL314336 introduced a new compile time warning about 
casting a const pointer to void*. Because of that the compiler fails to be 
built with `-Werror` flag. The suggested patch fixes the issue.


https://reviews.llvm.org/D38366

Files:
  clang/lib/AST/ExternalASTMerger.cpp


Index: clang/lib/AST/ExternalASTMerger.cpp
===
--- clang/lib/AST/ExternalASTMerger.cpp
+++ clang/lib/AST/ExternalASTMerger.cpp
@@ -212,7 +212,7 @@
 }
 if (!DidCallback && LoggingEnabled())
   logs() << "(ExternalASTMerger*)" << (void*)this
- << " asserting for (DeclContext*)" << (void*)DC
+ << " asserting for (DeclContext*)" << (const void*)DC
  << ", (ASTContext*)" << (void*)
  << "\n";
 assert(DidCallback && "Couldn't find a source context matching our DC");


Index: clang/lib/AST/ExternalASTMerger.cpp
===
--- clang/lib/AST/ExternalASTMerger.cpp
+++ clang/lib/AST/ExternalASTMerger.cpp
@@ -212,7 +212,7 @@
 }
 if (!DidCallback && LoggingEnabled())
   logs() << "(ExternalASTMerger*)" << (void*)this
- << " asserting for (DeclContext*)" << (void*)DC
+ << " asserting for (DeclContext*)" << (const void*)DC
  << ", (ASTContext*)" << (void*)
  << "\n";
 assert(DidCallback && "Couldn't find a source context matching our DC");
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits