LGTM with the dyn_cast mentioned inline changed.
|359 ↗||(On Diff #118644)|
Should this be a cast<>() rather than a dyn_cast<>()? Do you expect this to fail? If so then you should check the result for null. If not then the idiom is to use cast<>().
An alternative would be to change findMemberField() to return a FieldDecl.