[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.


https://reviews.llvm.org/D37023





More information about the cfe-commits mailing list