https://github.com/Xazax-hun requested changes to this pull request.
Actually, I am not 100% convinced if this is the proper fix here. `shouldTrackImplicitObjectArg` only tells if the analysis should continue tracking the lifetime of that object. The original goal was to potentially drill through multiple view objects in case we find an owner (transitively). So I think the original code might be correct. Is there a false positive you are seeing as the result of this? https://github.com/llvm/llvm-project/pull/174741 _______________________________________________ cfe-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
