[PATCH] D37023: [analyzer] Fix bugreporter::getDerefExpr() again.
Gábor Horváth via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Mon Aug 28 03:48:08 PDT 2017
xazax.hun accepted this revision.
xazax.hun added a comment.
This revision is now accepted and ready to land.
Generally, it looks good to me. Though it looks like some of the cases covered in the code do not have corresponding tests (e.g.: the parenexprs).
I think this approach is good in a sense there should be such a cast at the interesting places. But I wonder if there could be some cases where we have such cast earlier than expected.
More information about the cfe-commits