[PATCH] D33102: [clang] Implement -Wcast-qual for C++

David Blaikie via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Jun 5 14:45:06 PDT 2017


dblaikie accepted this revision.
dblaikie added a comment.
This revision is now accepted and ready to land.

I still feel like that's more testing than would be ideal (does the context of the cast matter? Wether it's dereferenced, a struct member access, assigned, initialized, etc - it doesn't look like it from the code, etc).

But sure. Could you also (manually, I guess) confirm that this matches GCC's cast-qual behavior (insofar as the warning fires in the same situations). If there are any deviations, let's chat about them.


Repository:
  rL LLVM

https://reviews.llvm.org/D33102





More information about the cfe-commits mailing list