iitianpushkar wrote:
Hey @usx95 @NeKon69 , I have updated the patch based on the asked changes. All
checks have also passed. Please review it once and let me know if anything else
is needed.
For the suggested `return {Param, /*IsInferred=*/...};` form, I tried it
locally but it does not compile because the function returns
`std::optional<LifetimeBoundParamInfo>`,so, I kept
`LifetimeBoundParamInfo{...}` construction.
Also keeping the operator-> / field access diag wording for a separate
follow-up PR as suggested.
https://github.com/llvm/llvm-project/pull/220248
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
