[clang] [LifetimeSafety] Implicit lifetimebound only for accessor methods of GSL owners (PR #174741)

[Gábor Horváth via cfe-commits]

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